feat: supported by dependency (#407)

Author: alexander-akait

src/utils.js

@@ -32,6 +32,7 @@ const MODULE_REQUEST_REGEX = /^[^?]*~/;
  */
 function createWebpackLessPlugin(loaderContext) {
   const resolve = loaderContext.getResolve({
+    dependencyType: "less",
     conditionNames: ["less", "style"],
     mainFields: ["less", "style", "main", "..."],
     mainFiles: ["index", "..."],

test/__snapshots__/loader.test.js.snap

@@ -525,6 +525,20 @@ exports[`loader should watch imports correctly: errors 1`] = `Array []`;
 
 exports[`loader should watch imports correctly: warnings 1`] = `Array []`;
 
+exports[`loader should work and respect the 'resolve.byDependecy.less' option: css 1`] = `
+".a {
+  color: red;
+}
+.b {
+  color: red;
+}
+"
+`;
+
+exports[`loader should work and respect the 'resolve.byDependecy.less' option: errors 1`] = `Array []`;
+
+exports[`loader should work and respect the 'resolve.byDependecy.less' option: warnings 1`] = `Array []`;
+
 exports[`loader should work lessOptions.relativeUrls is false: css 1`] = `
 ".top-import {
   background: red;

test/fixtures/by-dependency.less

@@ -0,0 +1,5 @@
+@import "custom-main-files";
+
+.b {
+  color: red
+}

test/fixtures/circular.less

@@ -0,0 +1,5 @@
+@import "circular";
+
+a {
+  color: red;
+}

test/fixtures/custom-main-files/custom.less

@@ -0,0 +1,3 @@
+.a {
+  color: red
+}

test/helpers/getCodeFromLess.js

@@ -108,6 +108,13 @@ const pathMap = {
     "prefer-relative",
     "index.less"
   ),
+  "custom-main-files": path.resolve(
+    __dirname,
+    "..",
+    "fixtures",
+    "custom-main-files",
+    "custom.less"
+  ),
 };
 
 class ResolvePlugin extends less.FileManager {

test/loader.test.js

@@ -838,4 +838,43 @@ describe("loader", () => {
     expect(getWarnings(stats)).toMatchSnapshot("warnings");
     expect(getErrors(stats)).toMatchSnapshot("errors");
   });
+
+  // TODO bug on windows
+  it.skip("should work with circular imports", async () => {
+    const testId = "./circular.less";
+    const compiler = getCompiler(testId);
+    const stats = await compile(compiler);
+    const codeFromBundle = getCodeFromBundle(stats, compiler);
+    const codeFromLess = await getCodeFromLess(testId);
+
+    expect(codeFromBundle.css).toBe(codeFromLess.css);
+    expect(codeFromBundle.css).toMatchSnapshot("css");
+    expect(getWarnings(stats)).toMatchSnapshot("warnings");
+    expect(getErrors(stats)).toMatchSnapshot("errors");
+  });
+
+  it("should work and respect the 'resolve.byDependecy.less' option", async () => {
+    const testId = "./by-dependency.less";
+    const compiler = getCompiler(
+      testId,
+      {},
+      {
+        resolve: {
+          byDependency: {
+            less: {
+              mainFiles: ["custom"],
+            },
+          },
+        },
+      }
+    );
+    const stats = await compile(compiler);
+    const codeFromBundle = getCodeFromBundle(stats, compiler);
+    const codeFromLess = await getCodeFromLess(testId);
+
+    expect(codeFromBundle.css).toBe(codeFromLess.css);
+    expect(codeFromBundle.css).toMatchSnapshot("css");
+    expect(getWarnings(stats)).toMatchSnapshot("warnings");
+    expect(getErrors(stats)).toMatchSnapshot("errors");
+  });
 });