fix existing tests

Author: lobsterkatie

packages/nextjs/test/config.test.ts

@@ -7,12 +7,24 @@ import {
   SentryWebpackPluginOptions,
   WebpackConfigObject,
 } from '../src/config/types';
-import {
-  CLIENT_SDK_CONFIG_FILE,
-  constructWebpackConfigFunction,
-  SentryWebpackPlugin,
-  SERVER_SDK_CONFIG_FILE,
-} from '../src/config/webpack';
+import { constructWebpackConfigFunction, SentryWebpackPlugin } from '../src/config/webpack';
+
+const SERVER_SDK_CONFIG_FILE = 'sentry.server.config.js';
+const CLIENT_SDK_CONFIG_FILE = 'sentry.client.config.js';
+
+// We use `fs.existsSync()` in `getUserConfigFile()`. When we're not testing `getUserConfigFile()` specifically, all we
+// need is for it to give us any valid answer, so make it always find what it's looking for. Since this is a core node
+// built-in, though, which jest itself uses, otherwise let it do the normal thing. Storing the real version of the
+// function also lets us restore the original when we do want to test `getUserConfigFile()`.
+const realExistsSync = jest.requireActual('fs').existsSync;
+const mockExistsSync = (path: fs.PathLike) => {
+  if ((path as string).endsWith(SERVER_SDK_CONFIG_FILE) || (path as string).endsWith(CLIENT_SDK_CONFIG_FILE)) {
+    return true;
+  }
+
+  return realExistsSync(path);
+};
+const exitsSync = jest.spyOn(fs, 'existsSync').mockImplementation(mockExistsSync);
 
 /** Mocks of the arguments passed to `withSentryConfig` */
 const userNextConfig = {
@@ -63,8 +75,13 @@ const clientWebpackConfig = {
   target: 'web',
   context: '/Users/Maisey/projects/squirrelChasingSimulator',
 };
-const serverBuildContext = { isServer: true, dev: false, buildId: 'doGsaREgReaT' };
-const clientBuildContext = { isServer: false, dev: false, buildId: 'doGsaREgReaT' };
+const baseBuildContext = {
+  dev: false,
+  buildId: 'doGsaREgReaT',
+  dir: '/Users/Maisey/projects/squirrelChasingSimulator',
+};
+const serverBuildContext = { isServer: true, ...baseBuildContext };
+const clientBuildContext = { isServer: false, ...baseBuildContext };
 
 /**
  * Derive the final values of all next config options, by first applying `withSentryConfig` and then, if it returns a
@@ -223,6 +240,9 @@ describe('webpack config', () => {
   });
 
   describe('webpack `entry` property config', () => {
+    const serverConfigFilePath = `./${SERVER_SDK_CONFIG_FILE}`;
+    const clientConfigFilePath = `./${CLIENT_SDK_CONFIG_FILE}`;
+
     it('handles various entrypoint shapes', async () => {
       const finalWebpackConfig = await materializeFinalWebpackConfig({
         userNextConfig,
@@ -234,23 +254,23 @@ describe('webpack config', () => {
         expect.objectContaining({
           // original entry point value is a string
           // (was 'private-next-pages/api/dogs/[name].js')
-          'pages/api/dogs/[name]': [SERVER_SDK_CONFIG_FILE, 'private-next-pages/api/dogs/[name].js'],
+          'pages/api/dogs/[name]': [serverConfigFilePath, 'private-next-pages/api/dogs/[name].js'],
 
           // original entry point value is a string array
           // (was ['./node_modules/smellOVision/index.js', 'private-next-pages/_app.js'])
-          'pages/_app': [SERVER_SDK_CONFIG_FILE, './node_modules/smellOVision/index.js', 'private-next-pages/_app.js'],
+          'pages/_app': [serverConfigFilePath, './node_modules/smellOVision/index.js', 'private-next-pages/_app.js'],
 
           // original entry point value is an object containing a string `import` value
           // (`import` was 'private-next-pages/api/simulator/dogStats/[name].js')
           'pages/api/simulator/dogStats/[name]': {
-            import: [SERVER_SDK_CONFIG_FILE, 'private-next-pages/api/simulator/dogStats/[name].js'],
+            import: [serverConfigFilePath, 'private-next-pages/api/simulator/dogStats/[name].js'],
           },
 
           // original entry point value is an object containing a string array `import` value
           // (`import` was ['./node_modules/dogPoints/converter.js', 'private-next-pages/api/simulator/leaderboard.js'])
           'pages/api/simulator/leaderboard': {
             import: [
-              SERVER_SDK_CONFIG_FILE,
+              serverConfigFilePath,
               './node_modules/dogPoints/converter.js',
               'private-next-pages/api/simulator/leaderboard.js',
             ],
@@ -259,7 +279,7 @@ describe('webpack config', () => {
           // original entry point value is an object containg properties besides `import`
           // (`dependOn` remains untouched)
           'pages/api/tricks/[trickName]': {
-            import: [SERVER_SDK_CONFIG_FILE, 'private-next-pages/api/tricks/[trickName].js'],
+            import: [serverConfigFilePath, 'private-next-pages/api/tricks/[trickName].js'],
             dependOn: 'treats',
           },
         }),
@@ -278,7 +298,7 @@ describe('webpack config', () => {
           // no injected file
           main: './src/index.ts',
           // was 'next-client-pages-loader?page=%2F_app'
-          'pages/_app': [CLIENT_SDK_CONFIG_FILE, 'next-client-pages-loader?page=%2F_app'],
+          'pages/_app': [clientConfigFilePath, 'next-client-pages-loader?page=%2F_app'],
         }),
       );
     });