address review

filecoin-project · Aug 2, 2023 · f620310 · f620310
1 parent 402895f
commit f620310
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/itests/kit/deals.go b/itests/kit/deals.go
@@ -89,8 +89,6 @@ func NewDealHarness(t *testing.T, client *TestFullNode, main *TestMiner, market
 func (dh *DealHarness) MakeOnlineDeal(ctx context.Context, params MakeFullDealParams) (deal *cid.Cid, res *api.ImportRes, path string) {
 	deal, res, path = dh.StartRandomDeal(ctx, params)
 
-	// TODO: this sleep is only necessary because deals don't immediately get logged in the dealstore, we should fix this
-	time.Sleep(time.Second)
 	fmt.Printf("WAIT DEAL SEALEDS START\n")
 	dh.WaitDealSealed(ctx, deal, false, false, nil)
 	fmt.Printf("WAIT DEAL SEALEDS END\n")

diff --git a/itests/worker_test.go b/itests/worker_test.go
@@ -750,7 +750,7 @@ func TestWorkerPledgeExpireCommit(t *testing.T) {
 		kit.ConstructorOpts(
 			node.Override(new(*sealing.Sealing), modules.SealingPipeline(fc)),
 		),
-		kit.SplitstoreDisable(), // enable splitstore because messages which take a long time may get dropped
+		kit.SplitstoreDisable(), // disable splitstore because messages which take a long time may get dropped
 		tasksNoC2)               // no mock proofs
 
 	ens.InterconnectAll().BeginMiningMustPost(2 * time.Millisecond)