[CMAKE] Factorize the test suite target name construction
ClosedPublic
Actions

Authored by Fabien on Feb 21 2020, 14:02.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING05556886c6c6: [CMAKE] Factorize the test suite target name construction
rABC05556886c6c6: [CMAKE] Factorize the test suite target name construction

Summary

This is a minor refactor that adds a facility for building a test suite
target name from the test suite name.

There is no change in behavior.

Test Plan

ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

cmake_get_target_from_suite

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 9535
Build 16980: Default Diff Build & Tests
Build 16979: arc lint + arc unit

Event Timeline

Fabien created this revision.Feb 21 2020, 14:02

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 21 2020, 14:02

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B9535: Diff 16451.Feb 21 2020, 14:12

Fabien added a child revision: D5312: [CMAKE] Make the list of tests a property of the test suite.Feb 21 2020, 15:02

jasonbcox accepted this revision.Feb 21 2020, 17:06

This revision is now accepted and ready to land.Feb 21 2020, 17:06

Closed by commit rABC05556886c6c6: [CMAKE] Factorize the test suite target name construction (authored by Fabien). · Explain WhyFeb 22 2020, 11:07

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC05556886c6c6: [CMAKE] Factorize the test suite target name construction.

Fabien added a commit: rSTAGING05556886c6c6: [CMAKE] Factorize the test suite target name construction.Apr 7 2020, 23:04

Revision Contents
Changeset List

Path

Size

cmake/

modules/

TestSuite.cmake

17 lines

Diff 16451

View Options

cmake/modules/TestSuite.cmake

[CMAKE] Factorize the test suite target name constructionClosedPublicActions