Delay E2E tests until other test projects have built #985
Conversation
This file contains hidden or 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
Contributor
Author
|
/azp run |
alisonlomaka
approved these changes
Mar 19, 2025
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.
When running
dotnet testfrom the solution folder, the tests in theMicrosoft.Sbom.Tool.Testsproject will sometimes fail due to file collisions. Upon investigation, the files are question are output files of other projects, and are occurring because the tests in theMicrosoft.Sbom.Tool.Testsare running while the other projects are still building.To address this, I used Visual Studio's Project Dependencies feature to mark all of the other test projects as upstream dependencies of the
Microsoft.Sbom.Tool.Testsproject. This is the same pattern that was used in #957, but this time we add all test projects to be sure thatMicrosoft.Sbom.Tool.Testsis the very last project to build, making it impossible to run the tests while other projects are still building. Here's a screenshot of the Project Build Order as displayed by Visual Studio with this change: