Skip to content

Commit 3884cda

Browse files
selzocaramprice
andcommitted
BWATS: Do not ignore errors when deleting deployments
If you do, the AfterSuite will run into errors because there are extant deployments using stemcells and releases. By failing when the "delete-deployment" command fails in the It, we will get faster feedback and not hide the errors. Co-authored-by: Aram Price <[email protected]>
1 parent 4b0fbae commit 3884cda

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

acceptance_test/main_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ var _ = AfterSuite(func() {
9090
return
9191
}
9292

93-
err := boshCommand.Run(fmt.Sprintf("--deployment=%s delete-deployment --force", deploymentName))
93+
err := boshCommand.Run(fmt.Sprintf("--deployment=%s delete-deployment", deploymentName))
9494
Expect(err).NotTo(HaveOccurred())
9595
err = boshCommand.Run(fmt.Sprintf("delete-stemcell %s/%s", stemcellName, stemcellVersion))
9696
Expect(err).NotTo(HaveOccurred())
@@ -162,7 +162,7 @@ var _ = Describe("BOSH Windows", func() {
162162
var slowCompilingDeploymentName string
163163

164164
AfterEach(func() {
165-
err := boshCommand.Run(fmt.Sprintf("--deployment=%s delete-deployment --force", slowCompilingDeploymentName))
165+
err := boshCommand.Run(fmt.Sprintf("--deployment=%s delete-deployment", slowCompilingDeploymentName))
166166
Expect(err).NotTo(HaveOccurred())
167167
})
168168

0 commit comments

Comments
 (0)