Skip to content

Commit 4f537f5

Browse files
emmanuelmathotemmanuelmathot
committed
Refactor Helm chart tests: Split service tests into individual files for raster, stac, vector, and multidim services; update deployment and configmap tests for backward compatibility; adjust ingress and HPA tests; clean up unused configurations in test.yaml.
1 parent 01e1e4f commit 4f537f5

File tree

9 files changed

+461
-855
lines changed

9 files changed

+461
-855
lines changed

‎helm-chart/eoapi/tests/config_tests.yaml‎

Lines changed: 5 additions & 67 deletions
Original file line numberDiff line numberDiff line change
@@ -1,67 +1,5 @@
1-
suite: service defaults configmap
2-
templates:
3-
- templates/services/raster/configmap.yaml
4-
- templates/services/stac/configmap.yaml
5-
- templates/services/vector/configmap.yaml
6-
- templates/services/multidim/configmap.yaml
7-
tests:
8-
- it: "vector configmap defaults"
9-
set:
10-
raster.enabled: false
11-
stac.enabled: false
12-
vector.enabled: true
13-
multidim.enabled: false
14-
asserts:
15-
- isKind:
16-
of: ConfigMap
17-
- matchRegex:
18-
path: metadata.name
19-
pattern: ^vector-envvar-configmap-RELEASE-NAME$
20-
- equal:
21-
path: data.TIPG_CATALOG_TTL
22-
value: "300"
23-
- it: "raster configmap defaults"
24-
set:
25-
raster.enabled: true
26-
stac.enabled: false
27-
vector.enabled: false
28-
multidim.enabled: false
29-
asserts:
30-
- isKind:
31-
of: ConfigMap
32-
- matchRegex:
33-
path: metadata.name
34-
pattern: ^raster-envvar-configmap-RELEASE-NAME$
35-
- equal:
36-
path: data.GDAL_HTTP_MULTIPLEX
37-
value: "YES"
38-
- it: "stac configmap defaults"
39-
set:
40-
raster.enabled: false
41-
stac.enabled: true
42-
vector.enabled: false
43-
multidim.enabled: false
44-
asserts:
45-
- isKind:
46-
of: ConfigMap
47-
- matchRegex:
48-
path: metadata.name
49-
pattern: ^stac-envvar-configmap-RELEASE-NAME$
50-
- equal:
51-
path: data.WEB_CONCURRENCY
52-
value: "5"
53-
- it: "multidim configmap defaults"
54-
set:
55-
raster.enabled: false
56-
stac.enabled: false
57-
vector.enabled: false
58-
multidim.enabled: true
59-
asserts:
60-
- isKind:
61-
of: ConfigMap
62-
- matchRegex:
63-
path: metadata.name
64-
pattern: ^multidim-envvar-configmap-RELEASE-NAME$
65-
- equal:
66-
path: data.GDAL_HTTP_MULTIPLEX
67-
value: "YES"
1+
# This file is kept for backward compatibility but tests have been moved to service-specific test files:
2+
# - raster_tests.yaml
3+
# - stac_tests.yaml
4+
# - vector_tests.yaml
5+
# - multidim_tests.yaml

‎helm-chart/eoapi/tests/deploy_tests.yaml‎

