Skip to content

chore: local test running improvements - #2809

Open
bitsandfoxes wants to merge 2 commits into
mainfrom
chore/testing-improvements
Open

chore: local test running improvements#2809
bitsandfoxes wants to merge 2 commits into
mainfrom
chore/testing-improvements

Conversation

@bitsandfoxes

Copy link
Copy Markdown
Contributor

#skip-changelog


if ($testPlatform -eq "playmode") {
$started = Invoke-UnityPipelineCommand -ProjectPath $projectPath -Command "run_tests" -CommandArguments ($commandArguments + @("--async_tests", "true"))
$started = Invoke-UnityPipelineCommand -ProjectPath $projectPath -Command "run_tests" -CommandArguments ($commandArguments + @("--async_tests", "true")) -TimeoutSeconds ($timeoutSeconds + 30)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Timeout was missing.


MonoBehaviour.StopCoroutine(_updateUiStatusCoroutine);
_updateUiStatusCoroutine = null;
if (_updateUiStatusCoroutine is not null)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not a real but but following the established pattern from below. If triggering the putting-on-background twice this would throw. It's a non-existing issue on actual builds in an already deprecated feature.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant