microsoft-authentication-library-for-android icon indicating copy to clipboard operation
microsoft-authentication-library-for-android copied to clipboard

Add multi cloud DCF test to cover ICM

Open rpdome opened this issue 3 years ago • 1 comments

  1. Init PCA in WW
  2. Init PCA in USGov
  3. both PCA should return URL from its own cloud.

https://portal.microsofticm.com/imp/v3/incidents/details/325344544/home

"Another observation from our local repro. We have done two tests:

  1. Start with public cloud in DCF and then switch to a GCC-H cloud.
  2. Start with GCC-H cloud, then switch to public cloud and then switch to GCC-H cloud.

For the first test, we get the user code for global endpoint for both times. While in the second test, we can successfully get the user code for us gov endpoint, while getting the user code for us gov endpoint in latter two times."

rpdome avatar Aug 08 '22 21:08 rpdome

Codecov Report

Merging #1674 (e0f420c) into dev (8942e56) will increase coverage by 0.11%. The diff coverage is n/a.

:exclamation: Current head e0f420c differs from pull request most recent head 895da9d. Consider uploading reports for the commit 895da9d to get more accurate results

@@             Coverage Diff              @@
##                dev    #1674      +/-   ##
============================================
+ Coverage     49.37%   49.48%   +0.11%     
- Complexity      372      373       +1     
============================================
  Files            59       59              
  Lines          2708     2708              
  Branches        333      333              
============================================
+ Hits           1337     1340       +3     
+ Misses         1227     1224       -3     
  Partials        144      144              
Impacted Files Coverage Δ
...osoft/identity/client/PublicClientApplication.java 44.14% <0.00%> (+0.48%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Aug 08 '22 22:08 codecov-commenter