-
Notifications
You must be signed in to change notification settings - Fork 703
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.
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
Fixes #3930 - Splits tests to Tests/UnitTests
, Tests/IntegrationTests
, Tests/StressTests
#3954
base: v2_develop
Are you sure you want to change the base?
Conversation
Another result (after tyring to fix stuff):
|
Tests/UnitTests
, Tests/IntegrationTests
, Tests/StressTests
Moved all obviously parallelizable tests. Updated yml.
…3952_Test_Failures
…3952_Test_Failures
…3952_Test_Failures
Perhaps this refactoring has hidden the root bug that was causing the crash, but I've not gotten a failure in 3 runs. Please review so I can merge and we can leverage all of this. THanks. |
Create link to the file TestHelpers.
…3952_Test_Failures
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonderful implementations with even more automation and shared resources. Thanks.
More coming.. I got ambitious... |
Ok. Take another look. There may be still some tests duplicated between The less tests we have in |
Fixes
Proposed Changes/Todos
Tests/UnitTests
,Tests/IntegrationTests
,Tests/StressTests
Directory.Build.props
to unify settingsDirecotry.Packages.props
to unify package versions./Tests/UnitTestsParallelizable
project and move obviously parallelizable tests.Pull Request checklist:
CTRL-K-D
to automatically reformat your files before committing.dotnet test
before commit///
style comments)