Fix gossip unit test (#3422)

joshua-kim · web-flow · commit f11da2f7c0ad · 2024-09-26T20:41:41.000Z
Signed-off-by: Joshua Kim &lt;20001595+joshua-kim@users.noreply.github.com&gt;
diff --git a/network/p2p/gossip/gossip_test.go b/network/p2p/gossip/gossip_test.go
@@ -209,7 +209,7 @@ func TestValidatorGossiper(t *testing.T) {
 
 	nodeID := ids.GenerateTestNodeID()
 
-	validators := testValidatorSet{
+	validators := &testValidatorSet{
 		validators: set.Of(nodeID),
 	}
 
@@ -232,7 +232,7 @@ func TestValidatorGossiper(t *testing.T) {
 	// we are not a validator, so we should not request gossip
 	validators.validators = set.Set[ids.NodeID]{}
 	require.NoError(gossiper.Gossip(context.Background()))
-	require.Equal(2, calls)
+	require.Equal(1, calls)
 }
 
 func TestPushGossiperNew(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -209,7 +209,7 @@ func TestValidatorGossiper(t *testing.T) {`
`209`	`209`
`210`	`210`	`nodeID := ids.GenerateTestNodeID()`
`211`	`211`
`212`		`- validators := testValidatorSet{`
	`212`	`+ validators := &testValidatorSet{`
`213`	`213`	`validators: set.Of(nodeID),`
`214`	`214`	`}`
`215`	`215`
`@@ -232,7 +232,7 @@ func TestValidatorGossiper(t *testing.T) {`
`232`	`232`	`// we are not a validator, so we should not request gossip`
`233`	`233`	`validators.validators = set.Set[ids.NodeID]{}`
`234`	`234`	`require.NoError(gossiper.Gossip(context.Background()))`
`235`		`- require.Equal(2, calls)`
	`235`	`+ require.Equal(1, calls)`
`236`	`236`	`}`
`237`	`237`
`238`	`238`	`func TestPushGossiperNew(t *testing.T) {`