test: don't prepare e2e-only fixtures ahead of time

Author: pieh

tests/integration/turborepo.test.ts

@@ -25,6 +25,8 @@ beforeEach<FixtureTestContext>(async (ctx) => {
 // monorepo test uses process.chdir which is not working inside vite workers
 // so I'm disabling that test for now will revisit later in a follow up PR.
 // we have at least a e2e test that tests the monorepo functionality
+// NOTE: turborepo-npm fixture is currently skipped in tests/prepare.mjs
+// be sure to unskip it there if you would be working on making this integration test work
 test.skip<FixtureTestContext>('should create the files in the correct directories', async (ctx) => {
   await createFixture('turborepo-npm', ctx)
   await runPlugin(ctx, { PACKAGE_PATH: 'apps/web' })

tests/prepare.mjs

@@ -18,12 +18,31 @@ const NEXT_VERSION = process.env.NEXT_VERSION ?? 'latest'
 const fixturesDir = fileURLToPath(new URL(`./fixtures`, import.meta.url))
 const fixtureFilter = argv[2] ?? ''
 
+// E2E tests run next builds, so we don't need to prepare those ahead of time for integration tests
+const e2eOnlyFixtures = new Set([
+  'after',
+  'cli-before-regional-blobs-support',
+  'dist-dir',
+  // There is also a bug on Windows on Node.js 18.20.6, that cause build failures on this fixture
+  // see https://github.com/opennextjs/opennextjs-netlify/actions/runs/13268839161/job/37043172448?pr=2749#step:12:78
+  'middleware-og',
+  'middleware-single-matcher',
+  'nx-integrated',
+  'turborepo',
+  // We do have a skipped test using this fixture (due to integration test setup not working for monorepos)
+  'turborepo-npm',
+  'unstable-cache',
+])
+
 const limit = pLimit(Math.max(2, cpus().length / 2))
 const fixtures = readdirSync(fixturesDir)
   // Ignoring things like `.DS_Store`.
   .filter((fixture) => !fixture.startsWith('.'))
   // Applying the filter, if one is set.
   .filter((fixture) => !fixtureFilter || fixture.startsWith(fixtureFilter))
+  // Filter out fixtures that are only needed for E2E tests
+  .filter((fixture) => !e2eOnlyFixtures.has(fixture))
+
 console.log(`🧪 Preparing fixtures: ${fixtures.join(', ')}`)
 const fixtureList = new Set(fixtures)
 const fixtureCount = fixtures.length