Ensure next-server prepare only execute once

vercel · Aug 7, 2024 · ab9c14b · ab9c14b
1 parent 070159a
commit ab9c14b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/test/e2e/instrumentation-hook/register-once/instrumentation.js b/test/e2e/instrumentation-hook/register-once/instrumentation.js
@@ -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++
   }
 }
diff --git a/test/e2e/instrumentation-hook/register-once/register-once.test.ts b/test/e2e/instrumentation-hook/register-once/register-once.test.ts
@@ -14,4 +14,10 @@ describe('instrumentation-hook - register-once', () => {
     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')
+  })
 })