Skip to content

Commit bfc01c1

Browse files
authoredJul 29, 2021
Update chart verifier (#579)
The test names now have "v1.0/" prepended. chart-verifier docker images are now tagged too.
1 parent a5c8562 commit bfc01c1

File tree

2 files changed

+15
-16
lines changed

2 files changed

+15
-16
lines changed
 

‎.circleci/config.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ jobs:
1616
- image: docker.mirror.hashicorp.services/cimg/go:1.16
1717
environment:
1818
BATS_VERSION: "1.3.0"
19-
CHART_VERIFIER_VERSION: "1.0.0"
19+
CHART_VERIFIER_VERSION: "1.2.1"
2020
steps:
2121
- checkout
2222
- run:

‎test/chart/verifier.bats

+14-15
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,9 @@ setup_file() {
66
cd `chart_dir`
77
export VERIFY_OUTPUT="/$BATS_RUN_TMPDIR/verify.json"
88
export CHART_VOLUME=vault-helm-chart-src
9-
# Note: currently `latest` is the only tag available in the chart-verifier repo.
10-
local IMAGE="quay.io/redhat-certification/chart-verifier:latest"
9+
local IMAGE="quay.io/redhat-certification/chart-verifier:1.2.1"
1110
# chart-verifier requires an openshift version if a cluster isn't available
12-
local OPENSHIFT_VERSION="4.7"
11+
local OPENSHIFT_VERSION="4.8"
1312
local DISABLED_TESTS="chart-testing"
1413

1514
local run_cmd="chart-verifier"
@@ -24,7 +23,7 @@ setup_file() {
2423
# Make sure we have the latest version of chart-verifier
2524
docker pull $IMAGE
2625
# Start chart-verifier using this volume
27-
run_cmd="docker run --rm --volumes-from $CHART_VOLUME $IMAGE"
26+
run_cmd="docker run --rm --volumes-from $CHART_VOLUME -w $chart_src $IMAGE"
2827
fi
2928

3029
$run_cmd verify $chart_src \
@@ -41,46 +40,46 @@ teardown_file() {
4140
}
4241

4342
@test "has-kubeversion" {
44-
check_result has-kubeversion
43+
check_result v1.0/has-kubeversion
4544
}
4645

4746
@test "is-helm-v3" {
48-
check_result is-helm-v3
47+
check_result v1.0/is-helm-v3
4948
}
5049

5150
@test "not-contains-crds" {
52-
check_result not-contains-crds
51+
check_result v1.0/not-contains-crds
5352
}
5453

5554
@test "helm-lint" {
56-
check_result helm-lint
55+
check_result v1.0/helm-lint
5756
}
5857

5958
@test "not-contain-csi-objects" {
60-
check_result not-contain-csi-objects
59+
check_result v1.0/not-contain-csi-objects
6160
}
6261

6362
@test "has-readme" {
64-
check_result has-readme
63+
check_result v1.0/has-readme
6564
}
6665

6766
@test "contains-values" {
68-
check_result contains-values
67+
check_result v1.0/contains-values
6968
}
7069

7170
@test "contains-values-schema" {
72-
check_result contains-values-schema
71+
check_result v1.0/contains-values-schema
7372
}
7473

7574
@test "contains-test" {
76-
check_result contains-test
75+
check_result v1.0/contains-test
7776
}
7877

7978
@test "images-are-certified" {
80-
check_result images-are-certified
79+
check_result v1.0/images-are-certified
8180
}
8281

8382
@test "chart-testing" {
8483
skip "Skipping since this test requires a kubernetes/openshift cluster"
85-
check_result chart-testing
84+
check_result v1.0/chart-testing
8685
}

0 commit comments

Comments
 (0)
Please sign in to comment.