[integration-test] skip deleting if test is skipped

ramonskie · ramonskie · commit f514d40acd8b · 2026-01-06T15:40:40.000+01:00
diff --git a/src/integration/frameworks_test.go b/src/integration/frameworks_test.go
@@ -30,7 +30,8 @@ func testFrameworks(platform switchblade.Platform, fixtures string) func(*testin
 				t.Logf("❌ FAILED TEST - App/Container: %s", name)
 				t.Logf("   Platform: %s", settings.Platform)
 			}
-			if name != "" && (!settings.KeepFailedContainers || !t.Failed()) {
+			// Only attempt cleanup if test actually ran (not skipped) and name was set
+			if name != "" && !t.Skipped() && (!settings.KeepFailedContainers || !t.Failed()) {
 				Expect(platform.Delete.Execute(name)).To(Succeed())
 			}
 		})

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,8 @@ func testFrameworks(platform switchblade.Platform, fixtures string) func(*testin`
`30`	`30`	`t.Logf("❌ FAILED TEST - App/Container: %s", name)`
`31`	`31`	`t.Logf(" Platform: %s", settings.Platform)`
`32`	`32`	`}`
`33`		`- if name != "" && (!settings.KeepFailedContainers \|\| !t.Failed()) {`
	`33`	`+ // Only attempt cleanup if test actually ran (not skipped) and name was set`
	`34`	`+ if name != "" && !t.Skipped() && (!settings.KeepFailedContainers \|\| !t.Failed()) {`
`34`	`35`	`Expect(platform.Delete.Execute(name)).To(Succeed())`
`35`	`36`	`}`
`36`	`37`	`})`