chore(#zimic): upgrade msw to v2.7.0

[diego-aquino]

Reopening #410.

Chore

• [#zimic] Upgrades msw to its latest version.

[diego-aquino]

The tests are still failing due to mswjs/msw#2297.

[diego-aquino]

Upgrading to 2.4.12 now results in many errors as:

zimic:test:turbo: ⎯⎯⎯⎯⎯⎯ Unhandled Errors ⎯⎯⎯⎯⎯⎯
zimic:test:turbo: 
zimic:test:turbo: Vitest caught 49 unhandled errors during the test run.
zimic:test:turbo: This might cause false positive tests. Resolve unhandled errors to make sure your tests are not affected.
zimic:test:turbo: 
zimic:test:turbo: ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Error ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
zimic:test:turbo: TypeError: Cannot read properties of undefined (reading 'url')
zimic:test:turbo:  ❯ ../../node_modules/.pnpm/[email protected][email protected]/node_modules/msw/lib/browser/index.mjs:1089:24
zimic:test:turbo:  ❯ ServiceWorkerContainer.<anonymous> ../../node_modules/.pnpm/[email protected][email protected]/node_modules/msw/lib/browser/index.mjs:2591:15

[diego-aquino]

Upgrading to [email protected] now results in many errors as:

zimic:test:turbo: ⎯⎯⎯⎯⎯⎯ Unhandled Errors ⎯⎯⎯⎯⎯⎯
zimic:test:turbo: 
zimic:test:turbo: Vitest caught 49 unhandled errors during the test run.
zimic:test:turbo: This might cause false positive tests. Resolve unhandled errors to make sure your tests are not affected.
zimic:test:turbo: 
zimic:test:turbo: ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Error ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
zimic:test:turbo: TypeError: Cannot read properties of undefined (reading 'url')
zimic:test:turbo:  ❯ ../../node_modules/.pnpm/[email protected][email protected]/node_modules/msw/lib/browser/index.mjs:1089:24
zimic:test:turbo:  ❯ ServiceWorkerContainer.<anonymous> ../../node_modules/.pnpm/[email protected][email protected]/node_modules/msw/lib/browser/index.mjs:2591:15

Those errors are likely related to mswjs/msw#2146.

[diego-aquino]

Apparently, [email protected] fixes all of the issues blocking this upgrade!

However, I'll keep this PR open for another week to make sure everything is fine. We'll re-run the tests after the dependency upgrade next Sunday and hopefully merge this.

[diego-aquino]

Actually, mswjs/msw#2146 does not appear to be fixed in [email protected]. CI just failed with the same error. 😕

Still happening in [email protected].

[diego-aquino]

Actually, mswjs/msw#2146 does not appear to be fixed in [email protected]. CI just failed with the same error. 😕

Still happening in [email protected].

[diego-aquino]

As mswjs/msw#2146 is still happening in the latest msw version and is causing #537, I added a manual postinstall patch to

https://github.com/mswjs/msw/blob/e3234fdce5a36f41c77719492e2eca0f77d7aaa6/src/browser/setupWorker/start/createResponseListener.ts#L31-L33

with the following diff:

-if (responseJson.type?.includes("opaque")) {
+if (!request || responseJson.type?.includes("opaque")) {
  return;
}

[github-actions]

Released in v0.12.4 🎉

• 📌 Release notes
• 📦 NPM package