fix: 🐛 scss with empty content

✅ Closes: Closes #166

Author: kaisermann

src/transformers/globalRule.ts

@@ -19,7 +19,7 @@ const globalifyRulePlugin = (root: any) => {
 const transformer: Transformer<never> = async ({ content, filename }) => {
   const { css, map: newMap } = await postcss()
     .use(globalifyRulePlugin)
-    .process(content, { from: filename, map: true });
+    .process(content, { from: filename, map: false });
 
   return { code: css, map: newMap };
 };

src/transformers/globalStyle.ts

@@ -22,7 +22,7 @@ const globalifyPlugin = (root: any) => {
 const transformer: Transformer<never> = async ({ content, filename }) => {
   const { css, map: newMap } = await postcss()
     .use(globalifyPlugin)
-    .process(content, { from: filename, map: true });
+    .process(content, { from: filename, map: false });
 
   return { code: css, map: newMap };
 };

src/transformers/scss.ts

@@ -45,7 +45,7 @@ const transformer: Transformer<Options.Sass> = async ({
 
   // scss errors if passed an empty string
   if (sassOptions.data.length === 0) {
-    return { code: options.data };
+    return { code: '' };
   }
 
   if (renderSync) {

test/autoProcess/style.test.ts

@@ -15,12 +15,11 @@ const STYLE_LANGS: Array<[string, string]> = [
 
 STYLE_LANGS.forEach(([lang, ext]) => {
   describe(`style - preprocessor - ${lang}`, () => {
-    const template = `<div></div><style lang="${lang}">${getFixtureContent(
-      `style.${ext}`,
-    )}</style>`;
-    const templateExternal = `<div></div><style src="./fixtures/style.${ext}"></style>`;
-
     it(`should throw parsing ${lang} when { ${lang}: false }`, async () => {
+      const template = `<div></div><style lang="${lang}">${getFixtureContent(
+        `style.${ext}`,
+      )}</style>`;
+
       const opts = getAutoPreprocess({
         [lang]: false,
       });
@@ -29,17 +28,40 @@ STYLE_LANGS.forEach(([lang, ext]) => {
     });
 
     it(`should parse ${lang}`, async () => {
+      const template = `<div></div><style lang="${lang}">${getFixtureContent(
+        `style.${ext}`,
+      )}</style>`;
+
       const opts = getAutoPreprocess();
       const preprocessed = await preprocess(template, opts);
 
       expect(preprocessed.toString()).toMatch(CSS_PATTERN);
     });
 
     it(`should parse external ${lang}`, async () => {
+      const templateExternal = `<div></div><style src="./fixtures/style.${ext}"></style>`;
+      const opts = getAutoPreprocess();
+      const preprocessed = await preprocess(templateExternal, opts);
+
+      expect(preprocessed.toString()).toMatch(CSS_PATTERN);
+    });
+
+    it(`should parse external ${lang}`, async () => {
+      const templateExternal = `<div></div><style src="./fixtures/style.${ext}"></style>`;
       const opts = getAutoPreprocess();
       const preprocessed = await preprocess(templateExternal, opts);
 
       expect(preprocessed.toString()).toMatch(CSS_PATTERN);
     });
+
+    it(`should return empty if content is empty`, async () => {
+      const templateExternal = `<div></div><style src="./potato/style.${ext}"></style>`;
+      const opts = getAutoPreprocess({
+        [lang]: { sourceMap: false, sourcemap: false, map: false },
+      });
+      const preprocessed = await preprocess(templateExternal, opts);
+
+      expect(preprocessed.toString()).toMatch(templateExternal);
+    });
   });
 });