Paths

Table of Contentst

Differential D16794

Interleave the debug.log with the unit test log when run via ninja
ClosedPublic
Actions

Authored by Fabien on Sep 20 2024, 09:23.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABCa7ca07264935: Interleave the debug.log with the unit test log when run via ninja

Summary

This diff adds the printtoconsole=1 option to the unit tests that are run via ninja.
Currently only boost content is part of the resulting log, which can be not enough to properly debug a failure. Now upon failure the output is much more verbose, but also more useful.

Test Plan

Edit any test with BOOST_CHECK(false) and run that test via ninja:

ninja check-bitcoin-<test>

Check that stdout and test/log/<test_name>.log contain the debug.log content.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Sep 20 2024, 09:23

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 20 2024, 09:23

Fabien requested review of this revision.Sep 20 2024, 09:23

Harbormaster completed remote builds in B30374: Diff 49756.Sep 20 2024, 09:37

Fabien mentioned this in D16791: Store the last 20 headers reception time to a file.Sep 20 2024, 09:44

PiRK accepted this revision.Sep 20 2024, 10:13

This revision is now accepted and ready to land.Sep 20 2024, 10:13

Closed by commit rABCa7ca07264935: Interleave the debug.log with the unit test log when run via ninja (authored by Fabien). · Explain WhySep 20 2024, 11:36

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCa7ca07264935: Interleave the debug.log with the unit test log when run via ninja.

Revision Contents
Changeset List

Path

Size

cmake/

modules/

TestSuite.cmake

2 lines

src/

test/

1 line

Diff 49758

cmake/modules/TestSuite.cmake

Loading...

src/test/CMakeLists.txt

Loading...