Skip to content

Migrate tests to using ID4SLAB2, Fixes AB#3525444#2471

Open
fadidurah wants to merge 51 commits into
devfrom
fadi/lab-migrate
Open

Migrate tests to using ID4SLAB2, Fixes AB#3525444#2471
fadidurah wants to merge 51 commits into
devfrom
fadi/lab-migrate

Conversation

@fadidurah
Copy link
Copy Markdown
Collaborator

@fadidurah fadidurah commented Feb 23, 2026

Update all ui automation test cases to work with the ID4SLAB2 resources

  • Update configurations to point ID4SLAB2 client ids and applications that replace what was previously accessed in lab4
  • Remove all usage of LabQuery
  • Remove all usage of RetryOnFailure, reduces clutter since we force a minimum of 3 retries anyway, and value below 3 placed in RetryOnFailure is pointless

AB#3525444

@fadidurah fadidurah requested a review from a team as a code owner February 23, 2026 07:06
Copilot AI review requested due to automatic review settings February 23, 2026 07:06
@fadidurah fadidurah requested a review from a team as a code owner February 23, 2026 07:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates test configurations to use new client IDs from the ID4SLAB2 environment, updating authentication settings across test applications and documentation.

Changes:

  • Updated client IDs in configuration files for AAD scenarios (from 4b0db8c2-9f26-4417-8bde-3f0e3656f8e0 to c6bb302a-1e38-408e-9754-87c18fe81c80)
  • Updated client IDs in configuration files for MSA scenarios (from 9668f2bd-6103-4292-9024-84fa2d1b6fb2 to 700c5153-ef50-4176-8d7a-4131d649020e)
  • Updated the common submodule reference

Reviewed changes

Copilot reviewed 39 out of 39 changed files in this pull request and generated no comments.

Show a summary per file
File Description
testapps/testapp/src/main/res/raw/*.json (14 files) Updated client IDs in test app MSAL configuration files to use new ID4SLAB2 values
msalautomationapp/src/main/res/raw/*.json (11 files) Updated client IDs in automation app MSAL configuration files to use new ID4SLAB2 values
msalautomationapp/src/androidTest/java/.../NestedAppHelper.java Updated nested app client ID constant to match new MSA client ID
msal/src/test/res/raw/*.json (6 files) Updated client IDs in unit test configuration files
msal/src/androidTest/res/raw/*.json (4 files) Updated client IDs in Android instrumentation test configuration files
docs/access_token-pop.md Updated example client ID in PoP documentation
common Updated submodule reference to include corresponding changes

@github-actions
Copy link
Copy Markdown

❌ Work item link check failed. Description does not contain AB#{ID}.

Click here to Learn more.

@fadidurah fadidurah added the No-Changelog This change does not update the changelog. label Mar 18, 2026
@github-actions
Copy link
Copy Markdown

✅ Work item link check complete. Description contains link AB#3525444 to an Azure Boards work item.

@github-actions github-actions Bot changed the title Migrate tests to using ID4SLAB2 Migrate tests to using ID4SLAB2, Fixes AB#3525444 Mar 18, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 143 out of 143 changed files in this pull request and generated 7 comments.

Comment thread msalautomationapp/src/main/res/raw/msal_config_default.json
Comment thread msalautomationapp/src/main/res/raw/msal_config_msidlab4.json
fadidurah and others added 3 commits April 28, 2026 23:12
…work/MultiAccountAndResourceAcquireTokenNetworkTests.java

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 144 out of 144 changed files in this pull request and generated 2 comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs msal No-Changelog This change does not update the changelog. test-automation testapps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants