test: extract cjs require tests to .spec.cts

Author: Shinigami92

eslint.config.ts

@@ -244,7 +244,7 @@ const config: ReturnType<typeof tseslint.config> = tseslint.config(
   },
   {
     name: 'test/**/*.ts overrides',
-    files: ['test/**/*.spec.ts', 'test/**/*.spec.d.ts'],
+    files: ['test/**/*.spec.ts', 'test/**/*.spec.cts', 'test/**/*.spec.d.ts'],
     plugins: {
       vitest: eslintPluginVitest,
     },

test/locale-imports.spec.ts

@@ -4,19 +4,6 @@ import { allLocales } from '../src';
 import { keys } from '../src/internal/keys';
 
 describe.each(keys(allLocales))('locale imports', (locale) => {
-  it(`should be possible to directly require('@faker-js/faker/locale/${locale}')`, () => {
-    // eslint-disable-next-line @typescript-eslint/no-require-imports, unicorn/prefer-module
-    const { faker } = require(`../dist/locale/${locale}.cjs`) as {
-      faker: Faker;
-    };
-
-    expect(faker).toBeDefined();
-    expect(faker.string.alpha()).toBeTypeOf('string');
-    expect(faker.definitions.metadata.title).toBe(
-      allLocales[locale].metadata?.title
-    );
-  });
-
   it(`should be possible to directly import('@faker-js/faker/locale/${locale}')`, async () => {
     const { faker } = (await import(`../dist/locale/${locale}.js`)) as {
       faker: Faker;

test/require.spec.cts

@@ -0,0 +1,49 @@
+/* eslint-disable @typescript-eslint/no-require-imports, unicorn/prefer-module */
+const { describe, expect, it, vi } = await import('vitest');
+const { allLocales, SimpleFaker } = require('../dist/index.cjs');
+
+describe('require (cjs)', () => {
+  describe.each(
+    Object.keys(
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
+      allLocales
+    )
+  )('locale imports', (locale) => {
+    it(`should be possible to directly require('@faker-js/faker/locale/${locale}')`, () => {
+      const { faker } = require(`../dist/locale/${locale}.cjs`);
+
+      expect(faker).toBeDefined();
+      expect(faker.string.alpha()).toBeTypeOf('string');
+      expect(faker.definitions.metadata.title).toBe(
+        allLocales[locale].metadata?.title
+      );
+    });
+  });
+
+  describe('simpleFaker', () => {
+    it('should not log anything on startup', () => {
+      const spies = Object.keys(console)
+        .filter(
+          (key) =>
+            // @ts-expect-error: cts cant use `as keyof typeof console`
+            typeof console[key] === 'function'
+        )
+        .map((methodName) =>
+          vi.spyOn(
+            console,
+            // @ts-expect-error: cts cant use `as keyof typeof console`
+            methodName
+          )
+        );
+
+      expect(require('..').simpleFaker).toBeDefined();
+
+      expect(new SimpleFaker()).toBeDefined();
+
+      for (const spy of spies) {
+        expect(spy).not.toHaveBeenCalled();
+        spy.mockRestore();
+      }
+    });
+  });
+});

test/simple-faker.spec.ts

@@ -4,13 +4,14 @@ import { generateMersenne32Randomizer, SimpleFaker, simpleFaker } from '../src';
 import { keys } from '../src/internal/keys';
 
 describe('simpleFaker', () => {
-  it('should not log anything on startup', () => {
+  it('should not log anything on startup', async () => {
     const spies: MockInstance[] = keys(console)
       .filter((key) => typeof console[key] === 'function')
       .map((methodName) => vi.spyOn(console, methodName));
 
-    // eslint-disable-next-line @typescript-eslint/no-require-imports, unicorn/prefer-module -- Using import() requires types being build but the CI / TS-Check runs without them.
-    expect(require('..').simpleFaker).toBeDefined();
+    // Using import() requires types being build but the CI / TS-Check runs without them.
+    const { simpleFaker: importedSimpleFaker } = await import('..');
+    expect(importedSimpleFaker).toBeDefined();
 
     expect(new SimpleFaker()).toBeDefined();
 

vitest.config.ts

@@ -9,7 +9,7 @@ console.log('VITEST_SEQUENCE_SEED', VITEST_SEQUENCE_SEED);
 export default defineConfig({
   test: {
     setupFiles: ['test/setup.ts'],
-    include: ['test/**/*.spec.ts'],
+    include: ['test/**/*.spec.ts', 'test/**/*.spec.cts'],
     exclude: ['test/integration/**/*.spec.ts'],
     coverage: {
       all: true,