Page MenuHomePhabricator

Fix a bug where the TeamCity build only reports one of the test_runner runs

Authored by jasonbcox on Jul 17 2019, 21:48.



The junit output of each successive run overwrites junit_results.xml.
This leads to a build failure, but tests reporting as passed:
It also leads to incorrect test failure reporting, making it impossible to track flaky or spurious test failures.

This diff fixes that by writing each run to it's own output file.

Test Plan

See the TeamCity build for this diff:
In the "View:" dropdown, select tests without grouping by name

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jasonbcox created this revision.Jul 17 2019, 21:48
jasonbcox planned changes to this revision.Jul 17 2019, 21:48
jasonbcox edited the summary of this revision. (Show Details)Jul 17 2019, 21:56
jasonbcox requested review of this revision.Jul 17 2019, 22:08
jasonbcox edited the test plan for this revision. (Show Details)
deadalnix accepted this revision.Jul 17 2019, 23:27

ju is probably about the worse name possible.

90 ↗(On Diff #10340)

Why is there a cutoff on master ?

This revision is now accepted and ready to land.Jul 17 2019, 23:27
jasonbcox added inline comments.Jul 18 2019, 01:25
90 ↗(On Diff #10340)

There doesn't seem to be a rationale for it here: but it's clearly intended to run extended tests.