Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: implement Helm chart unit tests #116

Merged
merged 1 commit into from
Oct 10, 2023
Merged

test: implement Helm chart unit tests #116

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/helm-test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ jobs:

- name: Run Tests
run: |
if [ ! -e "deploy/charts/verson-checker/tests" ]; then
echo "Not running tests, directory doesn't exist: deploy/charts/verson-checker/tests"
if [ ! -e "deploy/charts/version-checker/tests" ]; then
echo "Not running tests, directory doesn't exist: deploy/charts/version-checker/tests"
exit 0
fi
helm unittest --helm3 --color deploy/charts/verson-checker
helm unittest deploy/charts/version-checker
13 changes: 0 additions & 13 deletions deploy/charts/version-checker/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,16 +25,3 @@ app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
{{- end }}
app.kubernetes.io/managed-by: {{ .Release.Service }}
{{- end -}}

{{/*
Required claims serialized to CLI argument
*/}}
{{- define "requiredClaims" -}}
{{- if .Values.oidc.requiredClaims -}}
{{- $local := (list) -}}
{{- range $k, $v := .Values.oidc.requiredClaims -}}
{{- $local = (printf "%s=%s" $k $v | append $local) -}}
{{- end -}}
{{ join "," $local }}
{{- end -}}
{{- end -}}
3 changes: 2 additions & 1 deletion deploy/charts/version-checker/templates/deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ kind: Deployment
metadata:
name: {{ $chartname }}
labels:
{{ include "version-checker.labels" . | indent 4 }}
{{ include "version-checker.labels" . | nindent 2 }}
spec:
replicas: {{ .Values.replicaCount }}
selector:
Expand All @@ -18,6 +18,7 @@ spec:
metadata:
labels:
app: {{ $chartname }}
{{ include "version-checker.labels" . | nindent 8 }}
annotations:
prometheus.io/path: "/metrics"
prometheus.io/scrape: "true"
Expand Down
2 changes: 1 addition & 1 deletion deploy/charts/version-checker/templates/prometheus.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ metadata:
name: {{ include "version-checker.name" . }}
labels:
prometheus: {{ include "version-checker.name" . }}
{{ include "version-checker.labels" . | indent 4 }}
{{ include "version-checker.labels" . | nindent 4 }}
spec:
replicas: {{ .Values.prometheus.replicas }}
serviceAccountName: {{ .Values.prometheus.serviceAccountName }}
Expand Down
308 changes: 308 additions & 0 deletions deploy/charts/version-checker/tests/deployment_test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,308 @@
suite: test deployment
templates:
- deployment.yaml
tests:
- it: should work (defaults)
set:
image.tag: latest
asserts:
- isKind:
of: Deployment
- equal:
path: metadata.name
value: version-checker
- equal:
path: spec.template.spec.containers[0].image
value: quay.io/jetstack/version-checker:latest
- equal:
path: spec.template.spec.containers[0].resources
value: {}
- equal:
path: spec.template.metadata.labels["app.kubernetes.io/name"]
value: version-checker
- equal:
path: spec.template.metadata.labels["app.kubernetes.io/instance"]
value: RELEASE-NAME
- equal:
path: spec.template.spec.containers[0].name
value: version-checker
- equal:
path: spec.template.spec.containers[0].livenessProbe
value:
httpGet:
path: /readyz
port: 8080
initialDelaySeconds: 3
periodSeconds: 3
- equal:
path: spec.template.spec.containers[0].readinessProbe
value:
httpGet:
path: /readyz
port: 8080
initialDelaySeconds: 3
periodSeconds: 3
- isNullOrEmpty:
path: spec.template.spec.volumes

# ACR
- it: ACR should work
set:
acr.refreshToken: ajbhvdsbjvh
acr.username: jsgbjkas
acr.password: sgkjnabskjga
asserts:
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_ACR_REFRESH_TOKEN
valueFrom:
secretKeyRef:
key: acr.refreshToken
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_ACR_USERNAME
valueFrom:
secretKeyRef:
key: acr.username
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_ACR_PASSWORD
valueFrom:
secretKeyRef:
key: acr.password
name: version-checker

# ECR
- it: ECR should work
set:
ecr.iamRoleArn: ajbhvdsbjvh
ecr.accessKeyID: jsgbjkas
ecr.secretAccessKey: sgkjnabskjga
ecr.sessionToken: asgjasg
asserts:
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_ECR_IAM_ROLE_ARN
value: ajbhvdsbjvh
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_ECR_ACCESS_KEY_ID
valueFrom:
secretKeyRef:
key: ecr.accessKeyID
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_ECR_SECRET_ACCESS_KEY
valueFrom:
secretKeyRef:
key: ecr.secretAccessKey
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_ECR_SESSION_TOKEN
valueFrom:
secretKeyRef:
key: ecr.sessionToken
name: version-checker

# Docker
- it: Docker should work
set:
docker.token: ajbhvdsbjvh
docker.username: username
docker.password: hunter1
asserts:
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_DOCKER_TOKEN
valueFrom:
secretKeyRef:
key: docker.token
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_DOCKER_USERNAME
valueFrom:
secretKeyRef:
key: docker.username
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_DOCKER_PASSWORD
valueFrom:
secretKeyRef:
key: docker.password
name: version-checker

# GCR
- it: GCR should work
set:
gcr.token: ajbhvdsbjvh
asserts:
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_GCR_TOKEN
valueFrom:
secretKeyRef:
key: gcr.token
name: version-checker

# Quay
- it: Quay should work
set:
quay.token: ajbhvdsbjvh
asserts:
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_QUAY_TOKEN
valueFrom:
secretKeyRef:
key: quay.token
name: version-checker

# Self Hosted
- it: Self hosted should work
set:
selfhosted:
- name: bob
host: http://example.com
username: asgasasf
password: hunter1
asserts:
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_SELFHOSTED_HOST_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.host
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_SELFHOSTED_USERNAME_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.username
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_SELFHOSTED_PASSWORD_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.password
name: version-checker
- notContains:
path: spec.template.spec.containers[0].env
content:
name: VERSION_CHECKER_SELFHOSTED_TOKEN_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.password
name: version-checker

# Multiple Self Hosted
- it: Multiple Self hosted should work
set:
selfhosted:
- name: bob
host: http://example.com
username: asgasasf
password: hunter1
- name: bill
host: http://sub.example.com
token: askjgnasbjkgas
asserts:
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_SELFHOSTED_HOST_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.host
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_SELFHOSTED_USERNAME_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.username
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
count: 1
content:
name: VERSION_CHECKER_SELFHOSTED_PASSWORD_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.password
name: version-checker
- notContains:
path: spec.template.spec.containers[0].env
content:
name: VERSION_CHECKER_SELFHOSTED_TOKEN_bob
valueFrom:
secretKeyRef:
key: selfhosted.bob.password
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
content:
name: VERSION_CHECKER_SELFHOSTED_TOKEN_bill
valueFrom:
secretKeyRef:
key: selfhosted.bill.token
name: version-checker
- contains:
path: spec.template.spec.containers[0].env
content:
name: VERSION_CHECKER_SELFHOSTED_HOST_bill
valueFrom:
secretKeyRef:
key: selfhosted.bill.host
name: version-checker

- it: SecretEnabled
set:
acr.refreshToken: asgasga
asserts:
- contains:
path: spec.template.spec.volumes
content:
name: version-checker
secret:
secretName: version-checker
Loading