Fix bug in tests err vs. error

Author: RyanZim

test/error.js

@@ -4,24 +4,24 @@ import tmp from './helpers/tmp.js'
 import cli from './helpers/cli.js'
 
 test('multiple input files && --output', t => {
-  return cli(['test/fixtures/*.css', '-o', tmp()]).then(({ err, code }) => {
+  return cli(['test/fixtures/*.css', '-o', tmp()]).then(({ error, code }) => {
     t.is(code, 1, 'expected non-zero error code')
-    t.regex(err.toString(), /Input Error: Must use --dir or --replace/)
+    t.regex(error.toString(), /Input Error: Must use --dir or --replace/)
   })
 })
 
 test('multiple input files && writing to stdout', t => {
-  return cli(['test/fixtures/*.css']).then(({ err, code }) => {
+  return cli(['test/fixtures/*.css']).then(({ error, code }) => {
     t.is(code, 1, 'expected non-zero error code')
-    t.regex(err.toString(), /Input Error: Must use --dir or --replace/)
+    t.regex(error.toString(), /Input Error: Must use --dir or --replace/)
   })
 })
 
 test('--map && writing to stdout', t => {
-  return cli(['test/fixtures/a.css', '--map']).then(({ err, code }) => {
+  return cli(['test/fixtures/a.css', '--map']).then(({ error, code }) => {
     t.is(code, 1, 'expected non-zero error code')
     t.regex(
-      err.toString(),
+      error.toString(),
       /Output Error: Cannot output external sourcemaps when writing to STDOUT/
     )
   })
@@ -34,18 +34,18 @@ test.failing('invalid --config', t => {
     'test/postcss.config.js',
     '-d',
     tmp()
-  ]).then(({ err, code }) => {
+  ]).then(({ error, code }) => {
     t.is(code, 1, 'expected non-zero error code')
-    t.regex(err.toString(), /ENOENT: no such file or directory/)
+    t.regex(error.toString(), /ENOENT: no such file or directory/)
   })
 })
 
 test('plugin not found', t => {
   return cli(['test/fixtures/a.css', '-u', 'postcss-plugin', '-o', tmp()]).then(
-    ({ err, code }) => {
+    ({ error, code }) => {
       t.is(code, 1, 'expected non-zero error code')
       t.regex(
-        err.toString(),
+        error.toString(),
         /Plugin Error: Cannot find module 'postcss-plugin'/
       )
     }
@@ -59,18 +59,18 @@ test('plugin throws on require', t => {
     './test/fixtures/bad-plugin',
     '-o',
     tmp()
-  ]).then(({ err, code }) => {
+  ]).then(({ error, code }) => {
     t.is(code, 1, 'expected non-zero error code')
-    t.regex(err.toString(), /Plugin Error \(.*bad-plugin\): This fails/)
+    t.regex(error.toString(), /Plugin Error \(.*bad-plugin\): This fails/)
   })
 })
 
 test('CssSyntaxError', t => {
   return cli(['test/fixtures/a.css', '--parser', 'sugarss', '-o', tmp()]).then(
-    ({ err, code }) => {
+    ({ error, code }) => {
       t.is(code, 1, 'expected non-zero error code')
       t.regex(
-        err.toString(),
+        error.toString(),
         /CssSyntaxError: .*a.css:1:4: Unnecessary curly bracket/
       )
     }

test/helpers/cli.js

@@ -6,10 +6,10 @@ export default function(args, cwd) {
     exec(
       `node ${path.resolve('bin/postcss')} ${args.join(' ')}`,
       { cwd },
-      (err, stdout, stderr) => {
+      (error, stdout, stderr) => {
         resolve({
-          code: err && err.code ? err.code : 0,
-          err,
+          code: error && error.code ? error.code : 0,
+          error,
           stdout,
           stderr
         })