Fix unit tests with changed file paths

Author: marcleblanc2

charts/sourcegraph/tests/cadvisorPrivileged_test.yaml

@@ -1,6 +1,6 @@
 suite: cadvisorPrivileged
 templates:
-  - NOTES.txt
+  - other-resources/NOTES.txt
   - cadvisor/cadvisor.DaemonSet.yaml
 tests:
   - it: should not have the warning text when cadvisor.containerSecurityContext.privileged=true
@@ -11,7 +11,7 @@ tests:
     asserts:
       - notMatchRegexRaw:
           pattern: You have set 'cadvisor.containerSecurityContext.privileged' to 'false'
-        template: NOTES.txt
+        template: other-resources/NOTES.txt
   - it: should have the warning text when prometheus.privileged=false
     set:
       cadvisor:
@@ -20,7 +20,7 @@ tests:
     asserts:
       - matchRegexRaw:
           pattern: You have set 'cadvisor.containerSecurityContext.privileged' to 'false'
-        template: NOTES.txt
+        template: other-resources/NOTES.txt
   - it: should render /dev/kmsg in volumes and volumeMounts when cadvisor.containerSecurityContext.privileged=true
     set:
       cadvisor:

charts/sourcegraph/tests/nodeExporter_test.yaml

@@ -1,6 +1,6 @@
 suite: nodeExporter
 templates:
-  - NOTES.txt
+  - other-resources/NOTES.txt
   - node-exporter/node-exporter.ClusterRole.yaml
   - node-exporter/node-exporter.ClusterRoleBinding.yaml
   - node-exporter/node-exporter.DaemonSet.yaml
@@ -9,7 +9,7 @@ templates:
   - node-exporter/node-exporter.ServiceAccount.yaml
 tests:
   - it: should render the DaemonSet and Service if node-exporter is enabled
-    set: 
+    set:
       nodeExporter:
         enabled: true
     asserts:
@@ -25,11 +25,11 @@ tests:
         template: node-exporter/node-exporter.Service.yaml
 
   - it: should not render any resources if node-exporter is disabled
-    set: 
+    set:
       nodeExporter:
         enabled: false
     asserts:
-      - hasDocuments: 
+      - hasDocuments:
           count: 0
     templates:
       - node-exporter/node-exporter.ClusterRole.yaml
@@ -41,25 +41,25 @@ tests:
 
   - it: should not render the PodSecurityPolicy, Service Accounts, and ClusterRoles by default
     asserts:
-      - hasDocuments: 
+      - hasDocuments:
           count: 0
         template: node-exporter/node-exporter.PodSecurityPolicy.yaml
-      - hasDocuments: 
+      - hasDocuments:
           count: 0
         template: node-exporter/node-exporter.ClusterRole.yaml
-      - hasDocuments: 
+      - hasDocuments:
             count: 0
         template: node-exporter/node-exporter.ClusterRoleBinding.yaml
-      - hasDocuments: 
+      - hasDocuments:
             count: 0
         template: node-exporter/node-exporter.ServiceAccount.yaml
 
   - it: should render the PodSecurityPolicy, ServiceAccounts, and ClusterRoles if PodSecurityPolicy + Service Accounts are enabled
-    set: 
+    set:
       nodeExporter:
         serviceAccount:
           create: true
-        podSecurityPolicy: 
+        podSecurityPolicy:
           enabled: true
     asserts:
       - containsDocument:
@@ -82,9 +82,9 @@ tests:
           apiVersion: rbac.authorization.k8s.io/v1
           name: node-exporter
         template: node-exporter/node-exporter.ClusterRoleBinding.yaml
-  
+
   - it: should add the ServiceAccount name to the DaemonSet spec if the ServiceAccount is enabled
-    set: 
+    set:
       nodeExporter:
         serviceAccount:
           create: true
@@ -94,9 +94,9 @@ tests:
           path: spec.template.spec.serviceAccountName
           value: "test-service-account-name"
         template: node-exporter/node-exporter.DaemonSet.yaml
-    
+
   - it: should add not the ServiceAccount name to the DaemonSet spec if the ServiceAccount is disabled
-    set: 
+    set:
       nodeExporter:
         serviceAccount:
           create: false
@@ -105,30 +105,30 @@ tests:
       - notExists:
           path: spec.template.spec.serviceAccountName
         template: node-exporter/node-exporter.DaemonSet.yaml
-  
+
   - it: should fail to render if the PodSecurityPolicy is enabled, but the ServiceAccount isn't
-    set: 
+    set:
       nodeExporter:
         serviceAccount:
           create: false
-        podSecurityPolicy: 
+        podSecurityPolicy:
           enabled: true
     asserts:
       - failedTemplate:
           errorMessage: "Node Exporter's service account must be enabled in order to use its pod security policy (set 'nodeExporter.ServiceAccount.create' to true)"
         template: node-exporter/node-exporter.PodSecurityPolicy.yaml
 
