fix type errors in gatsby tests

Author: lobsterkatie

packages/gatsby/test/gatsby-browser.test.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/no-var-requires */
 /* eslint-disable @typescript-eslint/no-explicit-any */
 
-const { onClientEntry } = require('../gatsby-browser');
+import { onClientEntry } from '../gatsby-browser';
 
 (global as any).__SENTRY_RELEASE__ = '683f3a6ab819d47d23abfca9a914c81f0524d35b';
 (global as any).__SENTRY_DSN__ = 'https://[email protected]/0';
@@ -153,7 +153,7 @@ describe('onClientEntry', () => {
 
   // Run this last to check for any test side effects
   it('does not run if plugin params are undefined', () => {
-    onClientEntry();
+    onClientEntry(undefined, undefined);
     expect(sentryInit).toHaveBeenCalledTimes(0);
     expect(tracingAddExtensionMethods).toHaveBeenCalledTimes(0);
   });

packages/gatsby/test/gatsby-node.test.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/no-var-requires */
 /* eslint-disable @typescript-eslint/no-explicit-any */
-const { onCreateWebpackConfig } = require('../gatsby-node');
+import { onCreateWebpackConfig } from '../gatsby-node';
 
 describe('onCreateWebpackConfig', () => {
   it('sets a webpack config', () => {
@@ -12,7 +12,9 @@ describe('onCreateWebpackConfig', () => {
       setWebpackConfig: jest.fn(),
     };
 
-    onCreateWebpackConfig({ plugins, actions });
+    const getConfig = jest.fn();
+
+    onCreateWebpackConfig({ plugins, actions, getConfig });
 
     expect(plugins.define).toHaveBeenCalledTimes(1);
     expect(plugins.define).toHaveBeenLastCalledWith({

packages/gatsby/test/setEnvVars.ts

@@ -1,2 +1,6 @@
 // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
 process.env.SENTRY_RELEASE = '14abbb1678a2eb59d1a171ea33d630dd6c6eee70';
+
+// This file needs to have an import or an export to count as a module, which is necessary when using
+// the `isolatedModules` tsconfig option.
+export const _ = '';