Paths

Table of Contentst

[CMAKE] Unbreak the activated tests
ClosedPublic
Actions

Authored by Fabien on Dec 19 2019, 16:13.

Details

Reviewers

deadalnix
jasonbcox

Group Reviewers

Restricted Project

Commits

rSTAGING3c51eb3130c5: [CMAKE] Unbreak the activated tests
rABC3c51eb3130c5: [CMAKE] Unbreak the activated tests

Summary

The tests with next upgrade activated is silently broken since there is
no longer any test suite enabled (D4698).

This diff re-enables the upgraded tests, apply them to all the boost
unit tests (and not only test_bitcoin) and run them in parallel like
any regular test.

Test Plan

cmake -GNinja .. -DTEST_WITH_UPGRADE_ACTIVATED=ON
ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Dec 19 2019, 16:13

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 19 2019, 16:13

Herald added a reviewer: deadalnix. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8642: Diff 14994.Dec 19 2019, 16:22

jasonbcox accepted this revision.Dec 19 2019, 17:15

This revision is now accepted and ready to land.Dec 19 2019, 17:15

Closed by commit rABC3c51eb3130c5: [CMAKE] Unbreak the activated tests (authored by Fabien). · Explain WhyDec 19 2019, 21:24

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC3c51eb3130c5: [CMAKE] Unbreak the activated tests.

Fabien added a commit: rSTAGING3c51eb3130c5: [CMAKE] Unbreak the activated tests.Apr 7 2020, 22:10

Revision Contents
Changeset List

Path

Size

cmake/

modules/

TestSuite.cmake

8 lines

src/

test/

CMakeLists.txt

11 lines

Diff 15000

View Options

cmake/modules/TestSuite.cmake