Skip to content

Conversation

@jonfortescue
Copy link
Contributor

To double check:

Attempt to fix #9864 by switching to use exec and moving it up prior to running tests.

@jonfortescue jonfortescue self-assigned this Jul 7, 2022
alexperovich
alexperovich previously approved these changes Jul 7, 2022
@jonfortescue jonfortescue merged commit 15af466 into dotnet:main Jul 13, 2022
@jonfortescue jonfortescue deleted the fr9864 branch July 13, 2022 20:40
@rainersigwald
Copy link
Member

Belated but the exec of the actual tests is going to overwrite the command line you wrote unless you change this to >>:

<_TestRunnerCommand Condition="'$(TestCaptureOutput)' != 'false'">$(_TestRunnerCommand) > "%(TestToRun.ResultsStdOutPath)" 2>&amp;1</_TestRunnerCommand>

For local test execution you probably also want to use overwrite > for the very first line, or you'll get multiple copies of the log if you do build -test twice.

@jonfortescue jonfortescue mentioned this pull request Jul 15, 2022
1 task
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.

Runtime tests regularly hitting file locking issues in CI when writing test output

5 participants