chore: resolving nil pointer exception

ChainSafe · Jun 8, 2021 · cf5ae15 · cf5ae15
1 parent b2ce4b6
commit cf5ae15
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/dot/network/sync_test.go b/dot/network/sync_test.go
@@ -350,8 +350,8 @@ func TestSyncQueue_ProcessBlockRequests(t *testing.T) {
 		req: testBlockRequestMessage,
 	}
 
-	time.Sleep(time.Second * 2)
-	require.Equal(t, 128, len(nodeA.syncQueue.responses))
+	time.Sleep(time.Second * 3)
+	require.Len(t, nodeA.syncQueue.responses, 128)
 	testResp := testBlockResponseMessage()
 	require.Equal(t, testResp.BlockData, nodeA.syncQueue.responses)
 }

diff --git a/dot/network/test_helpers.go b/dot/network/test_helpers.go
@@ -47,7 +47,7 @@ func NewMockBlockState(n *big.Int) *MockBlockState {
 func NewMockSyncer() *MockSyncer {
 	mocksyncer := new(MockSyncer)
 	mocksyncer.On("HandleBlockAnnounce", mock.AnythingOfType("*network.BlockAnnounceMessage")).Return(nil, nil)
-	mocksyncer.On("CreateBlockResponse", mock.AnythingOfType("*network.BlockRequestMessage")).Return(nil, nil)
+	mocksyncer.On("CreateBlockResponse", mock.AnythingOfType("*network.BlockRequestMessage")).Return(testBlockResponseMessage(), nil)
 	mocksyncer.On("ProcessJustification", mock.AnythingOfType("[]*types.BlockData")).Return(0, nil)
 	mocksyncer.On("ProcessBlockData", mock.AnythingOfType("[]*types.BlockData")).Return(0, nil)
 	mocksyncer.On("SetSyncing", mock.AnythingOfType("bool"))