[release-1.5] test: update redis tests (#2480)

* Update redis tests

* remove useRedisSets option

* Update verify-redis-cache.spec.ts

* update redis test

* Update verify-redis-cache.spec.ts

* update redis test

---------

Co-authored-by: Alessandro Barbarossa <[email protected]>

Author: openshift-cherrypick-robot

.ibm/pipelines/resources/config_map/app-config-rhdh.yaml

@@ -36,7 +36,8 @@ backend:
   cache:
     store: redis
     connection: redis://${REDIS_USERNAME}:${REDIS_PASSWORD}@redis:6379
-    useRedisSets: true
+    # redis sets are no longer supported from 1.5
+    # useRedisSets: true
 integrations:
   # Plugin: GitHub
   github:
@@ -174,4 +175,4 @@ argocd:
           url: temp
           token: temp
 permission:
-  enabled: false
+  enabled: false

e2e-tests/playwright/e2e/verify-redis-cache.spec.ts

@@ -23,10 +23,20 @@ test.describe("Verify Redis Cache DB", () => {
   });
 
   test("Open techdoc and verify the cache generated in redis db", async () => {
+    test.setTimeout(120_000);
+
     await uiHelper.openSidebarButton("Favorites");
     await uiHelper.openSidebar("Docs");
     await uiHelper.clickLink("Backstage Showcase");
 
+    // ensure that the docs are generated. if redis configuration has an error, this page will hang and docs won't be generated
+    await expect(async () => {
+      await uiHelper.verifyHeading("rhdh");
+    }).toPass({
+      intervals: [3_000],
+      timeout: 60_000,
+    });
+
     // Wait for port-forward to be ready
     await new Promise<void>((resolve, reject) => {
       portForward.stdout.on("data", (data) => {
@@ -46,9 +56,15 @@ test.describe("Verify Redis Cache DB", () => {
     await expect(async () => {
       const keys = await redis.keys("*");
       expect(keys).toContainEqual(expect.stringContaining("techdocs"));
+
+      // Additionally, verify the format of the key
+      const key = keys.filter((k) => k.startsWith("techdocs"))[0];
+      expect(key).toMatch(
+        /techdocs:(?:[A-Za-z0-9+]{4})*(?:[A-Za-z0-9+]{2}==|[A-Za-z0-9+]{3}=)$/gm,
+      );
     }).toPass({
       intervals: [3_000],
-      timeout: 30_000,
+      timeout: 60_000,
     });
   });