Skip to content

Commit d830973

Browse files
authored
service/s3/s3crypto: fix dropped test errors (#4265)
This fixes three dropped err variables in the service/s3/s3crypto tests.
1 parent b5784eb commit d830973

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

Diff for: service/s3/s3crypto/cipher_util_test.go

+12
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,10 @@ func TestCEKFactory(t *testing.T) {
135135
MatDesc: `{"kms_cmk_id":""}`,
136136
}
137137
wrap, err := wrapFromEnvelope(o, env)
138+
if err != nil {
139+
t.Errorf("expected no error, but received %v", err)
140+
}
141+
138142
cek, err := cekFromEnvelope(o, aws.BackgroundContext(), env, wrap)
139143

140144
if err != nil {
@@ -195,6 +199,10 @@ func TestCEKFactoryNoCEK(t *testing.T) {
195199
MatDesc: `{"kms_cmk_id":""}`,
196200
}
197201
wrap, err := wrapFromEnvelope(o, env)
202+
if err != nil {
203+
t.Errorf("expected no error, but received %v", err)
204+
}
205+
198206
cek, err := cekFromEnvelope(o, aws.BackgroundContext(), env, wrap)
199207

200208
if err == nil {
@@ -251,6 +259,10 @@ func TestCEKFactoryCustomEntry(t *testing.T) {
251259
MatDesc: `{"kms_cmk_id":""}`,
252260
}
253261
wrap, err := wrapFromEnvelope(o, env)
262+
if err != nil {
263+
t.Errorf("expected no error, but received %v", err)
264+
}
265+
254266
cek, err := cekFromEnvelope(o, aws.BackgroundContext(), env, wrap)
255267

256268
if err != nil {

0 commit comments

Comments
 (0)