Lines changed: 5 additions & 127 deletions
Original file line numberDiff line numberDiff line change
@@ -1,127 +1,5 @@
1-
suite: service defaults deployment
2-
templates:
3-
- templates/services/raster/deployment.yaml
4-
- templates/services/stac/deployment.yaml
5-
- templates/services/vector/deployment.yaml
6-
- templates/services/multidim/deployment.yaml
7-
tests:
8-
- it: "vector deploy defaults"
9-
set:
10-
raster.enabled: false
11-
stac.enabled: false
12-
vector.enabled: true
13-
multidim.enabled: false
14-
asserts:
15-
- isKind:
16-
of: Deployment
17-
- matchRegex:
18-
path: metadata.name
19-
pattern: ^vector-RELEASE-NAME$
20-
- equal:
21-
path: spec.strategy.type
22-
value: "RollingUpdate"
23-
- equal:
24-
path: spec.template.spec.containers[0].resources.limits.cpu
25-
value: "768m"
26-
- equal:
27-
path: spec.template.spec.containers[0].resources.requests.cpu
28-
value: "256m"
29-
- equal:
30-
path: spec.template.spec.containers[0].resources.limits.memory
31-
value: "1024Mi"
32-
- equal:
33-
path: spec.template.spec.containers[0].resources.requests.memory
34-
value: "256Mi"
35-
- equal:
36-
path: metadata.labels.gitsha
37-
value: "ABC123"
38-
- it: "raster deploy defaults"
39-
set:
40-
raster.enabled: true
41-
stac.enabled: false
42-
vector.enabled: false
43-
multidim.enabled: false
44-
asserts:
45-
- isKind:
46-
of: Deployment
47-
- matchRegex:
48-
path: metadata.name
49-
pattern: ^raster-RELEASE-NAME$
50-
- equal:
51-
path: spec.strategy.type
52-
value: "RollingUpdate"
53-
- equal:
54-
path: spec.template.spec.containers[0].resources.limits.cpu
55-
value: "768m"
56-
- equal:
57-
path: spec.template.spec.containers[0].resources.requests.cpu
58-
value: "256m"
59-
- equal:
60-
path: spec.template.spec.containers[0].resources.limits.memory
61-
value: "4096Mi"
62-
- equal:
63-
path: spec.template.spec.containers[0].resources.requests.memory
64-
value: "3072Mi"
65-
- equal:
66-
path: metadata.labels.gitsha
67-
value: "ABC123"
68-
- it: "stac deploy defaults"
69-
set:
70-
raster.enabled: false
71-
stac.enabled: true
72-
vector.enabled: false
73-
multidim.enabled: false
74-
asserts:
75-
- isKind:
76-
of: Deployment
77-
- matchRegex:
78-
path: metadata.name
79-
pattern: ^stac-RELEASE-NAME$
80-
- equal:
81-
path: spec.strategy.type
82-
value: "RollingUpdate"
83-
- equal:
84-
path: spec.template.spec.containers[0].resources.limits.cpu
85-
value: "768m"
86-
- equal:
87-
path: spec.template.spec.containers[0].resources.requests.cpu
88-
value: "256m"
89-
- equal:
90-
path: spec.template.spec.containers[0].resources.limits.memory
91-
value: "1024Mi"
92-
- equal:
93-
path: spec.template.spec.containers[0].resources.requests.memory
94-
value: "1024Mi"
95-
- equal:
96-
path: metadata.labels.gitsha
97-
value: "ABC123"
98-
- it: "multidim deploy defaults"
99-
set:
100-
raster.enabled: false
101-
stac.enabled: false
102-
vector.enabled: false
103-
multidim.enabled: true
104-
asserts:
105-
- isKind:
106-
of: Deployment
107-
- matchRegex:
108-
path: metadata.name
109-
pattern: ^multidim-RELEASE-NAME$
110-
- equal:
111-
path: spec.strategy.type
112-
value: "RollingUpdate"
113-
- equal:
114-
path: spec.template.spec.containers[0].resources.limits.cpu
115-
value: "768m"
116-
- equal:
117-
path: spec.template.spec.containers[0].resources.requests.cpu
118-
value: "256m"
119-
- equal:
120-
path: spec.template.spec.containers[0].resources.limits.memory
121-
value: "4096Mi"
122-
- equal:
123-
path: spec.template.spec.containers[0].resources.requests.memory
124-
value: "3072Mi"
125-
- equal:
126-
path: metadata.labels.gitsha
127-
value: "ABC123"
1+
# This file is kept for backward compatibility but tests have been moved to service-specific test files:
2+
# - raster_tests.yaml
3+
# - stac_tests.yaml
4+
# - vector_tests.yaml
5+
# - multidim_tests.yaml

‎helm-chart/eoapi/tests/hpa_tests.yaml‎

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,2 @@
1-
suite: autoscaling feedback when className is not nginx
2-
templates:
3-
- templates/services/raster/hpa.yaml
4-
- templates/services/stac/hpa.yaml
5-
- templates/services/vector/hpa.yaml
6-
- templates/services/multidim/hpa.yaml
7-
tests:
8-
- it: "vector hpa fail for requestRate"
9-
set:
10-
raster.enabled: false
11-
stac.enabled: false
12-
vector.enabled: true
13-
ingress.className: "testing123"
14-
vector.autoscaling.enabled: true
15-
vector.autoscaling.type: "requestRate"
16-
asserts:
17-
- failedTemplate:
18-
errorMessage: When using an 'ingress.className' other than 'nginx' you cannot enable autoscaling by 'requestRate' at this time b/c it's solely an nginx metric
1+
# This file is kept for backward compatibility but tests have been moved to service-specific test files:
2+
# - vector_tests.yaml (for HPA tests)

‎helm-chart/eoapi/tests/ingress_tests.yaml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,4 +91,4 @@ tests:
9191
value: "/"
9292
- equal:
9393
path: spec.rules[0].http.paths[1].backend.service.name
94-
value: doc-server-RELEASE-NAME
94+
value: eoapi-doc-server

