Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

17 LTW broker test cases #1867

Merged
merged 38 commits into from
Nov 3, 2023
Merged

17 LTW broker test cases #1867

merged 38 commits into from
Nov 3, 2023

Conversation

amoyixi
Copy link
Contributor

@amoyixi amoyixi commented Jul 31, 2023

This pull request is about 17 test cases related to LTW broker, including:
Test Case 2517374: Add a UI test on MsalTestApp
Test Case 2517381: Add a UI testcase with update scenarios on OneAuthTest and MsalTest apps
Test Case 2571345: Samsung GA Coverage
Test Case 2571361: If Company Portal is installed after LTW, user should still get SSO
Test Case 2571508: If LTW without broker is installed, updated MSAL should still get SSO
Test Case 2571565: Uninstall scenario - with old Auth app new LTW
Test Case 2571580: Legacy auth app, updated LTW and uninstall LTW
Test Case 2572249: Updated LTW, Updated Auth app and uninstall LTW
Test Case 2572280: LTW has higher priority than CP
Test Case 2572283: LTW has higher priority than CP - Case2
Test Case 2572294: Authenticator has highest priority - Case1 (LTW, Auth, CP)
Test Case 2584409: Authenticator has highest priority - Case2 (LTW, CP, Auth)
Test Case 2584410: Authenticator has highest priority - Case3 (Auth, LTW, CP)
Test Case 2584411: Authenticator has highest priority - Case4 (Auth, CP, LTW)
Test Case 2584412: Authenticator has highest priority - Case5 (CP, Auth, LTW)
Test Case 2584414: Authenticator has highest priority - Case6 (CP, LTW, Auth)
Test Case 2579095: Even Authenticator has the highest priority, if CP already has an artifact, CP will remain the broker.

@amoyixi amoyixi requested review from somalaya and fadidurah July 31, 2023 02:53
@amoyixi amoyixi requested a review from a team as a code owner July 31, 2023 02:53
somalaya
somalaya reviewed Aug 29, 2023
View reviewed changes
...va/com/microsoft/identity/client/msal/automationapp/testpass/broker/ltw/TestCase2571345.java Show resolved Hide resolved
...va/com/microsoft/identity/client/msal/automationapp/testpass/broker/ltw/TestCase2571565.java Show resolved Hide resolved
...va/com/microsoft/identity/client/msal/automationapp/testpass/broker/ltw/TestCase2571565.java Show resolved Hide resolved
...va/com/microsoft/identity/client/msal/automationapp/testpass/broker/ltw/TestCase2571580.java Outdated Show resolved Hide resolved
...va/com/microsoft/identity/client/msal/automationapp/testpass/broker/ltw/TestCase2572249.java Show resolved Hide resolved
...va/com/microsoft/identity/client/msal/automationapp/testpass/broker/ltw/TestCase2579095.java Outdated Show resolved Hide resolved
...va/com/microsoft/identity/client/msal/automationapp/testpass/broker/ltw/TestCase2584410.java Show resolved Hide resolved
@amoyixi amoyixi requested review from fadidurah and somalaya August 29, 2023 15:41
@amoyixi amoyixi merged commit efc52ff into dev Nov 3, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fadidurah fadidurah fadidurah approved these changes

@somalaya somalaya somalaya approved these changes

Assignees
No one assigned
Labels
test-automation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@amoyixi @somalaya @fadidurah @yixilin09 @cikelengfeng