Adding CIAM E2E tests and improvements to build infrastructure #2753
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Refactoring
* Adding CIAM CUD tests * Updating comments * Updating test --------- Co-authored-by: trwalke <[email protected]>
This reverts commit c3f3d3a.
This reverts commit 5f2a384.
This reverts commit 7484c58.
…nto trwalke/UiTestingCiam
trwalke
changed the title
added 2 commits
April 12, 2024 01:22
bgavrilMS
commented
Apr 12, 2024
bgavrilMS
commented
Apr 12, 2024
bgavrilMS
commented
Apr 12, 2024
bgavrilMS
commented
Apr 12, 2024
|
I can't approve it because I opened the PR. Signing off. |
This reverts commit b7622cd.
I can approve for you :). Still need a second approval |
trwalke
approved these changes
Apr 12, 2024
jmprieur
requested changes
Apr 13, 2024
trwalke
added a commit
that referenced
this pull request
Apr 17, 2024
* Adding web app and web app call web api CIAM non CUD tests * Adding CIAM deamon test app for non CUD domain * Integrating lab client apps Refactoring * Refactoring * Setting ui test to headless * Update WebAppCallsApiCallsGraphLocally.cs * Adding CIAM CUD integration tests (#2737) * Adding CIAM CUD tests * Updating comments * Updating test --------- Co-authored-by: trwalke <[email protected]> * Enabling test runs with VSTest * Test updates * test * Testing * Disabling test * Test * Updating the CIAM tests to use the lab API (manual testing) * Revert "Test" This reverts commit c3f3d3a. * Revert "Disabling test" This reverts commit 5f2a384. * Using hosted agents * Always publish traces * Revert "Using hosted agents" This reverts commit 7484c58. * Updating process error * Publish screenshots * Updating tests * Adding additional statements * Updating screenshot staging * Fix typo * Updating endpoint to use Http * Reconfiguring ports * hiding UI * Enabling web app tests * Resolving test issue * Fixing run time and hiding ui * Adding asp auth * Enabling test * Refactoring test YAML * Disabling restore * Fixing build task * Test * Test * Fixing parameters * Unit tests * Install test platform * Changing test runner * Updating test directories * Updating test run * Fix typo * Updating test run * Updating check * Refactoring process restart logic * Clean up YAML * Revert "Adding asp auth" This reverts commit b7622cd. * Refactoring * Enabling code coverage settings * Fix indentation --------- Co-authored-by: trwalke <[email protected]> Co-authored-by: Travis Walker <[email protected]> Co-authored-by: Jean-Marc Prieur <[email protected]>
jennyf19
pushed a commit
that referenced
this pull request
Apr 17, 2024
#2769) * Adding web app and web app call web api CIAM non CUD tests * Adding CIAM deamon test app for non CUD domain * Integrating lab client apps Refactoring * Refactoring * Setting ui test to headless * Update WebAppCallsApiCallsGraphLocally.cs * Adding CIAM CUD integration tests (#2737) * Adding CIAM CUD tests * Updating comments * Updating test --------- * Enabling test runs with VSTest * Test updates * test * Testing * Disabling test * Test * Updating the CIAM tests to use the lab API (manual testing) * Revert "Test" This reverts commit c3f3d3a. * Revert "Disabling test" This reverts commit 5f2a384. * Using hosted agents * Always publish traces * Revert "Using hosted agents" This reverts commit 7484c58. * Updating process error * Publish screenshots * Updating tests * Adding additional statements * Updating screenshot staging * Fix typo * Updating endpoint to use Http * Reconfiguring ports * hiding UI * Enabling web app tests * Resolving test issue * Fixing run time and hiding ui * Adding asp auth * Enabling test * Refactoring test YAML * Disabling restore * Fixing build task * Test * Test * Fixing parameters * Unit tests * Install test platform * Changing test runner * Updating test directories * Updating test run * Fix typo * Updating test run * Updating check * Refactoring process restart logic * Clean up YAML * Revert "Adding asp auth" This reverts commit b7622cd. * Refactoring * Enabling code coverage settings * Fix indentation --------- Co-authored-by: Bogdan Gavril <[email protected]> Co-authored-by: trwalke <[email protected]> Co-authored-by: Jean-Marc Prieur <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adding CIAM E2E tests and improvements to build infrastructure
Summary of the changes (Less than 80 chars)
Description
CIAM tests
Webapp calls web api local
Web app calls web api (graph) disabled on the ADO for now. will be revisited(The entire file is disabled, not just this test)
Improvements to the YAML files
ID Web now has separate tasks for Building and running tests (One for unit tests, integration tests and E2E tests)
Separating these tasks should make it easier to onboard to Onebranch if we run into issues getting E2E tests running on the OneBranch agents.
ID Web can now leverage the test rerun feature in the VSTest ADO task to rerun failed tests. This will help improve overall test reliability.
Added logic to enable E2E tests to attempt a restart process restart a few times when it crashes prematurely. This should further improve reliability for the E2E tests
Screenshots are now captured when E2E Ui tests fail and are uploaded to the build run.
Fixes #{bug number} (in this specific format)
Continuation of #2728