Skip to content

Commit 4b6be34

Browse files
committed
chore: Add two tests for env key and fix failing test for helm chart
Signed-off-by: Peter Fiddes <[email protected]>
1 parent 32c4fd4 commit 4b6be34

File tree

1 file changed

+26
-3
lines changed

1 file changed

+26
-3
lines changed

deploy/charts/venafi-kubernetes-agent/tests/deployment_test.yaml

Lines changed: 26 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,11 +75,11 @@ tests:
7575
path: spec.template.spec.containers[0].command
7676
content: notpreflight
7777

78-
# Check the volumes and volumeMounts works correctly
78+
# Check the volumes and volumeMounts works correctly - FAILING
7979
- it: Volumes and VolumeMounts added correctly
8080
set:
81-
config.organisation: test_org
82-
config.cluster: test_cluster
81+
# config.organisation: test_org -> Should be removed?
82+
# config.cluster: test_cluster -> Should be config.clusterName?
8383
values:
8484
- ./values/custom-volumes.yaml
8585
asserts:
@@ -102,3 +102,26 @@ tests:
102102
items:
103103
- key: cabundle
104104
path: ca-certificates.crt
105+
106+
# Check proxy settings are additive not overriding
107+
- it: All environment variables present when all proxy settings are supplied
108+
set:
109+
http_proxy: "http://testing.proxy"
110+
no_proxy: "127.0.0.1,localhost"
111+
https_proxy: "https://testing.proxy"
112+
template: deployment.yaml
113+
asserts:
114+
- isKind:
115+
of: Deployment
116+
- lengthEqual :
117+
path: spec.template.spec.containers[0].env
118+
count: 7
119+
120+
- it: Only default environment variables are set when no proxy settings are provided
121+
template: deployment.yaml
122+
asserts:
123+
- isKind:
124+
of: Deployment
125+
- lengthEqual :
126+
path: spec.template.spec.containers[0].env
127+
count: 4

0 commit comments

Comments
 (0)