Page MenuHomePhabricator

qt: Use IsMine to validate custom change address
ClosedPublic

Authored by dagurval on Sep 13 2017, 13:28.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
qt-is-mine
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 874
Build 874: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Sep 14 2017, 16:45
deadalnix added a subscriber: deadalnix.

Honestly I'm not sure what this code is doing. It is rather concerning that changing it that way doesn't require changing any test. What would it take to write a test for this ?

src/qt/sendcoinsdialog.cpp
836 ↗(On Diff #1367)

Please keep the comment within the blocks.

840 ↗(On Diff #1367)

dito

This revision now requires changes to proceed.Sep 14 2017, 16:45

Looking at D540, I understand this better. It still would be beneficial to see if that can be unit tested in some way.

dagurval edited edge metadata.

formatting

Rebased and pushed formatting changes.

I don't see how SendCoinsDialog would be testable at all. WalletModel may be testable with some work.

But at least the IsMine function has some test coverage.

This revision is now accepted and ready to land.Sep 21 2017, 19:24
This revision was automatically updated to reflect the committed changes.