-
-
Notifications
You must be signed in to change notification settings - Fork 58
chore: Updated the testing environment for local development #2480
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
Conversation
| <Delete Files="$(UnityTestPlayModeResultFilePath)" /> | ||
| <Exec EnvironmentVariables="IgnoreExitCode=true" Command="pwsh "$(RepoRoot)scripts/unity.ps1" $(UnityExec) -batchmode -nographics -runTests -testPlatform PlayMode -projectPath $(UnitySampleProjectPath) -testResults $(UnityTestPlayModeResultFilePath)" /> | ||
| <UnityTestResults Path="$(UnityTestPlayModeResultFilePath)" /> | ||
| <Exec Command="pwsh "$(RepoRoot)scripts/report-test-results.ps1" "$(UnityTestPlayModeResultFilePath)"" /> |
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.
This now reuses the script that parses the test result for local development.
| <Delete Files="$(UnityTestEditModeResultFilePath)" /> | ||
| <Exec EnvironmentVariables="IgnoreExitCode=true" IgnoreStandardErrorWarningFormat="true" Command="pwsh "$(RepoRoot)scripts/unity.ps1" $(UnityExec) -batchmode -nographics -runTests -testPlatform EditMode -projectPath $(UnitySampleProjectPath) -testResults $(UnityTestEditModeResultFilePath)"/> | ||
| <UnityTestResults Path="$(UnityTestEditModeResultFilePath)" /> | ||
| <Exec Command="pwsh "$(RepoRoot)scripts/report-test-results.ps1" "$(UnityTestEditModeResultFilePath)"" /> |
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.
This now reuses the script that parses the test result for local development.
| </Task> | ||
| </UsingTask> | ||
|
|
||
| <!-- Parse test results --> |
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.
This now happens in report-test-results.ps1
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. This PR will not appear in the changelog. 🤖 This preview updates automatically when you update the PR. |
Problem
The SDK was lacking an easy to use way to run unit tests from commandline.
Proposal
Have a
scripts/run-tests.ps1that allows for running and parsing unit tests from commandline.#skip-changelog