Skip to content

Commit bd9da1f

Browse files
committed
test: don't prepare e2e-only fixtures ahead of time
1 parent e68d575 commit bd9da1f

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

tests/prepare.mjs

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,22 @@ const NEXT_VERSION = process.env.NEXT_VERSION ?? 'latest'
1818
const fixturesDir = fileURLToPath(new URL(`./fixtures`, import.meta.url))
1919
const fixtureFilter = argv[2] ?? ''
2020

21+
// E2E tests run next builds, so we don't need to prepare those ahead of time for integration tests
22+
const e2eOnlyFixtures = new Set([
23+
// There is also a bug on Windows on Node.js 18.20.6, that cause build failures on this fixture
24+
// see https://github.com/opennextjs/opennextjs-netlify/actions/runs/13268839161/job/37043172448?pr=2749#step:12:78
25+
'middleware-og',
26+
])
27+
2128
const limit = pLimit(Math.max(2, cpus().length / 2))
2229
const fixtures = readdirSync(fixturesDir)
2330
// Ignoring things like `.DS_Store`.
2431
.filter((fixture) => !fixture.startsWith('.'))
2532
// Applying the filter, if one is set.
2633
.filter((fixture) => !fixtureFilter || fixture.startsWith(fixtureFilter))
34+
// Filter out fixtures that are only needed for E2E tests
35+
.filter((fixture) => !e2eOnlyFixtures.has(fixture))
36+
2737
console.log(`🧪 Preparing fixtures: ${fixtures.join(', ')}`)
2838
const fixtureList = new Set(fixtures)
2939
const fixtureCount = fixtures.length

0 commit comments

Comments
 (0)