Skip to content

Commit de6c69e

Browse files
committed
Fix lint: check error return values in test handler
Signed-off-by: Sreekanth Vadigi <sreekanth.vadigi@databricks.com>
1 parent 65a8750 commit de6c69e

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

internal/rows/arrowbased/batchloader_test.go

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -356,13 +356,18 @@ func TestCloudFetchSchemaOverride(t *testing.T) {
356356
correctSchema := arrow.NewSchema(correctFields, nil)
357357
var schemaBuf bytes.Buffer
358358
schemaWriter := ipc.NewWriter(&schemaBuf, ipc.WithSchema(correctSchema))
359-
schemaWriter.Close()
359+
if err := schemaWriter.Close(); err != nil {
360+
t.Fatal(err)
361+
}
360362
correctSchemaBytes := schemaBuf.Bytes()
361363

362364
// Serve stale IPC data via mock HTTP
363365
server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
364366
w.WriteHeader(http.StatusOK)
365-
w.Write(staleIPCBytes)
367+
_, err := w.Write(staleIPCBytes)
368+
if err != nil {
369+
panic(err)
370+
}
366371
}))
367372
defer server.Close()
368373

0 commit comments

Comments
 (0)