Fix flakiness by removing the per-test wiremock.resend.resetRequests() call in newsletter-api.spec.ts

Author: webstackdev

eslint.config.ts

@@ -97,6 +97,7 @@ export default [
             '^astro:.*',  // Ignore Astro virtual modules
             '^virtual:pwa-register$',
             '^\\./.*',    // Ignore relative imports (let TypeScript handle these)
+            'test/e2e/specs/14-system/package-release.spec.ts', // Ignore dynamic imports
           ],
         },
       ],
@@ -260,7 +261,7 @@ export default [
     ],
     ignores: [
       'test/e2e/helpers/pageObjectModels/**/*',
-      'src/test/e2e/specs/__tests__/**',
+      'test/e2e/specs/__tests__/**',
     ],
     rules: {
       'no-restricted-syntax': [

test/e2e/specs/08-api/newsletter-api.spec.ts

@@ -10,10 +10,6 @@ test.describe('Newsletter API integrations', () => {
 
   test.describe.configure({ mode: 'serial' })
 
-  test.beforeEach(async () => {
-    await wiremock.resend.resetRequests()
-  })
-
   test('@mocks sends double opt-in email through Resend mock', async ({ request }) => {
     const uniqueEmail = `newsletter-${Date.now()}@example.com`
     const response = await request.post(NEWSLETTER_ENDPOINT, {
@@ -58,12 +54,13 @@ test.describe('Newsletter API integrations', () => {
   })
 
   test('@mocks requires consent before sending any emails', async ({ request }) => {
+    const emailWithoutConsent = 'noconsent@example.com'
     const response = await request.post(NEWSLETTER_ENDPOINT, {
       headers: {
         'x-e2e-mocks': '1',
       },
       data: {
-        email: 'noconsent@example.com',
+        email: emailWithoutConsent,
         consentGiven: false,
       },
     })
@@ -72,6 +69,7 @@ test.describe('Newsletter API integrations', () => {
     const requests = await wiremock.resend.findRequests({
       method: 'POST',
       urlPath: RESEND_EMAIL_PATH,
+      bodyIncludes: emailWithoutConsent,
     })
     expect(requests.length).toBe(0)
   })