Skip to content

Commit 05c5766

Browse files
committed
refactor: Remove unnecessary error handling in bundle test cleanup
1 parent 7b8dd11 commit 05c5766

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
@@ -163,8 +163,7 @@ func TestGenerateAndBind(t *testing.T) {
163163

164164
deployBundle(t, ctx, bundleRoot)
165165

166-
err = destroyBundle(t, ctx, bundleRoot)
167-
require.NoError(t, err)
166+
destroyBundle(t, ctx, bundleRoot)
168167

169168
// Check that job is bound and does not extsts after bundle is destroyed
170169
_, 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
@@ -183,8 +183,7 @@ func TestBundlePipelineRecreateWithoutAutoApprove(t *testing.T) {
183183
deployBundle(t, ctx, bundleRoot)
184184

185185
t.Cleanup(func() {
186-
err := destroyBundle(t, ctx, bundleRoot)
187-
require.NoError(t, err)
186+
destroyBundle(t, ctx, bundleRoot)
188187
})
189188

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

222221
t.Cleanup(func() {
223-
err := destroyBundle(t, ctx, root)
224-
require.NoError(t, err)
222+
destroyBundle(t, ctx, root)
225223
})
226224

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

0 commit comments

Comments
 (0)