From 71b9e8020bc858f1764cf1cd13543b531459374e Mon Sep 17 00:00:00 2001 From: Miguel Angel Simon Sierra Date: Fri, 18 Sep 2026 18:01:55 -0400 Subject: [PATCH] test(cli): lint registry components one test per item instead of one long loop --- .../src/registry/registryComponents.test.ts | 33 ++++++++----------- 1 file changed, 13 insertions(+), 20 deletions(-) diff --git a/packages/cli/src/registry/registryComponents.test.ts b/packages/cli/src/registry/registryComponents.test.ts index 37537c759b..db58837a52 100644 --- a/packages/cli/src/registry/registryComponents.test.ts +++ b/packages/cli/src/registry/registryComponents.test.ts @@ -44,26 +44,19 @@ async function invalidDemoMedia(entryName: string): Promise { .map((finding) => `${entryName}/demo.html: ${finding.code}`); } -describe("registry components", () => { - it("ships installable snippets without invalid nested media", async () => { - const invalidMedia: string[] = []; - - for (const entry of readdirSync(componentsDir, { withFileTypes: true })) { - if (!entry.isDirectory()) continue; - invalidMedia.push(...(await invalidInstallableMedia(entry.name))); - } - - expect(invalidMedia).toEqual([]); - }); +const entryNames = readdirSync(componentsDir, { withFileTypes: true }) + .filter((entry) => entry.isDirectory()) + .map((entry) => entry.name); - it("ships demos without source-less media", async () => { - const invalidMedia: string[] = []; - - for (const entry of readdirSync(componentsDir, { withFileTypes: true })) { - if (!entry.isDirectory()) continue; - invalidMedia.push(...(await invalidDemoMedia(entry.name))); - } - - expect(invalidMedia).toEqual([]); +describe("registry components", () => { + it.each(entryNames)( + "%s ships installable snippets without invalid nested media", + async (name) => { + expect(await invalidInstallableMedia(name)).toEqual([]); + }, + ); + + it.each(entryNames)("%s ships a demo without source-less media", async (name) => { + expect(await invalidDemoMedia(name)).toEqual([]); }); });