fix: handling error better (#515)

Author: alexander-akait

src/index.js

@@ -12,7 +12,15 @@ import LessError from "./LessError";
 async function lessLoader(source) {
  const options = this.getOptions(schema);
  const callback = this.async();
- const implementation = getLessImplementation(this, options.implementation);
+ let implementation;
+
+ try {
+ implementation = getLessImplementation(this, options.implementation);
+ } catch (error) {
+ callback(error);
+
+ return;
+ }
 
  if (!implementation) {
  callback(

src/utils.js

@@ -232,15 +232,8 @@ function getLessImplementation(loaderContext, implementation) {
  if (!implementation || typeof implementation === "string") {
  const lessImplPkg = implementation || "less";
 
- try {
- // eslint-disable-next-line import/no-dynamic-require, global-require
- resolvedImplementation = require(lessImplPkg);
- } catch (error) {
- loaderContext.emitError(error);
-
- // eslint-disable-next-line consistent-return
- return;
- }
+ // eslint-disable-next-line import/no-dynamic-require, global-require
+ resolvedImplementation = require(lessImplPkg);
  }
 
  // eslint-disable-next-line consistent-return

test/__snapshots__/implementation.test.js.snap

@@ -3,14 +3,21 @@
 exports[`"implementation" option should throw error when unresolved package: errors 1`] = `
 [
  "ModuleBuildError: Module build failed (from \`replaced original path\`):
-Error: The Less implementation "unresolved" not found",
- "ModuleError: Module Error (from \`replaced original path\`):
-(Emitted value instead of an instance of Error) Error: Cannot find module 'unresolved' from 'src/utils.js'",
+NonErrorEmittedError: (Emitted value instead of an instance of Error) Error: Cannot find module 'unresolved' from 'src/utils.js'",
+]
+`;
+
+exports[`"implementation" option should throw error when unresolved package: errors 2`] = `
+[
+ "ModuleBuildError: Module build failed (from \`replaced original path\`):
+Error: The Less implementation "/test/fixtures/implementation-error.js" not found",
 ]
 `;
 
 exports[`"implementation" option should throw error when unresolved package: warnings 1`] = `[]`;
 
+exports[`"implementation" option should throw error when unresolved package: warnings 2`] = `[]`;
+
 exports[`"implementation" option should work when implementation option is string: css 1`] = `
 ".box {
  color: #fe33ac;

test/fixtures/implementation-error.js

@@ -0,0 +1 @@
+module.exports = false;

test/implementation.test.js

@@ -49,4 +49,15 @@ describe('"implementation" option', () => {
  expect(getWarnings(stats)).toMatchSnapshot("warnings");
  expect(getErrors(stats)).toMatchSnapshot("errors");
  });
+
+ it("should throw error when unresolved package", async () => {
+ const testId = "./basic.less";
+ const compiler = getCompiler(testId, {
+ implementation: require.resolve("./fixtures/implementation-error.js"),
+ });
+ const stats = await compile(compiler);
+
+ expect(getWarnings(stats)).toMatchSnapshot("warnings");
+ expect(getErrors(stats)).toMatchSnapshot("errors");
+ });
 });