Fix ineffectual assignment to err

pingcap · Oct 22, 2023 · bac39af · bac39af
1 parent bec37b6
commit bac39af
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/pkg/server/internal/packetio_test.go b/pkg/server/internal/packetio_test.go
@@ -342,6 +342,7 @@ func TestCompressedReaderShort(t *testing.T) {
 
 	data := make([]byte, length)
 	_, err = cr.Read(data)
+	require.NoError(t, err)
 	expected := []byte{0x3, 0x73, 0x65, 0x6c, 0x65, 0x63, 0x74, 0x20, 0x40, 0x40,
 		0x76, 0x65, 0x72, 0x73, 0x69, 0x6f, 0x6e, 0x5f, 0x63, 0x6f, 0x6d, 0x6d,
 		0x65, 0x6e, 0x74, 0x20, 0x6c, 0x69, 0x6d, 0x69, 0x74, 0x20, 0x31}
@@ -368,6 +369,7 @@ func TestCompressedReaderLong(t *testing.T) {
 
 		data := make([]byte, length)
 		_, err = cr.Read(data)
+		require.NoError(t, err)
 		expected := []byte{0x3, 0x53, 0x45, 0x4c, 0x45, 0x43, 0x54, 0x20, 0x22,
 			0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41,
 			0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41,
@@ -404,6 +406,7 @@ func TestCompressedReaderLong(t *testing.T) {
 
 		data := make([]byte, length)
 		_, err = cr.Read(data)
+		require.NoError(t, err)
 		expected := []byte{0x3, 0x53, 0x45, 0x4c, 0x45, 0x43, 0x54, 0x20, 0x22,
 			0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41,
 			0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41,