From a4bf9fe47f636fd3a7c8f4940b76c19a38a0834b Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Wed, 27 May 2026 12:15:21 -0700 Subject: [PATCH] test(remote_storage/s3): assert WriteFile success in mime tests --- weed/remote_storage/s3/s3_storage_client_test.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/weed/remote_storage/s3/s3_storage_client_test.go b/weed/remote_storage/s3/s3_storage_client_test.go index 79bb794b6..46b6e7d42 100644 --- a/weed/remote_storage/s3/s3_storage_client_test.go +++ b/weed/remote_storage/s3/s3_storage_client_test.go @@ -130,7 +130,8 @@ func TestS3WriteFilePassesMimeAsContentType(t *testing.T) { Attributes: &filer_pb.FuseAttributes{Mime: "text/html"}, } - _, _ = client.WriteFile(loc, entry, bytes.NewReader([]byte(""))) + _, err := client.WriteFile(loc, entry, bytes.NewReader([]byte(""))) + require.NoError(t, err) require.NotNil(t, rt.uploadReq, "uploader should have issued a PUT") require.Equal(t, "text/html", rt.uploadContentType(), "Content-Type should match entry.Attributes.Mime") @@ -147,7 +148,8 @@ func TestS3WriteFileOmitsContentTypeWhenMimeMissing(t *testing.T) { Attributes: &filer_pb.FuseAttributes{}, } - _, _ = client.WriteFile(loc, entry, bytes.NewReader([]byte("data"))) + _, err := client.WriteFile(loc, entry, bytes.NewReader([]byte("data"))) + require.NoError(t, err) require.NotNil(t, rt.uploadReq, "uploader should have issued a PUT") // When entry.Attributes.Mime is empty we don't force a Content-Type so the