From 9c6694cad4cde5f37fbf876287006d507cfdbe14 Mon Sep 17 00:00:00 2001 From: igm Date: Mon, 12 Jan 2026 15:15:15 +0800 Subject: [PATCH] Auto-skip e2e tests when TEST_DATABASE_URL is not configured Previously, e2e tests would fail with a confusing URL parse error when TEST_DATABASE_URL was not set. Now SKIP_DB_TESTS automatically becomes true when the database URL is missing, gracefully skipping these tests. Co-Authored-By: Claude Opus 4.5 --- packages/testing/test-helpers/src/skip-db-tests.ts | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/testing/test-helpers/src/skip-db-tests.ts b/packages/testing/test-helpers/src/skip-db-tests.ts index cb836e6..5ca7830 100644 --- a/packages/testing/test-helpers/src/skip-db-tests.ts +++ b/packages/testing/test-helpers/src/skip-db-tests.ts @@ -2,9 +2,12 @@ import { describe } from "bun:test"; /** * Skip flag for database-dependent tests. - * Set SKIP_DB_TESTS=1 to skip e2e tests that require a database. + * Tests are skipped when: + * - SKIP_DB_TESTS=1 is explicitly set, OR + * - TEST_DATABASE_URL is not configured */ -export const SKIP_DB_TESTS: boolean = process.env.SKIP_DB_TESTS === "1"; +export const SKIP_DB_TESTS: boolean = + process.env.SKIP_DB_TESTS === "1" || !process.env.TEST_DATABASE_URL; const _describeSkipIf = describe.skipIf(SKIP_DB_TESTS);