refactor(experimental): delete Buffer in non-Node tests

solana-labs · Nov 17, 2023 · aa45e91 · aa45e91
1 parent fe489b3
commit aa45e91
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/packages/test-config/jest-unit.config.browser.ts b/packages/test-config/jest-unit.config.browser.ts
@@ -19,12 +19,13 @@ const config: Partial<Config.InitialProjectOptions> = {
     // this is required for @solana/compat, which uses the legacy web3js
     moduleNameMapper: {
         // Force module uuid to resolve with the CJS entry point, because Jest does not support package.json.exports. See https://github.com/uuidjs/uuid/issues/451
-        "uuid": require.resolve('uuid'),
+        uuid: require.resolve('uuid'),
     },
     setupFilesAfterEnv: [
         ...(commonConfig.setupFilesAfterEnv ?? []),
         path.resolve(__dirname, 'setup-secure-context.ts'),
         path.resolve(__dirname, 'setup-text-encoder.ts'),
+        path.resolve(__dirname, 'setup-web-buffer-global.ts'),
     ],
     testEnvironment: 'jsdom',
     testEnvironmentOptions: {},

diff --git a/packages/test-config/setup-web-buffer-global.ts b/packages/test-config/setup-web-buffer-global.ts
@@ -0,0 +1,8 @@
+/**
+ * Browsers don't have a `Buffer` global, so delete it now.
+ */
+beforeEach(() => {
+    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+    // @ts-ignore
+    delete globalThis.Buffer;
+});