Page MenuHomePhabricator

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

Authored by Fabien on Tue, Jul 28, 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.

Diff Detail

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

Event Timeline

Fabien created this revision.Tue, Jul 28, 15:48
Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Jul 28, 15:48
Fabien requested review of this revision.Tue, Jul 28, 15:48
Fabien planned changes to this revision.Tue, Jul 28, 16:20
Fabien updated this revision to Diff 22585.Tue, Jul 28, 16:29

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

Fabien planned changes to this revision.Tue, Jul 28, 16:29
Fabien updated this revision to Diff 22586.Tue, Jul 28, 16:49

Duplicated junit/ in path

Fabien planned changes to this revision.Tue, Jul 28, 16:49
Fabien updated this revision to Diff 22590.Tue, Jul 28, 17:54

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.Tue, Jul 28, 18:00
Fabien updated this revision to Diff 22592.Tue, Jul 28, 19:01

Make the path relative to the checkout directory

Fabien planned changes to this revision.Tue, Jul 28, 19:02
Fabien updated this revision to Diff 22593.Tue, Jul 28, 19:33

Use version from the artifact dir.

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

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

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