[wip] - adding support for plugin errors

Author: shakyShane

lib/async.js

@@ -126,7 +126,7 @@ module.exports = {
             });
 
         function logPluginError (plugin) {
-            utils.fail(true, new Error("Plugin: " + plugin.get("name") + " not found"), bs.cb);
+            utils.fail(true, plugin.getIn(["errors", 0]), bs.cb);
         }
 
         done();

lib/plugins.js

@@ -1,5 +1,6 @@
 var Immutable  = require("immutable");
 var Map        = Immutable.Map;
+var isMap      = Immutable.Map.isMap;
 var List       = Immutable.List;
 var qs         = require("qs");
 var path       = require("path");
@@ -32,6 +33,10 @@ function resolvePlugin(item) {
         return getFromString(item);
     }
 
+    if (!isMap(item)) {
+        return new Plugin().mergeDeep({errors: [new Error("Plugin not supported in this format")]});
+    }
+
     if (item.has("module")) {
 
         var nameOrObj = item.get("module");

test/specs/plugins/user.plugins.inline.error.js

@@ -0,0 +1,24 @@
+"use strict";
+
+var browserSync = require("../../../");
+
+var assert  = require("chai").assert;
+
+describe("Plugins: Retrieving user plugins when given inline, but with error", function () {
+    it.only("Should fail if a plugin error occurred", function (done) {
+        browserSync.reset();
+
+        browserSync({
+            plugins: {
+                module: {plugin: function () {
+
+                }}
+            },
+            open: false,
+            logLevel: "silent"
+        }, function (err, bs) {
+            console.log(err);
+            done();
+        });
+    });
+});