Skip to content

Commit 93dd3cb

Browse files
authored
Merge pull request #5173 from camilamacedo86/fix-helm
🐛 (helm/v2-alpha): ensure that certmanager option is exposed always
2 parents 464a909 + 49008e4 commit 93dd3cb

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/plugins/optional/helm/v2alpha/scaffolds/internal/templates/values_basic_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,8 @@ var _ = Describe("HelmValuesBasic", func() {
7272
It("should not include certManager configuration", func() {
7373
content := valuesTemplate.GetBody()
7474

75-
Expect(content).NotTo(ContainSubstring("certManager:"))
76-
Expect(content).NotTo(ContainSubstring("enable: true"))
75+
Expect(content).To(ContainSubstring("certManager:"))
76+
Expect(content).To(ContainSubstring("enable: false"))
7777
})
7878

7979
It("should still include other basic sections", func() {

0 commit comments

Comments
 (0)