diff --git a/.changeset/fix-empty-each-tests.md b/.changeset/fix-empty-each-tests.md deleted file mode 100644 index 71cc06c30d..0000000000 --- a/.changeset/fix-empty-each-tests.md +++ /dev/null @@ -1,5 +0,0 @@ ---- -'@backstage/backend-test-utils': patch ---- - -Fixed a crash in test suites that use `TestDatabases` or `TestCaches` when none of the requested database or cache engines are available. Previously, `eachSupportedId()` would return an empty array causing `describe.each` to throw and prevent the entire suite from running. Now returns a placeholder entry so individual tests report a clear failure instead. diff --git a/packages/backend-test-utils/src/cache/TestCaches.ts b/packages/backend-test-utils/src/cache/TestCaches.ts index 27c07daa32..d4553697bf 100644 --- a/packages/backend-test-utils/src/cache/TestCaches.ts +++ b/packages/backend-test-utils/src/cache/TestCaches.ts @@ -108,9 +108,6 @@ export class TestCaches { } eachSupportedId(): [TestCacheId][] { - if (this.supportedIds.length === 0) { - return [['MISSING' as TestCacheId]]; - } return this.supportedIds.map(id => [id]); } diff --git a/packages/backend-test-utils/src/database/TestDatabases.ts b/packages/backend-test-utils/src/database/TestDatabases.ts index 46d966b6ee..8baac04193 100644 --- a/packages/backend-test-utils/src/database/TestDatabases.ts +++ b/packages/backend-test-utils/src/database/TestDatabases.ts @@ -124,13 +124,6 @@ export class TestDatabases { } eachSupportedId(): [TestDatabaseId][] { - if (this.supportedIds.length === 0) { - // Return a placeholder so that describe.each/it.each does not throw - // when no databases are available. The init() call will throw for - // the unknown ID, causing the test to be reported as failed rather - // than crashing the entire suite. - return [['MISSING' as TestDatabaseId]]; - } return this.supportedIds.map(id => [id]); }