Page MenuHomePhabricator

[CI] Run the check target instead of a list of subtargets
ClosedPublic

Authored by Fabien on Jun 29 2020, 07:28.

Details

Summary

This will cause test_bitcoin to run twice when the run_test_bitcoin
facility is used, but this should not cause any noticeable time penalty
since the tests run in parallel with ninja check. As a reference the
whole ninja check run takes about 5s on my machine.

The main advantage is that it prevents from lowering the coverage when a
test suite is renamed or added.

Test Plan

Run any CI build.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable