⬆️

Author: princed

index.js

@@ -70,6 +70,7 @@ const walk = async (requiredDefinitions, walkFile, root, result) => {
 
       // We can use constants for path names
       if (existingDefinitions[pathString]) {
+        // eslint-disable-next-line prefer-destructuring
         pathString = existingDefinitions[pathString];
       }
 

package.json

@@ -28,13 +28,13 @@
     "postcss": "^6.0.1"
   },
   "devDependencies": {
-    "ava": "^0.21.0",
+    "ava": "^0.24.0",
     "babel-cli": "^6.18.0",
     "babel-core": "^6.18.2",
     "babel-polyfill": "^6.16.0",
     "babel-register": "^6.22.0",
-    "eslint": "^3.9.0",
-    "eslint-config-airbnb-base": "^11.2.0",
+    "eslint": "^4.15.0",
+    "eslint-config-airbnb-base": "^12.1.0",
     "eslint-plugin-import": "^2.5.0"
   },
   "scripts": {

test.js

@@ -90,7 +90,7 @@ test('should replace an import from several files', async (t) => {
 .a { margin: base; }
 .b { margin: level2base; }
 .foo { color: red; }`,
-`@value red from "./colors.css";
+    `@value red from "./colors.css";
 @value base from "./level1.css";
 @value level2base from "./level2.css";
 .a { margin: 10px; }
@@ -111,7 +111,8 @@ test('should import and alias a constant and replace usages', async (t) => {
   await run(
     t,
     '@value blue as green from "./colors.css";\n.foo { color: green; }',
-    '@value blue as green from "./colors.css";\n.foo { color: #0000FF; }');
+    '@value blue as green from "./colors.css";\n.foo { color: #0000FF; }',
+  );
 });
 
 test('should import and alias a constant (using a name from imported file) and replace usages', async (t) => {
@@ -304,8 +305,7 @@ test('variables are also present in messages', async (t) => {
   const input = '@value myColor: blue; @value myColor2: myColor';
   const processor = postcss([plugin]);
   const result = await processor.process(input);
-  const values = result.messages[0].values;
-  const type = result.messages[0].type;
+  const { values, type } = result.messages[0];
 
   t.is(type, 'values');
   t.is(values.myColor2, 'blue');