fix: don't crash when raws.before is undefined (#60)

css-modules · Feb 26, 2019 · 43168ab · 43168ab
1 parent 2bfed2e
commit 43168ab
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/src/extractICSS.js b/src/extractICSS.js
@@ -3,7 +3,8 @@ const importPattern = /^:import\(("[^"]*"|'[^']*'|[^"']+)\)$/;
 const getDeclsObject = rule => {
   const object = {};
   rule.walkDecls(decl => {
-    object[decl.raws.before.trim() + decl.prop] = decl.value;
+    const before = decl.raws.before ? decl.raws.before.trim() : "";
+    object[before + decl.prop] = decl.value;
   });
   return object;
 };

diff --git a/test/extractICSS.test.js b/test/extractICSS.test.js
@@ -27,7 +27,7 @@ test("extract :import statements with single quoted path", () => {
   });
 });
 
-test("extract :import statements with double quoted path", () => {
+test("extract manually added :import", () => {
   expect(runExtract(':import("./colors.css") {}')).toEqual({
     icssImports: {
       "./colors.css": {}
@@ -50,6 +50,25 @@ test("not extract :import with values", () => {
   });
 });
 
+test("extract :import statements manually created in postcss", () => {
+  const root = postcss.parse("");
+  root.append(
+    postcss
+      .rule({ selector: ":import(./colors.css)" })
+      .append(postcss.decl({ prop: "i__blue", value: "blue" }))
+      .append(postcss.decl({ prop: "i__red", value: "red" }))
+  );
+  expect(extractICSS(root)).toEqual({
+    icssImports: {
+      "./colors.css": {
+        i__blue: "blue",
+        i__red: "red"
+      }
+    },
+    icssExports: {}
+  });
+});
+
 test("not extract invalid :import", () => {
   expect(runExtract(":import(\\'./colors.css) {}")).toEqual({
     icssImports: {},