Page MenuHomePhabricator

[CMAKE] Actually run the seeder tests
ClosedPublic

Authored by Fabien on Fri, Jan 10, 21:57.

Details

Reviewers
deadalnix
nakihito
Group Reviewers
Restricted Project
Commits
rABC7ef9e4ef302e: [CMAKE] Actually run the seeder tests
Summary

The seeder test executable is built, but no test is declared so they are
not run. This diff fixes this bug.

Test Plan
ninja check-bitcoin-seeder

Edit src/seeder/test/seeder_tests.cpp to make the test fail, then
re-run:

ninja check-bitcoin-seeder

Check the test failure is reported correctly.

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.Fri, Jan 10, 21:57
Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, Jan 10, 21:57
nakihito accepted this revision.Fri, Jan 10, 21:59
This revision is now accepted and ready to land.Fri, Jan 10, 21:59
This revision was landed with ongoing or failed builds.Fri, Jan 10, 22:13
This revision was automatically updated to reflect the committed changes.