Page MenuHomePhabricator

[tests] Make rpcauth.py testable and add unit tests
ClosedPublic

Authored by Fabien on Mon, Jul 29, 15:43.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCe760b402e979: [tests] Make rpcauth.py testable and add unit tests
Summary
Test Plan
make check
ninja check

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.Mon, Jul 29, 15:43
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Jul 29, 15:43
Fabien updated this revision to Diff 10544.Wed, Jul 31, 13:02

Add cmake support.

Fabien edited the test plan for this revision. (Show Details)Wed, Jul 31, 13:02
deadalnix requested changes to this revision.Thu, Aug 1, 06:16
deadalnix added inline comments.
Makefile.am
266 ↗(On Diff #10544)

There is something wrong here.

test/CMakeLists.txt
83 ↗(On Diff #10544)

Remove -test

This revision now requires changes to proceed.Thu, Aug 1, 06:16
Fabien updated this revision to Diff 10558.Thu, Aug 1, 09:40

Fix space vs tab indent.
Rename the target to avoid the redondant check-test pattern.

deadalnix added inline comments.Sat, Aug 3, 07:24
test/CMakeLists.txt
55 ↗(On Diff #10558)

This is clearly doesn't happen in the autotool build.

90 ↗(On Diff #10558)

Why does this need to be executed in the binary dir?

Fabien added inline comments.Sat, Aug 3, 19:53
test/CMakeLists.txt
55 ↗(On Diff #10558)

There is an equivalent AC_CONFIG_LINKS for autotools (see above).

90 ↗(On Diff #10558)

rpcauth-test.py requires the config.ini file to be located at ../config.ini, and this file is generated to the build tree rather than the source tree.

deadalnix accepted this revision.Sun, Aug 4, 07:57
This revision is now accepted and ready to land.Sun, Aug 4, 07:57
This revision was automatically updated to reflect the committed changes.