test: import without quotes (#726)

webpack-contrib · Aug 16, 2019 · 6be93c8 · 6be93c8
1 parent dc23895
commit 6be93c8
Show file tree

Hide file tree

Showing 3 changed files with 48 additions and 0 deletions.
diff --git a/test/__snapshots__/loader.test.js.snap b/test/__snapshots__/loader.test.js.snap
@@ -602,6 +602,14 @@ exports[`loader should work with multiple "@import" at-rules (node-sass) (scss):
 
 exports[`loader should work with multiple "@import" at-rules (node-sass) (scss): warnings 1`] = `Array []`;
 
+exports[`loader should work with multiple "@import" at-rules without quotes (dart-sass) (sass): errors 1`] = `Array []`;
+
+exports[`loader should work with multiple "@import" at-rules without quotes (dart-sass) (sass): warnings 1`] = `Array []`;
+
+exports[`loader should work with multiple "@import" at-rules without quotes (node-sass) (sass): errors 1`] = `Array []`;
+
+exports[`loader should work with multiple "@import" at-rules without quotes (node-sass) (sass): warnings 1`] = `Array []`;
+
 exports[`loader should work with the "bootstrap" package, import as a package (dart-sass) (sass): errors 1`] = `Array []`;
 
 exports[`loader should work with the "bootstrap" package, import as a package (dart-sass) (sass): warnings 1`] = `Array []`;

diff --git a/test/loader.test.js b/test/loader.test.js
@@ -339,6 +339,25 @@ describe('loader', () => {
         expect(stats.compilation.errors).toMatchSnapshot('errors');
       });
 
+      it(`should work with multiple "@import" at-rules without quotes (${implementationName}) (${syntax})`, async () => {
+        if (syntax === 'scss') {
+          return;
+        }
+
+        const testId = getTestId('import-without-quotes', syntax);
+        const options = {
+          implementation: getImplementationByName(implementationName),
+        };
+        const stats = await compile(testId, { loader: { options } });
+
+        expect(getCodeFromBundle(stats).css).toBe(
+          getCodeFromSass(testId, options).css
+        );
+
+        expect(stats.compilation.warnings).toMatchSnapshot('warnings');
+        expect(stats.compilation.errors).toMatchSnapshot('errors');
+      });
+
       it(`should work and use the "sass" field (${implementationName}) (${syntax})`, async () => {
         const testId = getTestId('import-sass-field', syntax);
         const options = {

diff --git a/test/sass/import-without-quotes.sass b/test/sass/import-without-quotes.sass
@@ -0,0 +1,21 @@
+@import example
+@import ./example
+@import example.sass
+@import ./example.sass
+@import example-2
+@import ./example-2
+@import _example-2
+@import ./_example-2
+@import directory/file
+@import ./directory/file
+@import directory/_file
+@import ./directory/_file
+@import directory-6/file
+@import ./directory-6/file
+@import directory-6/file/index
+@import ./directory-6/file/index
+@import directory-6/file/index.sass
+@import ./directory-6/file/index.sass
+@import directory-6/file/_index.sass
+@import ./directory-6/file/_index.sass
+@import ~sass/underscore