Skip to content

Commit be6fefa

Browse files
committed
refactor: Remove unnecessary error handling in bundle test cleanup
1 parent 4ea1202 commit be6fefa

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

integration/bundle/bind_resource_test.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,8 +162,7 @@ func TestGenerateAndBind(t *testing.T) {
162162

163163
deployBundle(t, ctx, bundleRoot)
164164

165-
err = destroyBundle(t, ctx, bundleRoot)
166-
require.NoError(t, err)
165+
destroyBundle(t, ctx, bundleRoot)
167166

168167
// Check that job is bound and does not extsts after bundle is destroyed
169168
_, err = w.Jobs.Get(ctx, jobs.GetJobRequest{

integration/bundle/deploy_test.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -182,8 +182,7 @@ func TestBundlePipelineRecreateWithoutAutoApprove(t *testing.T) {
182182
deployBundle(t, ctx, bundleRoot)
183183

184184
t.Cleanup(func() {
185-
err := destroyBundle(t, ctx, bundleRoot)
186-
require.NoError(t, err)
185+
destroyBundle(t, ctx, bundleRoot)
187186
})
188187

189188
// Assert the pipeline is created
@@ -219,8 +218,7 @@ func TestDeployBasicBundleLogs(t *testing.T) {
219218
})
220219

221220
t.Cleanup(func() {
222-
err := destroyBundle(t, ctx, root)
223-
require.NoError(t, err)
221+
destroyBundle(t, ctx, root)
224222
})
225223

226224
currentUser, err := wt.W.CurrentUser.Me(ctx)

0 commit comments

Comments
 (0)