happy-css-modules 3.2.0 → 5.0.0

package/src/transformer/index.test.ts

@@ -13,10 +13,10 @@ const locator = new Locator({ transformer: createDefaultTransformer({ cwd }) });
 test('processes .scss with scss transformer', async () => {
   createFixtures({
     '/test/1.scss': dedent`
-    .a {
-      // scss feature test (nesting)
-      .a_1 { dummy: ''; }
-    }
+      .a {
+        // scss feature test (nesting)
+        .a_1 { dummy: ''; }
+      }
     `,
   });
   const result = await locator.load(getFixturePath('/test/1.scss'));
@@ -26,10 +26,10 @@ test('processes .scss with scss transformer', async () => {
 test('processes .less with less transformer', async () => {
   createFixtures({
     '/test/1.less': dedent`
-    .a {
-      // less feature test (nesting)
-      .a_1 { dummy: ''; }
-    }
+      .a {
+        // less feature test (nesting)
+        .a_1 { dummy: ''; }
+      }
     `,
   });
   const result = await locator.load(getFixturePath('/test/1.less'));
@@ -41,8 +41,8 @@ test('processes .css with postcss transformer if postcssrc is found', async () =
   const locator1 = new Locator({ transformer: createDefaultTransformer() });
   createFixtures({
     '/test/1.css': dedent`
-    $prefix: foo;
-    .$(prefix)_bar {}
+      $prefix: foo;
+      .$(prefix)_bar {}
     `,
   });
   const result1 = await locator1.load(getFixturePath('/test/1.css'));
@@ -55,15 +55,15 @@ test('processes .css with postcss transformer if postcssrc is found', async () =
   });
   createFixtures({
     [`/${uuid}/postcss.config.js`]: dedent`
-    module.exports = {
-      plugins: [
-        require('${require.resolve('postcss-simple-vars')}'),
-      ],
-    };
+      module.exports = {
+        plugins: [
+          require('${require.resolve('postcss-simple-vars')}'),
+        ],
+      };
     `,
     '/test/1.css': dedent`
-    $prefix: foo;
-    .$(prefix)_bar {}
+      $prefix: foo;
+      .$(prefix)_bar {}
     `,
   });
   const result2 = await locator2.load(getFixturePath('/test/1.css'));

package/src/transformer/index.ts

@@ -33,7 +33,7 @@ export type TransformerOptions = {
 /** The function to transform source code. */
 export type Transformer = (source: string, options: TransformerOptions) => TransformResult | Promise<TransformResult>;
 
-// eslint-disable-next-line @typescript-eslint/no-unused-vars -- TODO: pass `e` to Error's cause option
+// oxlint-disable-next-line no-unused-vars -- TODO: pass `e` to Error's cause option
 export const handleImportError = (packageName: string) => (e: unknown) => {
   throw new Error(`${packageName} import failed. Did you forget to \`npm install -D ${packageName}\`?`);
 };

package/src/transformer/less-transformer.test.ts

@@ -1,11 +1,10 @@
-import { jest } from '@jest/globals';
 import dedent from 'dedent';
 import { Locator } from '../locator/index.js';
-import { createFixtures, getFixturePath } from '../test-util/util.js';
+import { createFixtures, getFixturePath, replaceFixtureDir } from '../test-util/util.js';
 import { createLessTransformer } from './less-transformer.js';
 
 const locator = new Locator({ transformer: createLessTransformer() });
-const loadSpy = jest.spyOn(locator, 'load');
+const loadSpy = vi.spyOn(locator, 'load');
 
 afterEach(() => {
   loadSpy.mockClear();
@@ -24,60 +23,92 @@ test('handles less features', async () => {
         .b_1();
         .b_2();
       }
-      `,
+    `,
     '/test/2.less': dedent`
       .b_1 { dummy: ''; }
       .b_2() { dummy: ''; }
-      `,
+    `,
     '/test/3.less': dedent`
       .c { dummy: ''; }
-      `,
+    `,
   });
   const result = await locator.load(getFixturePath('/test/1.less'));
 
   // FIXME: The end position of 'a_2_2' is incorrect.
-  expect(result).toMatchInlineSnapshot(`
+  expect(replaceFixtureDir(result)).toMatchInlineSnapshot(`
     {
-      dependencies: ["<fixtures>/test/2.less"],
-      tokens: [
+      "dependencies": [
+        "<fixtures>/test/2.less",
+      ],
+      "tokens": [
         {
-          name: "b_1",
-          originalLocation: {
-            filePath: "<fixtures>/test/2.less",
-            start: { line: 1, column: 1 },
-            end: { line: 1, column: 4 },
+          "name": "b_1",
+          "originalLocation": {
+            "end": {
+              "column": 4,
+              "line": 1,
+            },
+            "filePath": "<fixtures>/test/2.less",
+            "start": {
+              "column": 1,
+              "line": 1,
+            },
           },
         },
         {
-          name: "a_1",
-          originalLocation: {
-            filePath: "<fixtures>/test/1.less",
-            start: { line: 2, column: 1 },
-            end: { line: 2, column: 4 },
+          "name": "a_1",
+          "originalLocation": {
+            "end": {
+              "column": 4,
+              "line": 2,
+            },
+            "filePath": "<fixtures>/test/1.less",
+            "start": {
+              "column": 1,
+              "line": 2,
+            },
           },
         },
         {
-          name: "a_2",
-          originalLocation: {
-            filePath: "<fixtures>/test/1.less",
-            start: { line: 3, column: 1 },
-            end: { line: 3, column: 4 },
+          "name": "a_2",
+          "originalLocation": {
+            "end": {
+              "column": 4,
+              "line": 3,
+            },
+            "filePath": "<fixtures>/test/1.less",
+            "start": {
+              "column": 1,
+              "line": 3,
+            },
           },
         },
         {
-          name: "a_2_1",
-          originalLocation: {
-            filePath: "<fixtures>/test/1.less",
-            start: { line: 6, column: 3 },
-            end: { line: 6, column: 8 },
+          "name": "a_2_1",
+          "originalLocation": {
+            "end": {
+              "column": 8,
+              "line": 6,
+            },
+            "filePath": "<fixtures>/test/1.less",
+            "start": {
+              "column": 3,
+              "line": 6,
+            },
           },
         },
         {
-          name: "a_2_2",
-          originalLocation: {
-            filePath: "<fixtures>/test/1.less",
-            start: { line: 7, column: 3 },
-            end: { line: 7, column: 8 },
+          "name": "a_2_2",
+          "originalLocation": {
+            "end": {
+              "column": 8,
+              "line": 7,
+            },
+            "filePath": "<fixtures>/test/1.less",
+            "start": {
+              "column": 3,
+              "line": 7,
+            },
           },
         },
       ],
@@ -88,13 +119,13 @@ test('handles less features', async () => {
 test('tracks dependencies that have been pre-bundled by less compiler', async () => {
   createFixtures({
     '/test/1.less': dedent`
-    @import './2.less';
-    @import './3.less';
+      @import './2.less';
+      @import './3.less';
     `,
     '/test/2.less': dedent`
     `,
     '/test/3.less': dedent`
-    @import './4.less';
+      @import './4.less';
     `,
     '/test/4.less': dedent`
     `,
@@ -103,13 +134,11 @@ test('tracks dependencies that have been pre-bundled by less compiler', async ()
 
   // The files imported using @import are pre-bundled by the compiler.
   // Therefore, `Locator#load` is not called to process other files.
-  expect(loadSpy).toBeCalledTimes(1);
+  expect(loadSpy).toHaveBeenCalledTimes(1);
   expect(loadSpy).toHaveBeenNthCalledWith(1, getFixturePath('/test/1.less'));
 
   // The files pre-bundled by the compiler are also included in `result.dependencies`
-  // eslint-disable-next-line @typescript-eslint/require-array-sort-compare
   expect(result.dependencies.sort()).toStrictEqual(
-    // eslint-disable-next-line @typescript-eslint/require-array-sort-compare
     ['/test/2.less', '/test/3.less', '/test/4.less'].map(getFixturePath).sort(),
   );
 });
@@ -117,16 +146,14 @@ test('tracks dependencies that have been pre-bundled by less compiler', async ()
 test('resolves specifier using resolver', async () => {
   createFixtures({
     '/test/1.less': dedent`
-    @import 'package-1';
-    @import 'package-2';
+      @import 'package-1';
+      @import 'package-2';
     `,
     '/node_modules/package-1/index.css': `.a {}`,
     '/node_modules/package-2/index.less': `.a {}`,
   });
   const result = await locator.load(getFixturePath('/test/1.less'));
-  // eslint-disable-next-line @typescript-eslint/require-array-sort-compare
   expect(result.dependencies.sort()).toStrictEqual(
-    // eslint-disable-next-line @typescript-eslint/require-array-sort-compare
     [getFixturePath('/node_modules/package-1/index.css'), getFixturePath('/node_modules/package-2/index.less')].sort(),
   );
 });
@@ -134,9 +161,9 @@ test('resolves specifier using resolver', async () => {
 test('ignores http(s) protocol file', async () => {
   createFixtures({
     '/test/1.less': dedent`
-    @import 'http://example.com/path/http.css';
-    @import 'https://example.com/path/https.css';
-    @import 'https://example.com/path/less.less';
+      @import 'http://example.com/path/http.css';
+      @import 'https://example.com/path/https.css';
+      @import 'https://example.com/path/less.less';
     `,
   });
   const result = await locator.load(getFixturePath('/test/1.less'));

package/src/transformer/less-transformer.ts

@@ -2,7 +2,6 @@ import type { Transformer } from '../index.js';
 import type { TransformerOptions } from './index.js';
 import { handleImportError } from './index.js';
 
-// eslint-disable-next-line @typescript-eslint/naming-convention
 function createLessPluginResolver(Less: typeof import('less'), options: TransformerOptions): Less.Plugin {
   class ResolverFileManager extends Less.FileManager {
     options: TransformerOptions;
@@ -32,7 +31,7 @@ function createLessPluginResolver(Less: typeof import('less'), options: Transfor
     constructor(options: TransformerOptions) {
       this.options = options;
     }
-    public install(less: LessStatic, pluginManager: Less.PluginManager): void {
+    public install(_less: LessStatic, pluginManager: Less.PluginManager): void {
       pluginManager.addFileManager(new ResolverFileManager(this.options));
     }
     public minVersion: [number, number, number] = [2, 1, 1];
@@ -44,7 +43,6 @@ function createLessPluginResolver(Less: typeof import('less'), options: Transfor
 export const createLessTransformer: () => Transformer = () => {
   let less: typeof import('less');
   return async (source, options) => {
-    // eslint-disable-next-line require-atomic-updates
     less ??= (await import('less').catch(handleImportError('less'))).default;
     const result = await less.render(source, {
       filename: options.from,

package/src/transformer/postcss-transformer.test.ts

@@ -1,9 +1,8 @@
 import { randomUUID } from 'node:crypto';
 import { createRequire } from 'node:module';
-import { jest } from '@jest/globals';
 import dedent from 'dedent';
 import { Locator } from '../locator/index.js';
-import { createFixtures, getFixturePath } from '../test-util/util.js';
+import { createFixtures, getFixturePath, replaceFixtureDir } from '../test-util/util.js';
 import { createPostcssTransformer } from './postcss-transformer.js';
 
 const cwd = getFixturePath('/');
@@ -22,29 +21,35 @@ test('handles postcss features', async () => {
   });
   createFixtures({
     [`/${uuid}/postcss.config.js`]: dedent`
-    module.exports = {
-      plugins: [
-        require('${require.resolve('postcss-simple-vars')}'),
-      ],
-    };
+      module.exports = {
+        plugins: [
+          require('${require.resolve('postcss-simple-vars')}'),
+        ],
+      };
     `,
     '/test/1.css': dedent`
-    $prefix: foo;
-    .$(prefix)_bar {}
+      $prefix: foo;
+      .$(prefix)_bar {}
     `,
   });
   const result = await locator.load(getFixturePath('/test/1.css'));
 
-  expect(result).toMatchInlineSnapshot(`
+  expect(replaceFixtureDir(result)).toMatchInlineSnapshot(`
     {
-      dependencies: [],
-      tokens: [
+      "dependencies": [],
+      "tokens": [
         {
-          name: "foo_bar",
-          originalLocation: {
-            filePath: "<fixtures>/test/1.css",
-            start: { line: 2, column: 1 },
-            end: { line: 2, column: 8 },
+          "name": "foo_bar",
+          "originalLocation": {
+            "end": {
+              "column": 8,
+              "line": 2,
+            },
+            "filePath": "<fixtures>/test/1.css",
+            "start": {
+              "column": 1,
+              "line": 2,
+            },
           },
         },
       ],
@@ -60,18 +65,18 @@ test('tracks dependencies that have been pre-bundled by postcss compiler', async
       postcssConfig: `${uuid}/postcss.config.js`,
     }),
   });
-  const loadSpy = jest.spyOn(locator, 'load');
+  const loadSpy = vi.spyOn(locator, 'load');
   createFixtures({
     [`/${uuid}/postcss.config.js`]: dedent`
-    module.exports = {
-      plugins: [
-        require('${require.resolve('postcss-import')}'),
-      ],
-    };
+      module.exports = {
+        plugins: [
+          require('${require.resolve('postcss-import')}'),
+        ],
+      };
     `,
     '/test/1.css': dedent`
-    @import './2.css';
-    @import './3.css';
+      @import './2.css';
+      @import './3.css';
     `,
     '/test/2.css': ``,
     '/test/3.css': `@import './4.css'`,
@@ -81,7 +86,7 @@ test('tracks dependencies that have been pre-bundled by postcss compiler', async
 
   // The files imported using @import are pre-bundled by the compiler.
   // Therefore, `Locator#load` is not called to process other files.
-  expect(loadSpy).toBeCalledTimes(1);
+  expect(loadSpy).toHaveBeenCalledTimes(1);
   expect(loadSpy).toHaveBeenNthCalledWith(1, getFixturePath('/test/1.css'));
 
   // The files pre-bundled by the compiler are also included in `result.dependencies`
@@ -98,16 +103,16 @@ test('resolves specifier using resolver', async () => {
   });
   createFixtures({
     [`/${uuid}/postcss.config.js`]: dedent`
-    module.exports = {
-      plugins: [
-        // When using postcss-import, the resolver of happy-css-modules is ignored.
-        // Therefore, we test here without postcss-import.
-        // require('${require.resolve('postcss-import')}'),
-      ],
-    };
+      module.exports = {
+        plugins: [
+          // When using postcss-import, the resolver of happy-css-modules is ignored.
+          // Therefore, we test here without postcss-import.
+          // require('${require.resolve('postcss-import')}'),
+        ],
+      };
     `,
     '/test/1.css': dedent`
-    @import 'package';
+      @import 'package';
     `,
     '/node_modules/package/index.css': `.a {}`,
   });
@@ -125,17 +130,17 @@ test('ignores http(s) protocol file', async () => {
   });
   createFixtures({
     [`/${uuid}/postcss.config.js`]: dedent`
-    module.exports = {
-      plugins: [
-        // When using postcss-import, the resolver of happy-css-modules is ignored.
-        // Therefore, we test here without postcss-import.
-        // require('${require.resolve('postcss-import')}'),
-      ],
-    };
+      module.exports = {
+        plugins: [
+          // When using postcss-import, the resolver of happy-css-modules is ignored.
+          // Therefore, we test here without postcss-import.
+          // require('${require.resolve('postcss-import')}'),
+        ],
+      };
     `,
     '/test/1.css': dedent`
-    @import 'http://example.com/path/http.css';
-    @import 'https://example.com/path/https.css';
+      @import 'http://example.com/path/http.css';
+      @import 'https://example.com/path/https.css';
     `,
   });
   const result = await locator.load(getFixturePath('/test/1.css'));
@@ -150,16 +155,15 @@ test('returns false if postcssrc is not found', async () => {
   });
   createFixtures({
     '/test/1.css': dedent`
-    @import 'http://example.com/path/http.css';
-    @import 'https://example.com/path/https.css';
+      @import 'http://example.com/path/http.css';
+      @import 'https://example.com/path/https.css';
     `,
   });
   expect(
     await transformer('', {
       from: getFixturePath('/test/1.css'),
       isIgnoredSpecifier: () => false,
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      resolver: jest.fn() as any,
+      resolver: vi.fn(),
     }),
   ).toBe(false);
 });
@@ -174,15 +178,15 @@ test('searches config from cwd if postcssConfig option is missing', async () =>
   });
   createFixtures({
     [`/${uuid}/postcss.config.js`]: dedent`
-    module.exports = {
-      plugins: [
-        require('${require.resolve('postcss-simple-vars')}'),
-      ],
-    };
+      module.exports = {
+        plugins: [
+          require('${require.resolve('postcss-simple-vars')}'),
+        ],
+      };
     `,
     '/test/1.css': dedent`
-    $prefix: foo;
-    .$(prefix)_bar {}
+      $prefix: foo;
+      .$(prefix)_bar {}
     `,
   });
   const result = await locator.load(getFixturePath('/test/1.css'));