Skip to content

Commit ee4a80f

Browse files
authored
test(options): refactor CriticalFusePodEnabled tests to use Ginkgo and add options suite (#5595)
Signed-off-by: adity1raut <araut7798@gmail.com>
1 parent 21c5683 commit ee4a80f

1 file changed

Lines changed: 49 additions & 39 deletions

File tree

pkg/utils/runtimes/options/critical_fuse_pod_test.go

Lines changed: 49 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -17,44 +17,54 @@ limitations under the License.
1717
package options
1818

1919
import (
20-
"testing"
20+
. "github.com/onsi/ginkgo/v2"
21+
. "github.com/onsi/gomega"
22+
"os"
2123
)
2224

23-
func TestCriticalFusePodEnabled(t *testing.T) {
24-
type testCase struct {
25-
name string
26-
env map[string]string
27-
expect bool
28-
}
29-
30-
testCases := []testCase{
31-
{
32-
name: "not_set",
33-
env: map[string]string{},
34-
expect: false,
35-
}, {
36-
name: "set_true",
37-
env: map[string]string{
38-
EnvCriticalFusePodEnabled: "true",
39-
},
40-
expect: true,
41-
}, {
42-
name: "set_false",
43-
env: map[string]string{
44-
EnvCriticalFusePodEnabled: "false",
45-
},
46-
expect: false,
47-
},
48-
}
49-
50-
for _, test := range testCases {
51-
for k, v := range test.env {
52-
t.Setenv(k, v)
53-
}
54-
setCriticalFusePodOption()
55-
got := CriticalFusePodEnabled()
56-
if got != test.expect {
57-
t.Errorf("testcase %s is failed due to expect %v, but got %v", test.name, test.expect, got)
58-
}
59-
}
60-
}
25+
var _ = Describe("CriticalFusePodEnabled", func() {
26+
27+
Context("when environment variable is not set", func() {
28+
BeforeEach(func() {
29+
os.Unsetenv(EnvCriticalFusePodEnabled)
30+
})
31+
32+
It("should return false", func() {
33+
setCriticalFusePodOption()
34+
got := CriticalFusePodEnabled()
35+
Expect(got).To(BeFalse())
36+
})
37+
})
38+
39+
Context("when environment variable is set to true", func() {
40+
BeforeEach(func() {
41+
os.Setenv(EnvCriticalFusePodEnabled, "true")
42+
})
43+
44+
AfterEach(func() {
45+
os.Unsetenv(EnvCriticalFusePodEnabled)
46+
})
47+
48+
It("should return true", func() {
49+
setCriticalFusePodOption()
50+
got := CriticalFusePodEnabled()
51+
Expect(got).To(BeTrue())
52+
})
53+
})
54+
55+
Context("when environment variable is set to false", func() {
56+
BeforeEach(func() {
57+
os.Setenv(EnvCriticalFusePodEnabled, "false")
58+
})
59+
60+
AfterEach(func() {
61+
os.Unsetenv(EnvCriticalFusePodEnabled)
62+
})
63+
64+
It("should return false", func() {
65+
setCriticalFusePodOption()
66+
got := CriticalFusePodEnabled()
67+
Expect(got).To(BeFalse())
68+
})
69+
})
70+
})

0 commit comments

Comments
 (0)