Update unit tests to be current with most recent helm unittest syntax (#619)

- Switch to the canonical helm unittest isntead of an outdated fork.
- Update the tests to work with latest syntax.

### Checklist

- [ ] Follow the [manual testing
process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md)
- [ ] Update
[changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md)
- [ ] Update [Kubernetes update
doc](https://docs.sourcegraph.com/admin/updates/kubernetes)

### Test plan

1. uninstall the outdated helm unittest fork: `helm plugin uninstall
unittest`
2. install the canonical tool: `helm plugin install
https://github.com/helm-unittest/helm-unittest`
3. run the unit tests: `helm unittest ./charts/sourcegraph/.`

Author: peterguy

TEST.md

@@ -12,12 +12,12 @@ helm lint charts/sourcegraph/.
 
 ## Unit testing
 
-We utilize [helm-unittest](https://github.com/quintush/helm-unittest/), a BDD styled unit test framework, to validate our helm chart.
+We utilize [helm-unittest](https://github.com/helm-unittest/helm-unittest/), a BDD styled unit test framework, to validate our helm chart.
 
 helm-unittest can be installed with:
 
 ```bash
-helm plugin install https://github.com/quintush/helm-unittest
+helm plugin install https://github.com/helm-unittest/helm-unittest
 ```
 
 Once the plugin is installed, you can run the unit tests using the following:
@@ -28,7 +28,7 @@ helm unittest --helm3 ./charts/sourcegraph/.
 
 We currently do not have testing best practices or require unit tests for new changes, so add test cases at your best judgement if possible.
 
-You may check out our [existing test cases](https://github.com/sourcegraph/deploy-sourcegraph-helm/tree/main/charts/sourcegraph/tests) and helm-unittest [docs](https://github.com/quintush/helm-unittest/blob/master/DOCUMENT.md).
+You may check out our [existing test cases](https://github.com/sourcegraph/deploy-sourcegraph-helm/tree/main/charts/sourcegraph/tests) and helm-unittest [docs](https://github.com/helm-unittest/helm-unittest/blob/master/DOCUMENT.md).
 
 ## Manual testing
 

charts/sourcegraph/tests/checksum_test.yaml

@@ -10,15 +10,15 @@ tests:
         existingSecret: "my-secret"
   asserts:
     - matchSnapshot:
-          path: spec.template.metadata.annotations.[checksum/auth]
+          path: spec.template.metadata.annotations["checksum/auth"]
 - it: should update checksum when codeintel-db secret is modified
   set:
     codeIntelDB:
       auth:
         newkey: "blank"
   asserts:
     - matchSnapshot:
-          path: spec.template.metadata.annotations.[checksum/auth]
+          path: spec.template.metadata.annotations["checksum/auth"]
 - it: should update checksum when codeinsights-db secret is modified
   set:
     codeInsightsDB:
@@ -27,20 +27,20 @@ tests:
         database: "anotherdb"
   asserts:
     - matchSnapshot:
-          path: spec.template.metadata.annotations.[checksum/auth]
+          path: spec.template.metadata.annotations["checksum/auth"]
 - it: should update checksum when redis-store secret is modified
   set:
     redisStore:
       connection:
         endpoint: "external-redis:6379"
   asserts:
     - matchSnapshot:
-          path: spec.template.metadata.annotations.[checksum/auth]
+          path: spec.template.metadata.annotations["checksum/auth"]
 - it: should update checksum when redis-cache secret is modified
   set:
     redisCache:
       connection:
         endpoint: "external-redis:6379"
   asserts:
     - matchSnapshot:
-          path: spec.template.metadata.annotations.[checksum/auth]
+          path: spec.template.metadata.annotations["checksum/auth"]

charts/sourcegraph/tests/codeInsightsDBAdditionalConfig_test.yaml

@@ -15,7 +15,7 @@ tests:
         max_connections = 3000000000000000000
   asserts:
   - matchRegex:
-      path: data.postgresql\.conf
+      path: data["postgresql.conf"]
       pattern: max_connections = 3000000000000000000
 - it: should fail to render when both codeInsightsDB.additionalConfig and codeInsightsDB.existingConfig are defined
   set:

charts/sourcegraph/tests/codeIntelDBAdditionalConfig_test.yaml

@@ -15,7 +15,7 @@ tests:
         max_connections = 3000000000000000000
   asserts:
   - matchRegex:
-      path: data.postgresql\.conf
+      path: data["postgresql.conf"]
       pattern: max_connections = 2000000000000000000
 - it: should fail to render when both codeIntelDB.additionalConfig and codeIntelDB.existingConfig are defined
   set:

charts/sourcegraph/tests/defaultTagPrefix_test.yaml

@@ -9,18 +9,18 @@ tests:
     frontend:
       image:
         defaultTag: test
-  matchRegexRaw:
-    path: "spec.template.spec.containers[0].image"
-    pattern: |
-      "[\w-]+\.\w+\.\w+\/.+\/.+:test"
+  asserts:
+    - matchRegex:
+        path: "spec.template.spec.containers[0].image"
+        pattern: ^[\w-]+\.\w+\.\w+\/.+\/.+:test$
 - it: should add defaultTagPrefix
   template: frontend/sourcegraph-frontend.Deployment.yaml
   set:
     frontend:
       image:
         defaultTag: test
         defaultTagPrefix: "prefix-"
-  matchRegexRaw:
-    path: "spec.template.spec.containers[0].image"
-    pattern: |
-      "[\w-]+\.\w+\.\w+\/.+\/.+:prefix-test"
+  asserts:
+    - matchRegex:
+        path: "spec.template.spec.containers[0].image"
+        pattern: ^[\w-]+\.\w+\.\w+\/.+\/.+:prefix-test$

charts/sourcegraph/tests/localDevMode_test.yaml

@@ -6,12 +6,12 @@ tests:
     set:
       sourcegraph.localDevMode: true
     asserts:
-      - isEmpty:
-          path: spec.template.spec.containers[0].resources
+      - notExists:
+          path: "spec.template.spec.containers[0].resources"
 
   - it: should have a resource block when localDevMode=false
     set:
       sourcegraph.localDevMode: false
     asserts:
-      - isNotEmpty:
-          path: spec.template.spec.containers[0].resources
+      - exists:
+          path: "spec.template.spec.containers[0].resources"

charts/sourcegraph/tests/nodeExporter_test.yaml

@@ -102,7 +102,7 @@ tests:
           create: false
           name: "test-service-account-name"
     asserts:
-      - isEmpty:
+      - notExists:
           path: spec.template.spec.serviceAccountName
         template: node-exporter/node-exporter.DaemonSet.yaml
 

charts/sourcegraph/tests/otelCollectorJaeger_test.yaml

@@ -65,7 +65,7 @@ tests:
           count: 1 # ConfigMap is rendered
         template: otel-collector/otel-collector.ConfigMap.yaml
       - matchRegex:
-          path: data.[config.yaml]
+          path: data["config.yaml"]
           pattern: "endpoint: otlp.service.com:443" # check if element of config is substituted correctly
         template: otel-collector/otel-collector.ConfigMap.yaml
   - it: should fail if a configuration is provided and jaeger is enabled

charts/sourcegraph/tests/pgsqlAdditionalConfig_test.yaml

@@ -16,7 +16,7 @@ tests:
         max_connections = 3000000000000000000
   asserts:
   - matchRegex:
-      path: data.postgresql\.conf
+      path: data["postgresql.conf"]
       pattern: max_connections = 1000000000000000000
 - it: should fail to render when both pgsql.additionalConfig and pgsql.existingConfig are defined
   set:

charts/sourcegraph/tests/pgsqlAuth_test.yaml

@@ -18,7 +18,7 @@ tests:
           path: spec.template.spec.containers[0].env[0].valueFrom.secretKeyRef.name
           value: "my-secret"
       - matchSnapshot:
-            path: spec.template.metadata.annotations.[checksum/pgsql.secret]
+            path: spec.template.metadata.annotations["checksum/pgsql.secret"]
 - it: should not generate a secret when existingSecret is passed
   template: pgsql/pgsql.Secret.yaml
   set:
@@ -62,4 +62,4 @@ tests:
           path: spec.template.spec.containers[0].env[0].valueFrom.secretKeyRef.name
           value: "pgsql-auth"
       - matchSnapshot:
-            path: spec.template.metadata.annotations.[checksum/pgsql.secret]
+            path: spec.template.metadata.annotations["checksum/pgsql.secret"]