Skip to content

Commit

Permalink
Commenting test case on jcasc-config.yaml in jenkins-controller-ingre…
Browse files Browse the repository at this point in the history
…ss-DRY-test.yaml

Due to limitation in helm-unittest about validating a YAML content into a string field,  the test case on jcasc-config.yaml is momentarily commented.

Nonetheless, a visual validation of https://github.com/jenkinsci/helm-charts/actions/runs/6493066967/job/17633246976?pr=929 actual value shows 'unclassified.location.url' = "https://jenkins.example.com" as expected (https://github.com/jenkinsci/helm-charts/actions/runs/6493066967/job/17633246976?pr=929#step:8:131)
  • Loading branch information
Ceddaerrix authored Oct 12, 2023
1 parent 1a7eaf6 commit c56d7b5
Showing 1 changed file with 16 additions and 16 deletions.
32 changes: 16 additions & 16 deletions charts/jenkins/unittests/jenkins-controller-ingress-DRY-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,23 +24,23 @@ tests:
- equal:
path: spec.tls[0].hosts[0]
value: "jenkins.example.com"
- it: jcasc-config
template: jcasc-config.yaml
set:
global.jenkinsHostname: "jenkins.example.com"
controller.ingress:
enabled: true
hostName: "{{ .Values.global.jenkinsHostname }}"
resourceRootUrl: "{{ .Values.global.jenkinsHostname }}"
tls:
- hosts:
- "{{ .Values.global.jenkinsHostname }}"
asserts:
# - it: jcasc-config
# template: jcasc-config.yaml
# set:
# global.jenkinsHostname: "jenkins.example.com"
# controller.ingress:
# enabled: true
# hostName: "{{ .Values.global.jenkinsHostname }}"
# resourceRootUrl: "{{ .Values.global.jenkinsHostname }}"
# tls:
# - hosts:
# - "{{ .Values.global.jenkinsHostname }}"
# asserts:
# - exists:
# path: data.[jcasc-default-config.yaml]
# - isNotNullOrEmpty:
# path: data.[jcasc-default-config.yaml]
- matchRegex:
path: data.[jcasc-default-config.yaml]
pattern: |
(?m).+url: https://jenkins.example.com.*
# - matchRegex:
# path: data.[jcasc-default-config.yaml]
# pattern: |
# url: https://jenkins.example.com

0 comments on commit c56d7b5

Please sign in to comment.