‎helm-chart/eoapi/tests/multidim_tests.yaml‎

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
suite: multidim service tests
2+
templates:
3+
- templates/services/multidim/deployment.yaml
4+
- templates/services/multidim/configmap.yaml
5+
- templates/services/multidim/service.yaml
6+
- templates/services/multidim/hpa.yaml
7+
tests:
8+
- it: "multidim deployment defaults"
9+
set:
10+
raster.enabled: false
11+
stac.enabled: false
12+
vector.enabled: false
13+
multidim.enabled: true
14+
gitSha: "ABC123"
15+
asserts:
16+
- isKind:
17+
of: Deployment
18+
- matchRegex:
19+
path: metadata.name
20+
pattern: ^multidim-RELEASE-NAME$
21+
- equal:
22+
path: spec.strategy.type
23+
value: "RollingUpdate"
24+
- equal:
25+
path: spec.template.spec.containers[0].resources.limits.cpu
26+
value: "768m"
27+
- equal:
28+
path: spec.template.spec.containers[0].resources.requests.cpu
29+
value: "256m"
30+
- equal:
31+
path: spec.template.spec.containers[0].resources.limits.memory
32+
value: "4096Mi"
33+
- equal:
34+
path: spec.template.spec.containers[0].resources.requests.memory
35+
value: "3072Mi"
36+
- equal:
37+
path: metadata.labels.gitsha
38+
value: "ABC123"
39+
40+
- it: "multidim configmap defaults"
41+
set:
42+
raster.enabled: false
43+
stac.enabled: false
44+
vector.enabled: false
45+
multidim.enabled: true
46+
asserts:
47+
- isKind:
48+
of: ConfigMap
49+
- matchRegex:
50+
path: metadata.name
51+
pattern: ^multidim-envvar-configmap-RELEASE-NAME$
52+
- equal:
53+
path: data.GDAL_HTTP_MULTIPLEX
54+
value: "YES"

‎helm-chart/eoapi/tests/raster_tests.yaml‎

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
suite: raster service tests
2+
templates:
3+
- templates/services/raster/deployment.yaml
4+
- templates/services/raster/configmap.yaml
5+
- templates/services/raster/service.yaml
6+
- templates/services/raster/hpa.yaml
7+
tests:
8+
- it: "raster deployment defaults"
9+
set:
10+
raster.enabled: true
11+
stac.enabled: false
12+
vector.enabled: false
13+
multidim.enabled: false
14+
gitSha: "ABC123"
15+
asserts:
16+
- isKind:
17+
of: Deployment
18+
- matchRegex:
19+
path: metadata.name
20+
pattern: ^raster-RELEASE-NAME$
21+
- equal:
22+
path: spec.strategy.type
23+
value: "RollingUpdate"
24+
- equal:
25+
path: spec.template.spec.containers[0].resources.limits.cpu
26+
value: "768m"
27+
- equal:
28+
path: spec.template.spec.containers[0].resources.requests.cpu
29+
value: "256m"
30+
- equal:
31+
path: spec.template.spec.containers[0].resources.limits.memory
32+
value: "4096Mi"
33+
- equal:
34+
path: spec.template.spec.containers[0].resources.requests.memory
35+
value: "3072Mi"
36+
- equal:
37+
path: metadata.labels.gitsha
38+
value: "ABC123"
39+
40+
- it: "raster configmap defaults"
41+
set:
42+
raster.enabled: true
43+
stac.enabled: false
44+
vector.enabled: false
45+
multidim.enabled: false
46+
asserts:
47+
- isKind:
48+
of: ConfigMap
49+
- matchRegex:
50+
path: metadata.name
51+
pattern: ^raster-envvar-configmap-RELEASE-NAME$
52+
- equal:
53+
path: data.GDAL_HTTP_MULTIPLEX
54+
value: "YES"

‎helm-chart/eoapi/tests/stac_tests.yaml‎

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
suite: stac service tests
2+
templates:
3+
- templates/services/stac/deployment.yaml
4+
- templates/services/stac/configmap.yaml
5+
- templates/services/stac/service.yaml
6+
- templates/services/stac/hpa.yaml
7+
tests:
8+
- it: "stac deployment defaults"
9+
set:
10+
raster.enabled: false
11+
stac.enabled: true
12+
vector.enabled: false
13+
multidim.enabled: false
14+
gitSha: "ABC123"
15+
asserts:
16+
- isKind:
17+
of: Deployment
18+
- matchRegex:
19+
path: metadata.name
20+
pattern: ^stac-RELEASE-NAME$
21+
- equal:
22+
path: spec.strategy.type
23+
value: "RollingUpdate"
24+
- equal:
25+
path: spec.template.spec.containers[0].resources.limits.cpu
26+
value: "768m"
27+
- equal:
28+
path: spec.template.spec.containers[0].resources.requests.cpu
29+
value: "256m"
30+
- equal:
31+
path: spec.template.spec.containers[0].resources.limits.memory
32+
value: "1024Mi"
33+
- equal:
34+
path: spec.template.spec.containers[0].resources.requests.memory
35+
value: "1024Mi"
36+
- equal:
37+
path: metadata.labels.gitsha
38+
value: "ABC123"
39+
40+
- it: "stac configmap defaults"
41+
set:
42+
raster.enabled: false
43+
stac.enabled: true
44+
vector.enabled: false
45+
multidim.enabled: false
46+
asserts:
47+
- isKind:
48+
of: ConfigMap
49+
- matchRegex:
50+
path: metadata.name
51+
pattern: ^stac-envvar-configmap-RELEASE-NAME$
52+
- equal:
53+
path: data.WEB_CONCURRENCY
54+
value: "5"

0 commit comments

Comments
 (0)