fix: Bare imports not working sometimes

Fixes #566 Supersedes #577
webpack-contrib · Jun 4, 2018 · 745e241 · 745e241
1 parent 2efb8cf
commit 745e241
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 1 deletion.
diff --git a/test/index.test.js b/test/index.test.js
@@ -205,7 +205,7 @@ describe("sass-loader", () => {
                     sourceMap.should.not.have.property("file");
                     sourceMap.should.have.property("sourceRoot", fakeCwd);
                     // This number needs to be updated if imports.scss or any dependency of that changes
-                    sourceMap.sources.should.have.length(11);
+                    sourceMap.sources.should.have.length(13);
                     sourceMap.sources.forEach(sourcePath =>
                         fs.existsSync(path.resolve(sourceMap.sourceRoot, sourcePath))
                     );

diff --git a/test/node_modules/module b/test/node_modules/module
diff --git a/test/node_modules/other-module b/test/node_modules/other-module
diff --git a/test/sass/imports.sass b/test/sass/imports.sass
@@ -22,3 +22,7 @@
 // See https://github.com/webpack-contrib/sass-loader/issues/556
 /* @import util */
 @import util
+/* @import ~module */
+@import ~module
+/* @import ~other-module */
+@import ~other-module
diff --git a/test/scss/imports.scss b/test/scss/imports.scss
@@ -23,4 +23,9 @@
 }
 // The local util file should take precedence over Node's util module
 // See https://github.com/webpack-contrib/sass-loader/issues/556
+/* @import "util"; */
 @import "util";
+/* @import "~module"; */
+@import "~module";
+/* @import "~other-module"; */
+@import "~other-module";