test: added

Author: alexander-akait

package.json

@@ -43,7 +43,6 @@
   ],
   "peerDependencies": {
     "@rspack/core": "0.x || 1.x",
-    "node-sass": "^4.0.0 || ^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0 || ^9.0.0",
     "sass": "^1.3.0",
     "sass-embedded": "*",
     "webpack": "^5.0.0"
@@ -96,7 +95,6 @@
     "lint-staged": "^15.2.0",
     "material-components-web": "^9.0.0",
     "memfs": "^4.7.7",
-    "node-sass": "^9.0.0",
     "node-sass-glob-importer": "^5.3.2",
     "npm-run-all": "^4.1.5",
     "prettier": "^3.2.2",

test/__snapshots__/implementation-option.test.js.no-node-sass.snap

@@ -48,6 +48,10 @@ exports[`implementation option not specify: errors 1`] = `[]`;
 
 exports[`implementation option not specify: warnings 1`] = `[]`;
 
+exports[`implementation option should dispose redundant compilers for \`modern-compiler\`: errors 1`] = `[]`;
+
+exports[`implementation option should dispose redundant compilers for \`modern-compiler\`: warnings 1`] = `[]`;
+
 exports[`implementation option should not swallow an error when trying to load a sass implementation: errors 1`] = `
 [
   "ModuleBuildError: Module build failed (from ../src/cjs.js):

test/__snapshots__/implementation-option.test.js.snap

@@ -52,6 +52,10 @@ exports[`implementation option not specify: errors 1`] = `[]`;
 
 exports[`implementation option not specify: warnings 1`] = `[]`;
 
+exports[`implementation option should dispose redundant compilers for \`modern-compiler\`: errors 1`] = `[]`;
+
+exports[`implementation option should dispose redundant compilers for \`modern-compiler\`: warnings 1`] = `[]`;
+
 exports[`implementation option should not swallow an error when trying to load a sass implementation: errors 1`] = `
 [
   "ModuleBuildError: Module build failed (from ../src/cjs.js):

test/implementation-option.test.js

@@ -398,6 +398,70 @@ describe("implementation option", () => {
     await close(compiler);
   });
 
+  it("should dispose redundant compilers for `modern-compiler`", async () => {
+    sassEmbeddedCompilerSpies.mockRestore();
+
+    let isInRace = false;
+
+    let firstDisposeSpy;
+    let secondDisposeSpy;
+
+    const actualFn = sassEmbedded.initAsyncCompiler.bind(sassEmbedded);
+
+    const initSpy = jest
+      .spyOn(sassEmbedded, "initAsyncCompiler")
+      .mockImplementation(async () => {
+        const compiler = await actualFn();
+
+        if (!isInRace) {
+          firstDisposeSpy = jest.spyOn(compiler, "dispose");
+          isInRace = true;
+
+          return new Promise((resolve) => {
+            const interval = setInterval(() => {
+              if (!isInRace) {
+                clearInterval(interval);
+                resolve(compiler);
+              }
+            });
+          });
+        }
+
+        isInRace = false;
+        secondDisposeSpy = jest.spyOn(compiler, "dispose");
+
+        return compiler;
+      });
+
+    const testId1 = getTestId("language", "scss");
+    const testId2 = getTestId("language", "sass");
+    const options = { api: "modern-compiler" };
+
+    // eslint-disable-next-line no-undefined
+    const compiler = getCompiler(undefined, {
+      entry: {
+        one: `./${testId1}`,
+        two: `./${testId2}`,
+      },
+      loader: { options },
+    });
+    const stats = await compile(compiler);
+
+    expect(getWarnings(stats)).toMatchSnapshot("warnings");
+    expect(getErrors(stats)).toMatchSnapshot("errors");
+    expect(initSpy).toHaveBeenCalledTimes(2);
+
+    await close(compiler);
+
+    initSpy.mockRestore();
+
+    expect(firstDisposeSpy).toHaveBeenCalledTimes(1);
+    firstDisposeSpy.mockRestore();
+
+    expect(secondDisposeSpy).toHaveBeenCalledTimes(1);
+    secondDisposeSpy.mockRestore();
+  });
+
   it("should try to load using valid order", async () => {
     jest.doMock("sass", () => {
       const error = new Error("Some error sass");