Revert test changes

apps/pigment-css-vite-app/src/pages/fixtures/index.test.js

@@ -104,7 +104,7 @@ async function main() {
       it(`creates screenshots of ${route}`, async function test() {
         // With the playwright inspector we might want to call `page.pause` which would lead to a timeout.
         if (process.env.PWDEBUG) {
-          this?.timeout?.(0);
+          this.timeout(0);
         }
 
         const testcase = await renderFixture(index);

packages-internal/scripts/tsconfig.typecheck.json

@@ -6,6 +6,6 @@
     "noEmit": true
   },
   "include": ["./**/*.ts"],
-  "exclude": ["./build", "./node_modules", "vitest.config.ts"],
+  "exclude": ["./build", "./node_modules"],
   "references": [{ "path": "../../packages-internal/docs-utils/tsconfig.build.json" }]
 }

packages-internal/scripts/typescript-to-proptypes/test/typescript-to-proptypes.test.ts

@@ -34,7 +34,7 @@ describe('typescript-to-proptypes', () => {
 
   before(function beforeHook() {
     // Creating a TS program might take a while.
-    this?.timeout?.(20000);
+    this.timeout(20000);
 
     const buildProject = createTypeScriptProjectBuilder({
       test: {

packages/api-docs-builder-core/.eslintrc.js

@@ -1,5 +1,6 @@
 module.exports = {
   rules: {
+    'import/no-default-export': 'error',
     'import/prefer-default-export': 'off',
   },
 };

packages/api-docs-builder-core/tsconfig.json

@@ -4,5 +4,5 @@
     "types": ["node", "mocha"]
   },
   "include": ["./**/*.ts", "./**/*.js"],
-  "exclude": ["node_modules", "vitest.config.ts"]
+  "exclude": ["node_modules"]
 }

packages/mui-base/src/Dropdown/Dropdown.test.tsx

@@ -30,6 +30,7 @@ describe('<Dropdown />', () => {
   it('registers a popup id correctly', async () => {
     function TestComponent() {
       const { registerPopup, popupId } = React.useContext(DropdownContext)!;
+      expect(context).not.to.equal(null);
 
       React.useEffect(() => {
         registerPopup('test-popup');
@@ -53,6 +54,7 @@ describe('<Dropdown />', () => {
 
     function TestComponent() {
       const { registerTrigger, triggerElement } = React.useContext(DropdownContext)!;
+      expect(context).not.to.equal(null);
 
       React.useEffect(() => {
         registerTrigger(trigger);

packages/mui-base/src/Transitions/CssTransition.test.tsx

@@ -30,7 +30,19 @@ describe('CssTransition', () => {
     onExitedSpy.resetHistory();
   });
 
-  const { render, clock } = createRenderer({ clock: 'fake' });
+  const { render, clock } = createRenderer();
+
+  describe('prop: className', () => {
+    it('applies it unconditionally', () => {
+      const { getByTestId } = render(
+        <TestTransitionContextProvider requestEnter>
+          <CssTransition className="foo" data-testid="root" />
+        </TestTransitionContextProvider>,
+      );
+      const root = getByTestId('root');
+      expect(root).to.have.class('foo');
+    });
+  });
 
   describe('prop: enterClassName, exitClassName', () => {
     clock.withFakeTimers();
@@ -53,7 +65,7 @@ describe('CssTransition', () => {
       clock.runToLast();
     });
 
-    it('applies exitClassName, then immediately enterClassName when requested to enter', async () => {
+    it('applies exitClassName, then immediately enterClassName when requested to enter', () => {
       const { getByTestId } = render(
         <TestTransitionContextProvider requestEnter>
           <CssTransition enterClassName="enter" exitClassName="exit" data-testid="root" />
@@ -67,21 +79,8 @@ describe('CssTransition', () => {
       expect(root).not.to.have.class('enter');
 
       clock.runToLast();
-
       expect(root).to.have.class('enter');
       expect(root).not.to.have.class('exit');
     });
   });
-
-  describe('prop: className', () => {
-    it('applies it unconditionally', () => {
-      const { getByTestId } = render(
-        <TestTransitionContextProvider requestEnter>
-          <CssTransition className="foo" data-testid="root" />
-        </TestTransitionContextProvider>,
-      );
-      const root = getByTestId('root');
-      expect(root).to.have.class('foo');
-    });
-  });
 });

packages/mui-base/src/useList/useList.ts

@@ -28,15 +28,13 @@ import { EventHandlers } from '../utils/types';
 const EMPTY_OBJECT = {};
 const NOOP = () => {};
 
-function defaultItemComparer<ItemValue>(optionA: ItemValue, optionB: ItemValue) {
-  return optionA === optionB;
-}
+const defaultItemComparer = <ItemValue>(optionA: ItemValue, optionB: ItemValue) =>
+  optionA === optionB;
 
 const defaultIsItemDisabled = () => false;
 
-function defaultItemStringifier<ItemValue>(item: ItemValue) {
-  return typeof item === 'string' ? item : String(item);
-}
+const defaultItemStringifier = <ItemValue>(item: ItemValue) =>
+  typeof item === 'string' ? item : String(item);
 
 const defaultGetInitialState = () => ({
   highlightedValue: null,

packages/mui-base/src/useSelect/defaultOptionStringifier.ts

@@ -1,6 +1,6 @@
 import { SelectOption } from '../useOption';
 
-function defaultOptionStringifier<OptionValue>(option: SelectOption<OptionValue>) {
+const defaultOptionStringifier = <OptionValue>(option: SelectOption<OptionValue>) => {
   const { label, value } = option;
   if (typeof label === 'string') {
     return label;
@@ -12,6 +12,6 @@ function defaultOptionStringifier<OptionValue>(option: SelectOption<OptionValue>
 
   // Fallback string representation
   return String(option);
-}
+};
 
 export { defaultOptionStringifier };

packages/mui-codemod/src/v4.0.0/top-level-imports.js

@@ -8,10 +8,7 @@ export default function transformer(fileInfo, api, options) {
   let requirePath = importModule;
 
   if (process.env.NODE_ENV === 'test') {
-    requirePath = requirePath.replace(
-      /^@material-ui\/core/,
-      process.env.VITEST ? '@mui/material' : '../../../mui-material/src',
-    );
+    requirePath = requirePath.replace(/^@material-ui\/core/, '../../../mui-material/src');
   }
 
   // eslint-disable-next-line global-require, import/no-dynamic-require

packages/mui-icons-material/builder.mjs

@@ -9,7 +9,7 @@ import { fileURLToPath } from 'url';
 import intersection from 'lodash/intersection.js';
 import { Queue } from '@mui/internal-waterfall';
 
-const currentDirectory = path.dirname(fileURLToPath(new URL(import.meta.url)));
+const currentDirectory = fileURLToPath(new URL('.', import.meta.url));
 
 export const RENAME_FILTER_DEFAULT = './renameFilters/default.mjs';
 export const RENAME_FILTER_MUI = './renameFilters/material-design-icons.mjs';

packages/mui-icons-material/builder.test.mjs

@@ -6,7 +6,7 @@ import { fileURLToPath } from 'url';
 import fse from 'fs-extra';
 import { RENAME_FILTER_MUI, RENAME_FILTER_DEFAULT, getComponentName, handler } from './builder.mjs';
 
-const currentDirectory = path.dirname(fileURLToPath(new URL(import.meta.url)));
+const currentDirectory = fileURLToPath(new URL('.', import.meta.url));
 
 const DISABLE_LOG = true;
 
@@ -43,28 +43,15 @@ describe('builder', () => {
       outputDir: null,
     };
 
-    // eslint-disable-next-line mocha/handle-done-callback
-    beforeEach(
-      process.env.VITEST
-        ? async function beforeEachHook(ctx) {
-            // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
-            options.outputDir = path.join(
-              os.tmpdir(),
-              'material-ui-icons-builder-test',
-              ctx.task.name,
-            );
-            await fse.emptyDir(options.outputDir);
-          }
-        : async function beforeEachHook() {
-            // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
-            options.outputDir = path.join(
-              os.tmpdir(),
-              'material-ui-icons-builder-test',
-              this.currentTest.fullTitle(),
-            );
-            await fse.emptyDir(options.outputDir);
-          },
-    );
+    beforeEach(async function beforeEachHook() {
+      // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
+      options.outputDir = path.join(
+        os.tmpdir(),
+        'material-ui-icons-builder-test',
+        this.currentTest.fullTitle(),
+      );
+      await fse.emptyDir(options.outputDir);
+    });
 
     it('script outputs to directory', async () => {
       await handler(options);
@@ -83,28 +70,15 @@ describe('builder', () => {
       outputDir: null,
     };
 
-    // eslint-disable-next-line mocha/handle-done-callback
-    beforeEach(
-      process.env.VITEST
-        ? async function beforeEachHook(ctx) {
-            // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
-            options.outputDir = path.join(
-              os.tmpdir(),
-              'material-ui-icons-builder-test',
-              ctx.task.name,
-            );
-            await fse.emptyDir(options.outputDir);
-          }
-        : async function beforeEachHook() {
-            // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
-            options.outputDir = path.join(
-              os.tmpdir(),
-              'material-ui-icons-builder-test',
-              this.currentTest.fullTitle(),
-            );
-            await fse.emptyDir(options.outputDir);
-          },
-    );
+    beforeEach(async function beforeEachHook() {
+      // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
+      options.outputDir = path.join(
+        os.tmpdir(),
+        'material-ui-icons-builder-test',
+        this.currentTest.fullTitle(),
+      );
+      await fse.emptyDir(options.outputDir);
+    });
 
     it('script outputs to directory', async () => {
       await handler(options);
@@ -137,28 +111,15 @@ describe('builder', () => {
       outputDir: null,
     };
 
-    // eslint-disable-next-line mocha/handle-done-callback
-    beforeEach(
-      process.env.VITEST
-        ? async function beforeEachHook(ctx) {
-            // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
-            options.outputDir = path.join(
-              os.tmpdir(),
-              'material-ui-icons-builder-test',
-              ctx.task.name,
-            );
-            await fse.emptyDir(options.outputDir);
-          }
-        : async function beforeEachHook() {
-            // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
-            options.outputDir = path.join(
-              os.tmpdir(),
-              'material-ui-icons-builder-test',
-              this.currentTest.fullTitle(),
-            );
-            await fse.emptyDir(options.outputDir);
-          },
-    );
+    beforeEach(async function beforeEachHook() {
+      // DON'T CLEAN UP TO MAKE TEST INSPECTABLE
+      options.outputDir = path.join(
+        os.tmpdir(),
+        'material-ui-icons-builder-test',
+        this.currentTest.fullTitle(),
+      );
+      await fse.emptyDir(options.outputDir);
+    });
 
     it('should produce the expected output', async () => {
       await handler(options);

packages/mui-joy/src/Autocomplete/Autocomplete.test.tsx

@@ -932,7 +932,7 @@ describe('Joy <Autocomplete />', () => {
     it('should ignore keydown event until the IME is confirmed', function test() {
       // TODO: Often times out in Firefox 78.
       // Is this slow because of testing-library or because of the implementation?
-      this?.timeout?.(4000);
+      this.timeout(4000);
 
       const { getByRole } = render(<Autocomplete open options={['가1', '가2']} autoFocus />);
       const textbox = getByRole('combobox');

packages/mui-joy/src/CssBaseline/CssBaseline.test.tsx

@@ -1,3 +1 @@
-describe('<CssBaseline />', () => {
-  it('To do', () => {});
-});
+describe('<CssBaseline />', () => {});

packages/mui-joy/src/Option/Option.test.tsx

@@ -1,3 +1 @@
-describe('<Option />', () => {
-  it('To do', () => {});
-});
+describe('<Option />', () => {});

packages/mui-joy/src/styles/CssVarsProvider.test.tsx

@@ -11,7 +11,7 @@ describe('[Joy] CssVarsProvider', () => {
   beforeEach(() => {
     originalMatchmedia = window.matchMedia;
     // Create mocks of localStorage getItem and setItem functions
-    Object.defineProperty(globalThis, 'localStorage', {
+    Object.defineProperty(global, 'localStorage', {
       value: {
         getItem: (key: string) => storage[key],
         setItem: (key: string, value: string) => {

packages/mui-joy/src/styles/extendTheme.test.js

@@ -205,7 +205,7 @@ describe('extendTheme', () => {
     beforeEach(() => {
       originalMatchmedia = window.matchMedia;
       // Create mocks of localStorage getItem and setItem functions
-      Object.defineProperty(globalThis, 'localStorage', {
+      Object.defineProperty(global, 'localStorage', {
         value: {
           getItem: (key) => storage[key],
           setItem: (key, value) => {