Page MenuHomePhabricator

Format BIP32 test cases manually
ClosedPublic

Authored by deadalnix on Feb 13 2019, 14:37.

Details

Summary

Simple formatting

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
bip32nit
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4992
Build 8047: Bitcoin ABC Buildbot (legacy)
Build 8046: arc lint + arc unit

Event Timeline

jasonbcox added inline comments.
src/test/bip32_tests.cpp
91 ↗(On Diff #7319)

Interesting how simple formatting introduces a new testcase :P

What new coverage does this provide? It's not clear how I can verify that it covers anything new.

Remove test cases added by mistake

markblundeberg added a subscriber: markblundeberg.

Yeah that was hideous before ... linters aren't always right. :-D

This revision is now accepted and ready to land.Feb 14 2019, 17:34
This revision was automatically updated to reflect the committed changes.