Page MenuHomePhabricator

[CI] Enable Junit reporting via the build system and use it in scripts
ClosedPublic

Authored by Fabien on Jul 28 2020, 15:48.

Details

Summary

This diff adds a --junit option to the build_cmake.sh script, which
is a shortcut to enable junit reports. This option is set appropriately
by the build scripts as well as the ninja -k0 flag that cause the
tests to continue in case of failure until dependencies can no longer be
met, so we gather as much result as possible in the report. This also
makes it possible to get rid of the hacky run_test_bitcoin wrapper
from the ci fixture.

The Junit reports are added to the artifacts and submitted to Teamcity.

Test Plan

Run the CI build plans.

Event Timeline

Fabien requested review of this revision.Jul 28 2020, 15:48
Fabien planned changes to this revision.Jul 28 2020, 16:20

Update importdata message.
Since the doc contradicts itself some more iterations are expected...

Fabien planned changes to this revision.Jul 28 2020, 16:29

Duplicated junit/ in path

Fabien planned changes to this revision.Jul 28 2020, 16:49

Try to send the message before copying the result file, as Teamcity doc says it monitors for the files to appear

Fabien planned changes to this revision.Jul 28 2020, 18:00

Make the path relative to the checkout directory

Fabien planned changes to this revision.Jul 28 2020, 19:02

Use version from the artifact dir.

Fabien planned changes to this revision.Jul 28 2020, 19:33
Fabien requested review of this revision.Jul 28 2020, 19:56

The path is finally what Teamcity expects. You can see the result here.

This revision is now accepted and ready to land.Jul 28 2020, 21:12