From 167e3e6625b90546e00990c8d0cd6f7fc1090003 Mon Sep 17 00:00:00 2001
From: William Banfield <wbanfield@gmail.com>
Date: Fri, 3 Dec 2021 13:38:23 -0500
Subject: [PATCH] simplify collector loop

---
 internal/consensus/pbts_test.go | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/internal/consensus/pbts_test.go b/internal/consensus/pbts_test.go
index acd72b9e4..434640226 100644
--- a/internal/consensus/pbts_test.go
+++ b/internal/consensus/pbts_test.go
@@ -276,10 +276,12 @@ func collectHeightResults(ctx context.Context, t *testing.T, eventCh <-chan time
 			res.proposalIssuedAt = event.ts
 		}
 		if res.isComplete() {
-			break
+			return res
 		}
 	}
-	return res
+	t.Fatalf("complete height result never seen for height %d", height)
+
+	panic("unreachable")
 }
 
 type timestampedEvent struct {