From 2f99c97e5ea7350554da74afb3dfb5e3c0c42f9a Mon Sep 17 00:00:00 2001 From: Chris Cotter Date: Tue, 17 Dec 2024 10:58:23 -0500 Subject: [PATCH] test(storage): fix go vet for test errors (#11299) Calling t.Errorf without a proper format string causes an error in go 1.24. --- storage/bucket_test.go | 48 +++++++++++++++++++++--------------------- storage/option_test.go | 3 ++- 2 files changed, 26 insertions(+), 25 deletions(-) diff --git a/storage/bucket_test.go b/storage/bucket_test.go index f1bd45e52775..d46f802a7128 100644 --- a/storage/bucket_test.go +++ b/storage/bucket_test.go @@ -279,7 +279,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set BucketPolicyOnly.Enabled = true --> UBLA should be set to enabled in @@ -294,7 +294,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set both BucketPolicyOnly.Enabled = true and @@ -310,7 +310,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set UBLA.Enabled=false and BucketPolicyOnly.Enabled=false --> UBLA @@ -322,7 +322,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Test that setting PublicAccessPrevention to "unspecified" leads to the @@ -333,7 +333,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { PublicAccessPrevention: "inherited", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Test that setting PublicAccessPrevention to "inherited" leads to the @@ -344,7 +344,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { PublicAccessPrevention: "inherited", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Test that setting RPO to default is propagated in the proto. @@ -352,7 +352,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { got = attrs.toRawBucket() want.Rpo = rpoDefault if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Re-enable UBLA and confirm that it does not affect the PAP setting. @@ -365,7 +365,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { PublicAccessPrevention: "inherited", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Disable UBLA and reset PAP to default. Confirm that the IAM config is set @@ -375,7 +375,7 @@ func TestBucketAttrsToRawBucket(t *testing.T) { got = attrs.toRawBucket() want.IamConfiguration = nil if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } } @@ -504,7 +504,7 @@ func TestBucketAttrsToUpdateToRawBucket(t *testing.T) { }, } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set BucketPolicyOnly.Enabled = true --> UBLA should be set to enabled in @@ -522,7 +522,7 @@ func TestBucketAttrsToUpdateToRawBucket(t *testing.T) { }, } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set both BucketPolicyOnly.Enabled = true and @@ -542,7 +542,7 @@ func TestBucketAttrsToUpdateToRawBucket(t *testing.T) { }, } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set UBLA.Enabled=false and BucketPolicyOnly.Enabled=false --> UBLA @@ -561,7 +561,7 @@ func TestBucketAttrsToUpdateToRawBucket(t *testing.T) { }, } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // UBLA.Enabled will have precedence above BucketPolicyOnly.Enabled if both @@ -580,7 +580,7 @@ func TestBucketAttrsToUpdateToRawBucket(t *testing.T) { }, } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set an empty Lifecycle and verify that it will be sent. @@ -595,7 +595,7 @@ func TestBucketAttrsToUpdateToRawBucket(t *testing.T) { ForceSendFields: []string{"Lifecycle"}, } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } } @@ -992,7 +992,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set BucketPolicyOnly.Enabled = true --> UBLA should be set to enabled in @@ -1007,7 +1007,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set both BucketPolicyOnly.Enabled = true and @@ -1023,7 +1023,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Set UBLA.Enabled=false and BucketPolicyOnly.Enabled=false --> UBLA @@ -1035,7 +1035,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { PublicAccessPrevention: "enforced", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Test that setting PublicAccessPrevention to "unspecified" leads to the @@ -1046,7 +1046,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { PublicAccessPrevention: "inherited", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Test that setting PublicAccessPrevention to "inherited" leads to the @@ -1057,7 +1057,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { PublicAccessPrevention: "inherited", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Test that setting RPO to default is propagated in the proto. @@ -1065,7 +1065,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { got = attrs.toProtoBucket() want.Rpo = rpoDefault if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Re-enable UBLA and confirm that it does not affect the PAP setting. @@ -1078,7 +1078,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { PublicAccessPrevention: "inherited", } if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } // Disable UBLA and reset PAP to default. Confirm that the IAM config is set @@ -1088,7 +1088,7 @@ func TestBucketAttrsToProtoBucket(t *testing.T) { got = attrs.toProtoBucket() want.IamConfig = nil if msg := testutil.Diff(got, want); msg != "" { - t.Errorf(msg) + t.Errorf("%v", msg) } } diff --git a/storage/option_test.go b/storage/option_test.go index ea7c3915fab8..dfb30d3667cc 100644 --- a/storage/option_test.go +++ b/storage/option_test.go @@ -146,7 +146,8 @@ func TestApplyStorageOpt(t *testing.T) { } } if !cmp.Equal(got, test.want, cmp.AllowUnexported(storageConfig{}, experimental.ReadStallTimeoutConfig{})) { - t.Errorf(cmp.Diff(got, test.want, cmp.AllowUnexported(storageConfig{}, experimental.ReadStallTimeoutConfig{}))) + diff := cmp.Diff(got, test.want, cmp.AllowUnexported(storageConfig{}, experimental.ReadStallTimeoutConfig{})) + t.Errorf("options: diff got, want: %v", diff) } }) }