-  - it: should not render the podSecurityPolicy if disabled 
-    set: 
+  - it: should not render the podSecurityPolicy if disabled
+    set:
       nodeExporter:
-        podSecurityPolicy: 
+        podSecurityPolicy:
           enabled: false
     asserts:
       - hasDocuments:
           count: 0
         template: node-exporter/node-exporter.PodSecurityPolicy.yaml
-  
-  - it: should ensure that the namespace is properly propagated to the cluster role binding 
+
+  - it: should ensure that the namespace is properly propagated to the cluster role binding
     release:
       namespace: "my-test-namespace"
     set:
@@ -151,16 +151,16 @@ tests:
           create: true
         podSecurityPolicy:
           enabled: true
-    asserts:  
-      - equal: 
+    asserts:
+      - equal:
           path: spec.template.spec.hostPID
           value: true
         template: node-exporter/node-exporter.DaemonSet.yaml
-      - equal: 
+      - equal:
           path: spec.hostPID
           value: true
         template: node-exporter/node-exporter.PodSecurityPolicy.yaml
-  
+
   - it: should propagate host PID settings to both the daemonset and podSecurityPolicy
     set:
       nodeExporter:
@@ -170,12 +170,12 @@ tests:
           create: true
         podSecurityPolicy:
           enabled: true
-    asserts:  
-      - equal: 
+    asserts:
+      - equal:
           path: spec.template.spec.hostPID
           value: false
         template: node-exporter/node-exporter.DaemonSet.yaml
-      - equal: 
+      - equal:
           path: spec.hostPID
           value: false
         template: node-exporter/node-exporter.PodSecurityPolicy.yaml
@@ -184,34 +184,34 @@ tests:
     set:
       nodeExporter:
         enabled: true
-    asserts:  
+    asserts:
       - notMatchRegexRaw:
           pattern: You have set 'nodeExporter.enabled' to 'false'
-        template: NOTES.txt
-  
+        template: other-resources/NOTES.txt
+
   - it: should generate warnings if node-exporter is disabled
     set:
       nodeExporter:
         enabled: false
-    asserts:  
+    asserts:
       - matchRegexRaw:
           pattern: You have set 'nodeExporter.enabled' to 'false'
-        template: NOTES.txt
+        template: other-resources/NOTES.txt
 
   - it: should not generate warnings if hostPID is true
     set:
       nodeExporter:
         hostPID: true
-    asserts:  
+    asserts:
       - notMatchRegexRaw:
           pattern: You have set 'nodeExporter.hostPID' to 'false'
-        template: NOTES.txt
+        template: other-resources/NOTES.txt
 
-  - it: should generate warnings if hostPID is false 
+  - it: should generate warnings if hostPID is false
     set:
       nodeExporter:
         hostPID: false
-    asserts:  
+    asserts:
       - matchRegexRaw:
           pattern: You have set 'nodeExporter.hostPID' to 'false'
-        template: NOTES.txt
+        template: other-resources/NOTES.txt

charts/sourcegraph/tests/priorityClass_test.yaml

@@ -4,7 +4,7 @@ release:
   namespace: sourcegraph
 tests:
 - it: should render single priority class
-  template: priorityClass.yaml
+  template: other-resources/priorityClass.yaml
   set:
     priorityClasses:
     - name: gitserver-class
@@ -25,7 +25,7 @@ tests:
       path: description
       value: "gitserver priority class"
 - it: should render two priority classes
-  template: priorityClass.yaml
+  template: other-resources/priorityClass.yaml
   set:
     priorityClasses:
     - name: gitserver-class
@@ -97,7 +97,7 @@ tests:
       path: spec.template.spec.priorityClassName
       value: searcher-class
 - it: set priority class on redisCache
-  template: redis/redis-cache.Deployment.yaml
+  template: redis-cache/redis-cache.Deployment.yaml
   set:
     redisCache:
       priorityClassName: redis-cache-class
@@ -106,7 +106,7 @@ tests:
       path: spec.template.spec.priorityClassName
       value: redis-cache-class
 - it: set priority class on redisStore
-  template: redis/redis-store.Deployment.yaml
+  template: redis-store/redis-store.Deployment.yaml
   set:
     redisStore:
       priorityClassName: redis-store-class

charts/sourcegraph/tests/prometheusPrivileged_test.yaml

@@ -1,6 +1,6 @@
 suite: prometheusPrivileged
 templates:
-  - NOTES.txt
+  - other-resources/NOTES.txt
 tests:
   - it: should not have the warning text when prometheus.privileged=true
     set:

charts/sourcegraph/tests/storageClass_test.yaml

@@ -1,6 +1,6 @@
 suite: storageClass
 templates:
-- storageclass.yaml
+- other-resources/storageclass.yaml
 tests:
 - it: should have parameters rendered when storageClass.parameters.zones=us-central1-f
   set: