Skip to content

Commit 922263e

Browse files
committed
[MERGE #2938 @dilijev] Fix summary.log by appending to file instead of overwriting it.
Merge pull request #2938 from dilijev:fix-test-summary For a while now, summary.log has been empty in CI artifacts when unit tests fail. Turns out we were overwriting the file with native test results instead of appending to it.
2 parents 4332106 + 7194222 commit 922263e

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

test/jenkins.testall.cmd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ set _HadFailures=0
102102

103103
pushd %_TestDir%\logs
104104
findstr /sp failed rl.results.log > summary.log
105-
findstr /sip failed nativetests.log > summary.log
105+
findstr /sip failed nativetests.log >> summary.log
106106
rem Echo to stderr so that VSO includes the output in the build summary
107107
type summary.log 1>&2
108108
popd

test/jenkins.testone.cmd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ set _HadFailures=0
9898

9999
pushd %_LogDir%
100100
findstr /sp failed rl.results.log > summary.log
101-
findstr /sip failed nativetests.log > summary.log
101+
findstr /sip failed nativetests.log >> summary.log
102102
rem Echo to stderr so that VSO includes the output in the build summary
103103
type summary.log 1>&2
104104
popd

test/runcitests.cmd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ set _HadFailures=0
158158

159159
pushd %_TestDir%\logs
160160
findstr /sp failed rl.results.log > %1
161-
findstr /sip failed nativetests.log > %1
161+
findstr /sip failed nativetests.log >> %1
162162
rem Echo to stderr so that VSO includes the output in the build summary
163163
type %1 1>&2
164164
popd

0 commit comments

Comments
 (0)