Skip to content

Commit 016016b

Browse files
committed
itest: save about 40 seconds in itest
By reducing the hashmail proof receiver ACK timeout from 15 seconds to 5 seconds, we can save quite some time during the integration tests.
1 parent b1e73ad commit 016016b

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

itest/send_test.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,10 @@ import (
2121
"github.com/stretchr/testify/require"
2222
)
2323

24+
var (
25+
transferTypeSend = taprpc.ProofTransferType_PROOF_TRANSFER_TYPE_SEND
26+
)
27+
2428
// testBasicSendUnidirectional tests that we can properly send assets back and
2529
// forth between nodes.
2630
func testBasicSendUnidirectional(t *harnessTest) {
@@ -619,7 +623,7 @@ func testReattemptFailedSendHashmailCourier(t *harnessTest) {
619623
switch eventTyped := event.Event.(type) {
620624
case *taprpc.SendAssetEvent_ProofTransferBackoffWaitEvent:
621625
ev := eventTyped.ProofTransferBackoffWaitEvent
622-
if ev.TransferType != taprpc.ProofTransferType_PROOF_TRANSFER_TYPE_SEND {
626+
if ev.TransferType != transferTypeSend {
623627
return false
624628
}
625629

@@ -726,7 +730,7 @@ func testReattemptFailedSendUniCourier(t *harnessTest) {
726730
switch eventTyped := event.Event.(type) {
727731
case *taprpc.SendAssetEvent_ProofTransferBackoffWaitEvent:
728732
ev := eventTyped.ProofTransferBackoffWaitEvent
729-
if ev.TransferType != taprpc.ProofTransferType_PROOF_TRANSFER_TYPE_SEND {
733+
if ev.TransferType != transferTypeSend {
730734
return false
731735
}
732736

@@ -1000,7 +1004,7 @@ func testOfflineReceiverEventuallyReceives(t *harnessTest) {
10001004
// node. We therefore expect to receive
10011005
// deliver transfer type backoff wait events
10021006
// for sending transfers.
1003-
if ev.TransferType != taprpc.ProofTransferType_PROOF_TRANSFER_TYPE_SEND {
1007+
if ev.TransferType != transferTypeSend {
10041008
return false
10051009
}
10061010

itest/tapd_harness.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ const (
7676
// defaultProofTransferReceiverAckTimeout is the default itest specific
7777
// timeout we'll use for waiting for a receiver to acknowledge a proof
7878
// transfer.
79-
defaultProofTransferReceiverAckTimeout = 15 * time.Second
79+
defaultProofTransferReceiverAckTimeout = 5 * time.Second
8080
)
8181

8282
// tapdHarness is a test harness that holds everything that is needed to

0 commit comments

Comments
 (0)