fix: make no-pattern error print correctly

Fixes #8
xz64 · Dec 18, 2016 · 28f1e1d · 28f1e1d
1 parent 16c6b59
commit 28f1e1d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -242,8 +242,7 @@ var instance = function() {
 
 var licensePlugin = function(opts) {
   if(!opts || !opts.pattern || !(opts.pattern instanceof RegExp)) {
-    this.errors.push(plugin.errorMessages['no-pattern']);
-    throw plugin.errorMessages['no-pattern'];
+    throw new Error(plugin.errorMessages['no-pattern']);
   }
   objectAssign(this, composedPlugin, instance(), opts);
   this.apply = this.apply.bind(this);

diff --git a/test/index.test.js b/test/index.test.js
@@ -566,5 +566,11 @@ test('plugin', function (t) {
     t.end();
   });
 
+  t.test('the plugin throws a descriptive error when no pattern is specified',
+    function(t) {
+    t.throws(createPlugin.bind(null, {}));
+    t.end();
+  });
+
   t.end();
 });