Skip to content

Commit f53cb89

Browse files
authored
July Ui maintenance (#1870)
July UI Maintenance, some cross cloud tests have a second speed bump, 2 MAM_CA tests fail on daily build due to version check (our daily versions start with 0.0, seems ESTS has some check to block sign ins on these versions for MAM_CA, to fix, we'd have to bump our daily versions past 0.0) Pre-requisite: https://github.com/AzureAD/microsoft-authentication-library-common-for-android/compare/fadi/july-ui?expand=1
1 parent c5f5f45 commit f53cb89

File tree

12 files changed

+14
-7
lines changed

12 files changed

+14
-7
lines changed

common

Submodule common updated 19 files

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/broker/flw/TestCase833513.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@
5757
// End My Shift - In Shared device mode, only account from the same tenant should be able to acquire token.
5858
// https://identitydivision.visualstudio.com/DevEx/_workitems/edit/833513
5959
@SupportedBrokers(brokers = {BrokerMicrosoftAuthenticator.class, BrokerHost.class})
60-
@RetryOnFailure(retryCount = 2)
6160
@RunOnAPI29Minus("Checking for text error in WebView")
6261
public class TestCase833513 extends AbstractMsalBrokerTest {
6362

@@ -117,7 +116,8 @@ public void test_833513() throws MsalException, InterruptedException, LabApiExce
117116
.broker(mBroker)
118117
.prompt(PromptParameter.SELECT_ACCOUNT)
119118
.expectingBrokerAccountChooserActivity(false)
120-
.howWouldYouLikeToSignInExpected(false)
119+
.howWouldYouLikeToSignInExpected(true)
120+
.chooseCertificateExpected(true)
121121
.build();
122122

123123
AdfsPromptHandler adfsPromptHandler = new AdfsPromptHandler(promptHandlerParameters);

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/broker/joined/TestCase796050.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import com.microsoft.identity.client.msal.automationapp.sdk.MsalSdk;
3232
import com.microsoft.identity.client.msal.automationapp.testpass.broker.AbstractMsalBrokerTest;
3333
import com.microsoft.identity.client.ui.automation.TestContext;
34+
import com.microsoft.identity.client.ui.automation.annotations.FailsWithDailyVersions;
3435
import com.microsoft.identity.client.ui.automation.annotations.RetryOnFailure;
3536
import com.microsoft.identity.client.ui.automation.interaction.OnInteractionRequired;
3637
import com.microsoft.identity.client.ui.automation.TokenRequestTimeout;
@@ -55,6 +56,7 @@
5556
// "Add another account" option in Account Chooser Activity
5657
// https://identitydivision.visualstudio.com/DevEx/_workitems/edit/796050
5758
@RetryOnFailure(retryCount = 2)
59+
@FailsWithDailyVersions("ESTS Seems to have a check against 0.0.X versions for MAM CA")
5860
public class TestCase796050 extends AbstractMsalBrokerTest {
5961

6062
@Test

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/broker/joined/TestCase832430.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import com.microsoft.identity.client.msal.automationapp.testpass.broker.AbstractMsalBrokerTest;
3232
import com.microsoft.identity.client.ui.automation.TestContext;
3333
import com.microsoft.identity.client.ui.automation.TokenRequestTimeout;
34+
import com.microsoft.identity.client.ui.automation.annotations.FailsWithDailyVersions;
3435
import com.microsoft.identity.client.ui.automation.annotations.RetryOnFailure;
3536
import com.microsoft.identity.client.ui.automation.interaction.PromptHandlerParameters;
3637
import com.microsoft.identity.client.ui.automation.interaction.PromptParameter;
@@ -46,6 +47,7 @@
4647
// [Joined][MSAL] Acquire Token + Acquire Token Silent with resource (Prompt.SELECT_ACCOUNT)
4748
// https://identitydivision.visualstudio.com/DevEx/_workitems/edit/832430
4849
@RetryOnFailure(retryCount = 2)
50+
@FailsWithDailyVersions("ESTS Seems to have a check against 0.0.X versions for MAM CA")
4951
public class TestCase832430 extends AbstractMsalBrokerTest {
5052

5153
@Test

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/broker/mdm/TestCase833526.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@
4343
import com.microsoft.identity.labapi.utilities.constants.ProtectionPolicy;
4444
import com.microsoft.identity.labapi.utilities.constants.TempUserType;
4545

46-
import org.junit.Ignore;
4746
import org.junit.Test;
4847

4948
import java.util.Arrays;

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/broker/mwpj/TestCase2579654.kt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ import com.microsoft.identity.client.msal.automationapp.sdk.MsalSdk
3131
import com.microsoft.identity.client.msal.automationapp.testpass.broker.AbstractMsalBrokerTest
3232
import com.microsoft.identity.client.ui.automation.TokenRequestTimeout
3333
import com.microsoft.identity.client.ui.automation.annotations.LocalBrokerHostDebugUiTest
34+
import com.microsoft.identity.client.ui.automation.annotations.RetryOnFailure
3435
import com.microsoft.identity.client.ui.automation.annotations.SupportedBrokers
3536
import com.microsoft.identity.client.ui.automation.broker.BrokerHost
3637
import com.microsoft.identity.client.ui.automation.interaction.PromptParameter
@@ -53,6 +54,7 @@ import org.junit.rules.TestRule
5354
// [MWPJ] After entry migration PRT is still usable without extra prompts.
5455
@LocalBrokerHostDebugUiTest
5556
@SupportedBrokers(brokers = [BrokerHost::class])
57+
@RetryOnFailure
5658
class TestCase2579654 : AbstractMsalBrokerTest() {
5759

5860
private lateinit var mUsGovAccount: ILabAccount

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/msalonly/basic/TestCase497038.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ public void handleUserInteraction() {
112112
.sessionExpected(true)
113113
.consentPageExpected(false)
114114
.speedBumpExpected(true)
115-
.speedBumpResponse(UiResponse.ACCEPT)
116115
.build();
117116

118117
new AadPromptHandler(promptHandlerParameters)

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/msalonly/basic/TestCase99267.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
// Interactive Auth with select_account (no consent record)
4747
// https://identitydivision.visualstudio.com/DefaultCollection/IDDP/_workitems/edit/99267
4848
@RunOnAPI29Minus("Consent Page")
49-
@RetryOnFailure(retryCount = 2)
49+
@RetryOnFailure
5050
public class TestCase99267 extends AbstractMsalUiTest {
5151

5252
@Test

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/msalonly/basic/TestCase99274.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
// Interactive Auth with select_account (with consent record)
4747
// https://identitydivision.visualstudio.com/DefaultCollection/IDDP/_workitems/edit/99274
4848
@RunOnAPI29Minus("Consent Page")
49-
@RetryOnFailure
49+
@RetryOnFailure(retryCount = 2)
5050
public class TestCase99274 extends AbstractMsalUiTest {
5151

5252
@Test

msalautomationapp/src/androidTest/java/com/microsoft/identity/client/msal/automationapp/testpass/msalonly/crosscloud/TestCase1420484.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ public void test_1420484() throws Throwable {
9797
.loginHint(userName)
9898
.staySignedInPageExpected(GlobalConstants.IS_STAY_SIGN_IN_PAGE_EXPECTED)
9999
.speedBumpExpected(true)
100+
.secondSpeedBumpExpected(GlobalConstants.IS_EXPECTING_SECOND_SPEED_BUMP)
100101
.build();
101102
final AadPromptHandler promptHandler = new AadPromptHandler(promptHandlerParameters);
102103
promptHandler.handlePrompt(userName, password);

0 commit comments

Comments
 (0)