add timeout for sending cancel message to peer when retrieval cancelled #531
Conversation
| if testCase.failsUnseal { | ||
| assert.Equal(t, retrievalmarket.DealStatusErrored, clientDealState.Status) | ||
| } else if testCase.cancelled { | ||
| if testCase.failsUnseal || testCase.cancelled { |
There was a problem hiding this comment.
If there's an error unsealing on the provider, the best we can hope for on the client side is to receive a cancel message.
This test only passed before because the statemachine on the client was already shutdown when it received the Cancel event, so firing the event caused an error.
I fixed that bug so that the statemachine just ignores a Cancel event when the statemachine has already been shutdown
Codecov Report
@@ Coverage Diff @@
## master #531 +/- ##
==========================================
- Coverage 65.65% 65.54% -0.10%
==========================================
Files 52 53 +1
Lines 3534 3549 +15
==========================================
+ Hits 2320 2326 +6
- Misses 983 990 +7
- Partials 231 233 +2
Continue to review full report at Codecov.
|
No description provided.