Skip to content

Commit

Permalink
Ensure next-server prepare only execute once
Browse files Browse the repository at this point in the history
  • Loading branch information
huozhi committed Aug 7, 2024
1 parent a5e2a02 commit f2e42c2
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 6 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
let count = 0

export function register() {
if (process.env.NEXT_RUNTIME === 'nodejs') {
if (count > 0) {
throw new Error('duplicated-register')
}
console.log('register-log')
count++
}
}
Original file line number Diff line number Diff line change
@@ -1,17 +1,18 @@
import { nextTestSetup } from 'e2e-utils'

describe('instrumentation-hook - register-once', () => {
const { next, skipped } = nextTestSetup({
const { next } = nextTestSetup({
files: __dirname,
skipDeployment: true,
})

if (skipped) {
return
}

it('should only register once', async () => {
await next.fetch('/foo')
expect(next.cliOutput).toIncludeRepeated('register-log', 1)
})

it('should not error when concurrent requests are made', async () => {
await Promise.all([next.fetch('/foo'), next.fetch('/foo')])
expect(next.cliOutput).toIncludeRepeated('register-log', 1)
expect(next.cliOutput).not.toInclude('duplicated-register')
})
})

0 comments on commit f2e42c2

Please sign in to comment.