Page MenuHomePhabricator

qt: Use IsMine to validate custom change address

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

Diff Detail

rABC Bitcoin ABC
Lint Not Applicable
Tests Not Applicable

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 ?

836 ↗(On Diff #1367)

Please keep the comment within the blocks.

840 ↗(On Diff #1367)


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.


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.