Electric-Coin-Company / zashi-ios

The Zashi Zcash Wallet, iOS Edition
MIT License
38 stars 12 forks source link

Refactor send tests #1295

Closed LukasKorba closed 3 months ago

LukasKorba commented 3 months ago

Send tests were done for a single reducer that handled both send and confirmation scenarios. Confirmation has been refactored into a standalone reducer so now the logic consists of 2 independent reducers.

The goal:

example: Very first test testSendSucceeded() is supposed to check whether flag is set and proper sendDone action is called afterwards. This logic has been moved to Confirmation reducer but send has something similar, instead of sending, it's reviewing and getting the proposal, opening confirmation is the success of this operation and should be checked.

LukasKorba commented 3 months ago

Fixed during refactor