chore: prepare 2.0.1 release

Author: evilebottnawi

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
 
 This project adheres to [Semantic Versioning](http://semver.org).
 
+## 2.0.1 - 2018-04-19
+
+* Fix: don't crash when argument is empty (string/array/object).
+* Fix: improve output error on empty or invalid event.
+
 ## 2.0.0 - 2018-04-19
 
 * Changed: remove `webpack@2` and `webpack@3` support.

__tests__/index.test.js

@@ -56,9 +56,21 @@ describe("execa-webpack-plugin", () => {
   const dir = path.join(__dirname, "dir");
   const otherDir = path.join(__dirname, "other-dir");
 
-  it("should throw error when `on*` options are empty (no events)", () =>
+  it("should throw error on `on*` options are empty (no events)", () =>
     expect(() => run()).toThrow());
 
+  it("should throw error invalid `onFooBar` option (no events)", () =>
+    expect(() =>
+      run({
+        onFooBar: [
+          {
+            args: [path.join(resourcesDir, "nothing.js")],
+            cmd: "node"
+          }
+        ]
+      })
+    ).toThrow());
+
   it("should works with `on*` options", () =>
     run({
       onAdditionalPass: [
@@ -234,6 +246,29 @@ describe("execa-webpack-plugin", () => {
     });
   });
 
+  it("should works with `onCompile` option with empty argument (sync event) ", () => {
+    expect.assertions(2);
+
+    mkdirSyncSafe(dir);
+
+    expect(fs.statSync(dir).isDirectory()).toBe(true);
+
+    return run({
+      onCompile: [
+        {
+          args: [dir, "", [], {}],
+          cmd: "del"
+        }
+      ]
+    }).then(() => {
+      expect(() => fs.statSync(dir)).toThrow();
+
+      unlinkSyncSafe(dir);
+
+      return Promise.resolve();
+    });
+  });
+
   it("should works with `onCompile` and `dev` is `false` option (sync event)", () => {
     expect.assertions(2);
 
@@ -282,6 +317,29 @@ describe("execa-webpack-plugin", () => {
     });
   });
 
+  it("should works with `onDone` option with empty argument (async event) ", () => {
+    expect.assertions(2);
+
+    mkdirSyncSafe(dir);
+
+    expect(fs.statSync(dir).isDirectory()).toBe(true);
+
+    return run({
+      onDone: [
+        {
+          args: [dir, "", [], {}],
+          cmd: "del"
+        }
+      ]
+    }).then(() => {
+      expect(() => fs.statSync(dir)).toThrow();
+
+      unlinkSyncSafe(dir);
+
+      return Promise.resolve();
+    });
+  });
+
   it("should works with `onDone` and `dev` is `false` options (async event)", () => {
     expect.assertions(2);
 

index.js

@@ -56,7 +56,7 @@ class ChildProcessWebpackPlugin {
     });
 
     if (Object.keys(this.eventMap).length === 0) {
-      throw new TypeError("No events found");
+      throw new TypeError("Known events not found");
     }
 
     this.log = weblog({
@@ -139,7 +139,11 @@ class ChildProcessWebpackPlugin {
       const args = process.args || [];
 
       args.forEach((arg, index) => {
-        if (typeof arg === "object" && Boolean(arg)) {
+        if (
+          typeof arg === "object" &&
+          Boolean(arg) &&
+          Object.keys(arg).length > 0
+        ) {
           const commandResult = this.execute([arg], async);
 
           process.args[index] = Array.isArray(commandResult)
@@ -155,7 +159,7 @@ class ChildProcessWebpackPlugin {
       if (async) {
         result = Promise.all(args).then(resolvedArgs => {
           process.args = resolvedArgs.map(
-            item => (item[0].stdout ? item[0].stdout : item)
+            item => (item[0] && item[0].stdout ? item[0].stdout : item)
           );
 
           return this.runCommand(process, async);
@@ -167,11 +171,7 @@ class ChildProcessWebpackPlugin {
       results.push(result);
     });
 
-    return async
-      ? Promise.all(results).catch(error => {
-          this.handleError(error, process);
-        })
-      : results;
+    return async ? Promise.all(results) : results;
   }
 
   apply(compiler) {

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "execa-webpack-plugin",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "A better `child_process` for `webpack`",
   "keywords": [
     "exec",