closes #239 (The issue says it's not necessary to do an integration test but I'd rather have real blocks stepping over. Plus mocking all the intermediate sync state responses would add a lot of complexity IMHO)
On #219 we added the Pay To ID with Recall transaction along with an integration test for the "happy path" (sender executes the tx, target receives it and consumes it).
On this PR we check the following cases that were left out (The P2IDR tx has one output note which handles the funds transfer):
The sender tries to reclaim (consume) the note too early and fails
The sender tries to consume the note after waiting for the recall height and does so successfully
The sender consumes the note and the target can't consume the note anymore
closes #239 (The issue says it's not necessary to do an integration test but I'd rather have real blocks stepping over. Plus mocking all the intermediate sync state responses would add a lot of complexity IMHO)
On #219 we added the Pay To ID with Recall transaction along with an integration test for the "happy path" (sender executes the tx, target receives it and consumes it).
On this PR we check the following cases that were left out (The P2IDR tx has one output note which handles the funds transfer):