From 9dd85e5bd273bb888c66ef49588e06ae811ffa79 Mon Sep 17 00:00:00 2001
From: Matt Joiner <anacrolix@gmail.com>
Date: Fri, 14 May 2021 10:23:18 +1000
Subject: [PATCH] Fix non-deterministic test

---
 request-strategy/order_test.go | 25 ++++++++++---------------
 1 file changed, 10 insertions(+), 15 deletions(-)

diff --git a/request-strategy/order_test.go b/request-strategy/order_test.go
index 213b6f52..8c518b50 100644
--- a/request-strategy/order_test.go
+++ b/request-strategy/order_test.go
@@ -86,6 +86,11 @@ func TestStealingFromSlowerPeer(t *testing.T) {
 	check(secondStealer.Id, 2)
 }
 
+func checkNumRequestsAndInterest(c *qt.C, next PeerNextRequestState, num int, interest bool) {
+	c.Check(next.Requests, qt.HasLen, num)
+	c.Check(next.Interested, qt.Equals, interest)
+}
+
 func TestStealingFromSlowerPeersBasic(t *testing.T) {
 	c := qt.New(t)
 	order := ClientPieceOrder{}
@@ -106,7 +111,7 @@ func TestStealingFromSlowerPeersBasic(t *testing.T) {
 	firstStealer.Id = intPeerId(2)
 	secondStealer := basePeer
 	secondStealer.Id = intPeerId(3)
-	c.Assert(order.DoRequests([]*Torrent{{
+	results := order.DoRequests([]*Torrent{{
 		Pieces: []Piece{{
 			Request:           true,
 			NumPendingChunks:  2,
@@ -117,20 +122,10 @@ func TestStealingFromSlowerPeersBasic(t *testing.T) {
 			firstStealer,
 			secondStealer,
 		},
-	}}), qt.ContentEquals, map[PeerId]PeerNextRequestState{
-		intPeerId(2): {
-			Interested: true,
-			Requests:   requestSetFromSlice(r(0, 0)),
-		},
-		intPeerId(3): {
-			Interested: true,
-			Requests:   requestSetFromSlice(r(0, 1)),
-		},
-		stealee.Id: {
-			Interested: false,
-			Requests:   requestSetFromSlice(),
-		},
-	})
+	}})
+	checkNumRequestsAndInterest(c, results[firstStealer.Id], 1, true)
+	checkNumRequestsAndInterest(c, results[secondStealer.Id], 1, true)
+	checkNumRequestsAndInterest(c, results[stealee.Id], 0, false)
 }
 
 func TestPeerKeepsExistingIfReasonable(t *testing.T) {
-- 
2.44.0