fix: update test

snyk · Jun 25, 2024 · 98989ef · 98989ef
1 parent 5bd1482
commit 98989ef
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/charts/snyk-broker/tests/__snapshot__/broker_deployment_test.yaml.snap b/charts/snyk-broker/tests/__snapshot__/broker_deployment_test.yaml.snap
@@ -719,6 +719,7 @@ handles duplicative labels:
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: snyk-broker-RELEASE-NAME
         helm.sh/chart: snyk-broker-0.0.0
+        test: node1
       name: github-com-broker-RELEASE-NAME
       namespace: NAMESPACE
     spec:
@@ -817,6 +818,7 @@ handles duplicative labels:
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: snyk-broker-RELEASE-NAME
         helm.sh/chart: snyk-broker-0.0.0
+        test: node1
       name: github-com-broker-service-RELEASE-NAME
       namespace: NAMESPACE
     spec:
@@ -844,6 +846,7 @@ handles duplicative labels:
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: snyk-broker-RELEASE-NAME
         helm.sh/chart: snyk-broker-0.0.0
+        test: node1
       name: snyk-broker-RELEASE-NAME
       namespace: NAMESPACE
 preflight checks off:

diff --git a/charts/snyk-broker/tests/broker_deployment_test.yaml b/charts/snyk-broker/tests/broker_deployment_test.yaml
@@ -33,19 +33,12 @@ tests:
       - ./fixtures/default_values_ha_on_4_replicas.yaml
     asserts:
       - matchSnapshot: {}
-  - it: add deployment labels
-    values:
-      - ./fixtures/default_values.yaml
-    set:
-      labels:
-        test: node1
-    asserts:
-      - matchSnapshot: {}
-  - it: handles duplicative labels
+  - it: handles duplicative labels and adds new labels
     values:
       - ./fixtures/default_values.yaml
     set:
       labels:
         app.kubernetes.io/name: "my duplicated label"
+        test: node1
     asserts:
       - matchSnapshot: {}