reduce diff

snow/engine/avalanche/getter/getter_test.go

@@ -73,6 +73,9 @@ func TestAcceptedFrontier(t *testing.T) {
 	vtxID2 := ids.GenerateTestID()
 
 	bsIntf, err := New(manager, config)
+	if err != nil {
+		t.Fatal(err)
+	}
 	require.IsType(&getter{}, bsIntf)
 	bs := bsIntf.(*getter)
 
@@ -127,7 +130,9 @@ func TestFilterAccepted(t *testing.T) {
 	}}
 
 	bsIntf, err := New(manager, config)
-	require.NoError(err)
+	if err != nil {
+		t.Fatal(err)
+	}
 	require.IsType(&getter{}, bsIntf)
 	bs := bsIntf.(*getter)
 

vms/platformvm/blocks/executor/manager_test.go

@@ -65,14 +65,12 @@ func TestGetBlock(t *testing.T) {
 }
 
 func TestManagerLastAccepted(t *testing.T) {
-	require := require.New(t)
-
 	lastAcceptedID := ids.GenerateTestID()
 	manager := &manager{
 		backend: &backend{
 			lastAccepted: lastAcceptedID,
 		},
 	}
 
-	require.Equal(lastAcceptedID, manager.LastAccepted())
+	require.Equal(t, lastAcceptedID, manager.LastAccepted())
 }

vms/platformvm/service_test.go

@@ -353,6 +353,12 @@ func TestGetTx(t *testing.T) {
 
 						commit := options[0].(*blockexecutor.Block)
 						require.IsType(&blocks.BanffCommitBlock{}, commit.Block)
+
+						err := commit.Verify(context.Background())
+						require.NoError(err)
+
+						err = commit.Accept(context.Background())
+						require.NoError(err)
 					}
 				}
 

vms/proposervm/vm_test.go

@@ -790,7 +790,16 @@ func TestPreFork_ParseBlock(t *testing.T) {
 	}
 
 	parsedBlk, err := proVM.ParseBlock(context.Background(), coreBlk.Bytes())
+	if err != nil {
+		t.Fatal("Could not parse naked core block")
+	}
 	require.IsType(&preForkBlock{}, parsedBlk)
+	if parsedBlk.ID() != coreBlk.ID() {
+		t.Fatal("Parsed block does not match expected block")
+	}
+	if !bytes.Equal(parsedBlk.Bytes(), coreBlk.Bytes()) {
+		t.Fatal("Parsed block does not match expected block")
+	}
 
 	coreVM.GetBlockF = func(_ context.Context, id ids.ID) (snowman.Block, error) {
 		if id != coreBlk.ID() {