Quick and dirty - does not error with not supported plugin

Related #192

Author: filipesperandio

lib/eslint-patch.js

@@ -1,33 +1,30 @@
 'use strict';
-var meld = require('meld');
+var fs = require('fs');
+var path = require('path');
+
+var Plugins = require("eslint/lib/config/plugins");
+
 var docs = require('./docs');
 var Config = require("eslint/lib/config");
 var ConfigUpgrader = require('./config_upgrader');
 
-var supportedPlugins = ['react', 'babel'];
 
-module.exports = function patcher(eslint) {
+module.exports = function patcher(engine) {
 
-  meld.around(eslint.CLIEngine, 'loadPlugins', function(joinPoint) {
-    var pluginNames = joinPoint.args[0];
-    var filteredPluginNames = pluginNames.filter(function(pluginName) {
-      return supportedPlugins.indexOf(pluginName) >= 0;
-    });
-    return joinPoint.proceed(filteredPluginNames);
-  });
+  Plugins.loadAll = function(pluginNames) {
+    for(var index in pluginNames) {
+      var name = pluginNames[index];
 
-  meld.around(eslint.CLIEngine, 'addPlugin', function() {
-    return;
-  });
+      try {
+        Plugins.load(name);
+        console.error('>>>> Plugin loaded', name);
+      } catch(e) {
+        console.error('>>>> Problem loading plugin', name);
+      }
+    }
+
+  }
 
-  // meld.around(eslint.CLIEngine.Config, 'loadPackage', function(joinPoint) {
-  //   var filePath = joinPoint.args[0];
-  //   if (filePath.match(/^eslint-config-airbnb.*/)) {
-  //     return joinPoint.proceed();
-  //   } else {
-  //     return {};
-  //   }
-  // });
 
   const originalGetConfig = Config.prototype.getConfig;
   Config.prototype.getConfig = function(filePath) {
@@ -37,7 +34,8 @@ module.exports = function patcher(eslint) {
     return configUpgrader.upgrade(originalConfig);
   };
 
-  eslint.docs = docs;
+  engine.docs = docs;
+
 
-  return eslint;
+  return engine;
 };

test/eslint-patch_test.js

@@ -0,0 +1,14 @@
+var eslint = require("eslint");
+var expect = require("chai").expect;
+
+describe('eslint-patch', function() {
+  it('intercept plugins', function() {
+    var Plugins = require('eslint/lib/config/plugins');
+    var loadAll = Plugins.loadAll;
+
+    var patch = require("../lib/eslint-patch");
+    patch(eslint);
+
+    expect(loadAll).to.not.equal(Plugins.loadAll, 'Plugins.loadAll is not patched');
+  });
+});