Add graceful cleanup for redis cluster e2e test (#1493)

Signed-off-by: Deepak <sah.sslpu@gmail.com>
kedacore · Jan 9, 2021 · 38c76c5 · 38c76c5
1 parent 0928097
commit 38c76c5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/tests/scalers/redis-cluster-lists.test.ts b/tests/scalers/redis-cluster-lists.test.ts
@@ -251,8 +251,8 @@ test.after.always.cb('clean up deployment', t => {
         `scaledobject.keda.sh/${redisWorkerHostPortRefDeploymentName}`,
         `scaledobject.keda.sh/${redisWorkerAddressRefDeploymentName}`,
         `scaledobject.keda.sh/${redisWorkerHostPortRefTriggerAuthDeploymentName}`,
-        'triggerauthentication.keda.sh/keda-redis-list-triggerauth',
-        'triggerauthentication.keda.sh/keda-redis-list-triggerauth-host-port',
+        'triggerauthentication.keda.sh/keda-redis-cluster-list-triggerauth',
+        'triggerauthentication.keda.sh/keda-redis-cluster-list-triggerauth-host-port',
         `deployment/${redisWorkerAddressRefDeploymentName}`,
         `deployment/${redisWorkerHostPortRefTriggerAuthDeploymentName}`,
         `deployment/${redisWorkerHostPortRefDeploymentName}`,
@@ -264,6 +264,7 @@ test.after.always.cb('clean up deployment', t => {
     }
     sh.exec(`kubectl delete namespace ${testNamespace}`)
 
+    sh.exec(`helm delete ${redisClusterName} --namespace ${redisNamespace}`)
     sh.exec(`kubectl delete namespace ${redisNamespace}`)
     t.end()
 })

diff --git a/tests/scalers/redis-cluster-streams.test.ts b/tests/scalers/redis-cluster-streams.test.ts
@@ -10,8 +10,6 @@ const redisService = 'redis-cluster-streams'
 const testNamespace = 'redis-cluster-streams-test'
 const redisPassword = 'foobared'
 let redisHost = ''
-let redisAddress = ''
-const redisPort = 6379
 const numMessages = 100
 
 test.before(t => {
@@ -35,7 +33,6 @@ test.before(t => {
 
     // Get Redis cluster address.
     redisHost = sh.exec(`kubectl get svc ${redisService} -n ${redisNamespace} -o jsonpath='{.spec.clusterIP}'`)
-    redisAddress = `${redisHost}:${redisPort}`
 
     // Create test namespace.
     sh.exec(`kubectl create namespace ${testNamespace}`)
@@ -114,13 +111,15 @@ test.after.always.cb('clean up deployment', t => {
     'triggerauthentications.keda.sh/keda-redis-stream-triggerauth',
     'secret/redis-password',
     'deployment/redis-streams-consumer',
+    'job/redis-streams-producer',
   ]
 
   for (const resource of resources) {
     sh.exec(`kubectl delete ${resource} --namespace ${testNamespace}`)
   }
   sh.exec(`kubectl delete namespace ${testNamespace}`)
 
+  sh.exec(`helm delete ${redisClusterName} --namespace ${redisNamespace}`)
   sh.exec(`kubectl delete namespace ${redisNamespace}`)
   t.end()
 })