ai comments

Author: fadidurah

common

@@ -23,8 +23,6 @@
 package com.microsoft.identity.client.e2e.tests.network;
 
 import com.microsoft.identity.client.e2e.tests.IAcquireTokenTest;
-import com.microsoft.identity.internal.testutils.labutils.LabConstants;
-import com.microsoft.identity.internal.testutils.labutils.LabUserQuery;
 import com.microsoft.identity.labapi.utilities.constants.UserType;
 
 public interface IAcquireTokenNetworkTest extends IAcquireTokenTest {

msal/src/test/java/com/microsoft/identity/client/e2e/tests/network/IAcquireTokenNetworkTest.java

@@ -101,7 +101,7 @@ public void test_1592465_CrossCloud_CrossCloudATThenHomeCloudAT() throws Throwab
         // We expect that this does not prompt for credentials
         final OnInteractionRequired crossCloudInteractionHandler = () -> { };
         final MsalAuthResult acquireTokenCrossCloudResult = msalSdk.acquireTokenInteractive(acquireTokenCrossCloudAuthParams, crossCloudInteractionHandler, TokenRequestTimeout.SILENT);
-        Assert.assertFalse("Verify accessToken is empty", TextUtils.isEmpty(acquireTokenCrossCloudResult.getAccessToken()));
+        Assert.assertFalse("Verify accessToken is not empty", TextUtils.isEmpty(acquireTokenCrossCloudResult.getAccessToken()));
 
         Assert.assertNotEquals("CrossCloud request gets new access token", acquireTokenCrossCloudResult.getAccessToken(), acquireTokenHomeCloudResult.getAccessToken());
     }

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/broker/crosscloud/TestCase1592465.java

@@ -86,7 +86,7 @@ public void handleUserInteraction() {
 
         authResult.assertSuccess();
 
-        // Change the policy to MAM_CA
+        // Change the policy to GLOBAL_MFA, any policy will work as long as it blocks silent request
         mLabClient.enablePolicy(username, ProtectionPolicy.GLOBAL_MFA);
 
         // It takes some time for the policy change to reflect