Skip to content

Commit 5ca4a43

Browse files
nil-receiver test for deposeCurrentObject (no panic)
1 parent e123492 commit 5ca4a43

File tree

1 file changed

+6
-17
lines changed

1 file changed

+6
-17
lines changed

internal/states/resource_test.go

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -8,24 +8,13 @@ import (
88
)
99

1010
func TestResourceInstanceDeposeCurrentObject(t *testing.T) {
11-
t.Run("nil receiver HasCurrent", func(t *testing.T) {
12-
// small helper to catch panics
13-
callHasCurrent := func(ri *ResourceInstance) (ret bool, panicked bool) {
14-
defer func() {
15-
if r := recover(); r != nil {
16-
panicked = true
17-
}
18-
}()
19-
return ri.HasCurrent(), false
20-
}
21-
11+
t.Run("nil resource", func(t *testing.T) {
2212
var nilRI *ResourceInstance
23-
got, panicked := callHasCurrent(nilRI)
24-
if panicked {
25-
t.Fatalf("HasCurrent(nil) panicked; want no panic and false")
26-
}
27-
if got {
28-
t.Fatalf("HasCurrent(nil) = true; want false")
13+
dk := nilRI.deposeCurrentObject(NotDeposed)
14+
t.Logf("deposedKey (nil receiver) is %q", dk)
15+
16+
if dk != NotDeposed {
17+
t.Fatalf("expected NotDeposed for nil receiver, got %q", dk)
2918
}
3019
})
3120
obj := &ResourceInstanceObjectSrc{

0 commit comments

Comments
 (0)