Fix config.test.ts

Author: iker-barriocanal

packages/nextjs/test/config.test.ts

@@ -30,15 +30,19 @@ const CLIENT_SDK_CONFIG_FILE = 'sentry.client.config.js';
 // 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) => {
+const realFs = {
+  existsSync: jest.requireActual('fs').existsSync,
+  mkdtempSync: jest.requireActual('fs').mkdtempSync,
+  writeFileSync: jest.requireActual('fs').writeFileSync,
+};
+jest.mock('fs');
+const existsSyncMockImplementation = (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);
+  return realFs.existsSync(path);
 };
-const exitsSync = jest.spyOn(fs, 'existsSync').mockImplementation(mockExistsSync);
+jest.spyOn(fs, 'existsSync').mockImplementation(existsSyncMockImplementation);
 
 /** Mocks of the arguments passed to `withSentryConfig` */
 const userNextConfig: Partial<NextConfigObject> = {
@@ -328,7 +332,7 @@ describe('webpack config', () => {
     });
   });
 
-  describe.only('`distDir` value in default server-side `RewriteFrames` integration', () => {
+  describe('`distDir` value in default server-side `RewriteFrames` integration', () => {
     describe('`DefinePlugin` existence and inclusion of `__rewriteFramesDistDir__` definition', () => {
       it.each([
         ['no custom `distDir`, pre-existing `DefinePlugin`', undefined, true],
@@ -353,10 +357,8 @@ describe('webpack config', () => {
             incomingWebpackBuildContext: getBuildContext('server', userNextConfigMaybeWithDistDir),
           });
 
-          expect(finalWebpackConfig.plugins).toEqual(expect.arrayContaining([expect.any(DefinePlugin)]));
-
           const definePluginInstance = findWebpackPlugin(finalWebpackConfig, 'DefinePlugin') as DefinePlugin;
-          expect(definePluginInstance.definitions.__rewriteFramesDistDir__).toEqual(customDistDir);
+          expect(definePluginInstance.definitions.__rewriteFramesDistDir__).toStrictEqual(customDistDir);
         },
       );
     });
@@ -622,37 +624,33 @@ describe('Sentry webpack plugin config', () => {
   describe('getUserConfigFile', () => {
     let tempDir: string;
 
-    beforeAll(() => {
-      exitsSync.mockImplementation(realExistsSync);
-    });
+    beforeAll(() => jest.spyOn(fs, 'existsSync').mockImplementation(realFs.existsSync));
 
     beforeEach(() => {
       const tempDirPathPrefix = path.join(os.tmpdir(), 'sentry-nextjs-test-');
-      tempDir = fs.mkdtempSync(tempDirPathPrefix);
+      tempDir = realFs.mkdtempSync(tempDirPathPrefix);
     });
 
     afterEach(() => {
       rimraf.sync(tempDir);
     });
 
-    afterAll(() => {
-      exitsSync.mockImplementation(mockExistsSync);
-    });
+    afterAll(() => jest.spyOn(fs, 'existsSync').mockImplementation(existsSyncMockImplementation));
 
     it('successfully finds js files', () => {
-      fs.writeFileSync(path.resolve(tempDir, 'sentry.server.config.js'), 'Dogs are great!');
-      fs.writeFileSync(path.resolve(tempDir, 'sentry.client.config.js'), 'Squirrel!');
+      realFs.writeFileSync(path.resolve(tempDir, 'sentry.server.config.js'), 'Dogs are great!');
+      realFs.writeFileSync(path.resolve(tempDir, 'sentry.client.config.js'), 'Squirrel!');
 
-      expect(getUserConfigFile(tempDir, 'server')).toEqual('sentry.server.config.js');
-      expect(getUserConfigFile(tempDir, 'client')).toEqual('sentry.client.config.js');
+      expect(getUserConfigFile(tempDir, 'server')).toStrictEqual('sentry.server.config.js');
+      expect(getUserConfigFile(tempDir, 'client')).toStrictEqual('sentry.client.config.js');
     });
 
     it('successfully finds ts files', () => {
-      fs.writeFileSync(path.resolve(tempDir, 'sentry.server.config.ts'), 'Sit. Stay. Lie Down.');
-      fs.writeFileSync(path.resolve(tempDir, 'sentry.client.config.ts'), 'Good dog!');
+      realFs.writeFileSync(path.resolve(tempDir, 'sentry.server.config.ts'), 'Sit. Stay. Lie Down.');
+      realFs.writeFileSync(path.resolve(tempDir, 'sentry.client.config.ts'), 'Good dog!');
 
-      expect(getUserConfigFile(tempDir, 'server')).toEqual('sentry.server.config.ts');
-      expect(getUserConfigFile(tempDir, 'client')).toEqual('sentry.client.config.ts');
+      expect(getUserConfigFile(tempDir, 'server')).toStrictEqual('sentry.server.config.ts');
+      expect(getUserConfigFile(tempDir, 'client')).toStrictEqual('sentry.client.config.ts');
     });
 
     it('errors when files are missing', () => {