test: more (#1026)

Author: alexander-akait

src/utils.js

@@ -11,15 +11,15 @@ function getDefaultSassImplementation() {
 
   try {
     require.resolve("sass");
-  } catch (error) {
+  } catch (ignoreError) {
     try {
       require.resolve("node-sass");
       sassImplPkg = "node-sass";
-    } catch (error) {
+    } catch (_ignoreError) {
       try {
         require.resolve("sass-embedded");
         sassImplPkg = "sass-embedded";
-      } catch (ignoreError) {
+      } catch (__ignoreError) {
         sassImplPkg = "sass";
       }
     }

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

@@ -80,3 +80,12 @@ Cannot find module 'unresolved' from 'src/utils.js'",
 `;
 
 exports[`implementation option should throw error when unresolved package: warnings 1`] = `Array []`;
+
+exports[`implementation option should try to load using valid order: errors 1`] = `
+Array [
+  "ModuleError: Module Error (from ../src/cjs.js):
+Some error sass",
+]
+`;
+
+exports[`implementation option should try to load using valid order: warnings 1`] = `Array []`;

test/implementation-option.test.js

@@ -246,6 +246,44 @@ describe("implementation option", () => {
     expect(getErrors(stats)).toMatchSnapshot("errors");
   });
 
+  it("should try to load using valid order", async () => {
+    jest.doMock("sass", () => {
+      const error = new Error("Some error sass");
+
+      error.code = "MODULE_NOT_FOUND";
+      error.stack = null;
+
+      throw error;
+    });
+
+    jest.doMock("node-sass", () => {
+      const error = new Error("Some error node-sass");
+
+      error.code = "MODULE_NOT_FOUND";
+      error.stack = null;
+
+      throw error;
+    });
+
+    jest.doMock("sass-embedded", () => {
+      const error = new Error("Some error sass-embedded");
+
+      error.code = "MODULE_NOT_FOUND";
+      error.stack = null;
+
+      throw error;
+    });
+
+    const testId = getTestId("language", "scss");
+    const options = {};
+
+    const compiler = getCompiler(testId, { loader: { options } });
+    const stats = await compile(compiler);
+
+    expect(getWarnings(stats)).toMatchSnapshot("warnings");
+    expect(getErrors(stats)).toMatchSnapshot("errors");
+  });
+
   it("should not swallow an error when trying to load a sass implementation", async () => {
     jest.doMock("node-sass", () => {
       const error = new Error("Some error");