chore(repo): Add new test instance for protect service (#7242)

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: wobsoriano

.changeset/beige-sloths-provide.md

@@ -3,4 +3,4 @@
 "@clerk/shared": minor
 ---
 
-Introduce ProtectConfig resource and Protect loader integration.
+Introduced initial Clerk Protect dynamic loader and related types to support dynamically enabling and rolling out Protect in the environment.

integration/.keys.json.sample

@@ -58,5 +58,9 @@
   "with-api-keys": {
     "pk": "",
     "sk": ""
+  },
+  "with-protect-service": {
+    "pk": "",
+    "sk": ""
   }
 }

integration/presets/envs.ts

@@ -185,6 +185,12 @@ const withAPIKeys = base
   .setEnvVariable('private', 'CLERK_SECRET_KEY', instanceKeys.get('with-api-keys').sk)
   .setEnvVariable('public', 'CLERK_PUBLISHABLE_KEY', instanceKeys.get('with-api-keys').pk);
 
+const withProtectService = base
+  .clone()
+  .setId('withProtectService')
+  .setEnvVariable('private', 'CLERK_SECRET_KEY', instanceKeys.get('with-protect-service').sk)
+  .setEnvVariable('public', 'CLERK_PUBLISHABLE_KEY', instanceKeys.get('with-protect-service').pk);
+
 export const envs = {
   base,
   sessionsProd1,
@@ -212,4 +218,5 @@ export const envs = {
   withSignInOrUpwithRestrictedModeFlow,
   withWaitlistdMode,
   withWhatsappPhoneCode,
+  withProtectService,
 } as const;

integration/tests/protect-service.test.ts

@@ -17,60 +17,66 @@ const mockProtectSettings = async (page: Page, config?: ProtectConfigJSON) => {
   });
 };
 
-testAgainstRunningApps({ withEnv: [appConfigs.envs.withEmailCodes] })('Clerk Protect checks @generic', ({ app }) => {
-  test.describe.configure({ mode: 'parallel' });
+testAgainstRunningApps({ withEnv: [appConfigs.envs.withProtectService] })(
+  'Clerk Protect checks @generic',
+  ({ app }) => {
+    test.describe.configure({ mode: 'parallel' });
 
-  test.afterAll(async () => {
-    await app.teardown();
-  });
-
-  test('should add loader script when protect_config.loader is set', async ({ page, context }) => {
-    const u = createTestUtils({ app, page, context });
-    await mockProtectSettings(page, {
-      object: 'protect_config',
-      id: 'n',
-      loaders: [
-        {
-          rollout: 1.0,
-          type: 'script',
-          target: 'body',
-          attributes: { id: 'test-protect-loader-1', type: 'module', src: 'data:application/json;base64,Cgo=' },
-        },
-      ],
+    test.afterAll(async () => {
+      await app.teardown();
     });
-    await u.page.goToAppHome();
-    await u.page.waitForClerkJsLoaded();
 
-    await expect(page.locator('#test-protect-loader-1')).toHaveAttribute('type', 'module');
-  });
+    test('should add loader script when protect_config.loader is set', async ({ page, context }) => {
+      const u = createTestUtils({ app, page, context });
+      await mockProtectSettings(page, {
+        object: 'protect_config',
+        id: 'n',
+        loaders: [
+          {
+            rollout: 1.0,
+            type: 'script',
+            target: 'body',
+            attributes: { id: 'test-protect-loader-1', type: 'module', src: 'data:application/json;base64,Cgo=' },
+          },
+        ],
+      });
+      await u.page.goToAppHome();
+      await u.page.waitForClerkJsLoaded();
 
-  test('should not add loader script when protect_config.loader is set and rollout 0.00', async ({ page, context }) => {
-    const u = createTestUtils({ app, page, context });
-    await mockProtectSettings(page, {
-      object: 'protect_config',
-      id: 'n',
-      loaders: [
-        {
-          rollout: 0, // force 0% rollout, should not materialize
-          type: 'script',
-          target: 'body',
-          attributes: { id: 'test-protect-loader-2', type: 'module', src: 'data:application/json;base64,Cgo=' },
-        },
-      ],
+      await expect(page.locator('#test-protect-loader-1')).toHaveAttribute('type', 'module');
     });
-    await u.page.goToAppHome();
-    await u.page.waitForClerkJsLoaded();
 
-    await expect(page.locator('#test-protect-loader-2')).toHaveCount(0);
-  });
+    test('should not add loader script when protect_config.loader is set and rollout 0.00', async ({
+      page,
+      context,
+    }) => {
+      const u = createTestUtils({ app, page, context });
+      await mockProtectSettings(page, {
+        object: 'protect_config',
+        id: 'n',
+        loaders: [
+          {
+            rollout: 0, // force 0% rollout, should not materialize
+            type: 'script',
+            target: 'body',
+            attributes: { id: 'test-protect-loader-2', type: 'module', src: 'data:application/json;base64,Cgo=' },
+          },
+        ],
+      });
+      await u.page.goToAppHome();
+      await u.page.waitForClerkJsLoaded();
 
-  test('should not create loader element when protect_config.loader is not set', async ({ page, context }) => {
-    const u = createTestUtils({ app, page, context });
-    await mockProtectSettings(page);
-    await u.page.goToAppHome();
-    await u.page.waitForClerkJsLoaded();
+      await expect(page.locator('#test-protect-loader-2')).toHaveCount(0);
+    });
 
-    // Playwright locators are always objects, never undefined
-    await expect(page.locator('#test-protect-loader')).toHaveCount(0);
-  });
-});
+    test('should not create loader element when protect_config.loader is not set', async ({ page, context }) => {
+      const u = createTestUtils({ app, page, context });
+      await mockProtectSettings(page);
+      await u.page.goToAppHome();
+      await u.page.waitForClerkJsLoaded();
+
+      // Playwright locators are always objects, never undefined
+      await expect(page.locator('#test-protect-loader')).toHaveCount(0);
+    });
+  },
+);