From 52ab027cc3ff8407ef7ea442799f391b2e98ca10 Mon Sep 17 00:00:00 2001
From: Jake Barnby <jakeb994@gmail.com>
Date: Wed, 1 Apr 2026 18:18:47 +1300
Subject: [PATCH] (fix): use test.skip in beforeEach for CI-skipped E2E tests

---
 e2e/journeys/dedicated-databases.spec.ts | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/e2e/journeys/dedicated-databases.spec.ts b/e2e/journeys/dedicated-databases.spec.ts
index 261a14f18..bbdf79e5e 100644
--- a/e2e/journeys/dedicated-databases.spec.ts
+++ b/e2e/journeys/dedicated-databases.spec.ts
@@ -1,8 +1,4 @@
-import { test as base, expect, type Page } from '@playwright/test';
-
-// @todo These tests require a pre-seeded project with dedicated database support.
-// Skip in CI until a dedicated test environment is available.
-const test = process.env.CI ? base.skip : base;
+import { test, expect, type Page } from '@playwright/test';
 
 const PROJECT_ID = '69c5061ee68ebce1a541';
 const REGION = 'fra';
@@ -120,6 +116,9 @@ async function navigateToFirstDatabase(page: Page): Promise<boolean> {
 
 test.describe('Dedicated databases', () => {
     test.beforeEach(async ({ page }) => {
+        // @todo These tests require a pre-seeded project with dedicated database support.
+        // Skip in CI until a dedicated test environment is available.
+        test.skip(!!process.env.CI, 'Requires pre-seeded dedicated database environment');
         await authenticate(page);
     });