fix(methods): Restirct warnings on NewExpressions to Meteor.Error

Previously the rule would warn on all NewExpressions with no arguments.

lib/rules/methods.js

@@ -8,7 +8,7 @@
 
 import {NON_METEOR} from '../util/environment'
 import {getExecutors} from '../util'
-import {isMeteorCall, getPropertyName, refersTo, hasContext} from '../util/ast'
+import {isMeteorCall, isMeteorProp, getPropertyName, refersTo, hasContext} from '../util/ast'
 
 // -----------------------------------------------------------------------------
 // Rule Definition
@@ -117,6 +117,11 @@ module.exports = getMeta => context => {
   return {
 
     NewExpression: function (node) {
+
+      if (!isMeteorProp(node.callee, 'Error')) {
+        return
+      }
+
       const executors = getExecutors(env, context.getAncestors())
       if (executors.size === 0) {
         return

tests/lib/rules/methods.js

@@ -14,6 +14,7 @@ const rule = require('../../../dist/rules/methods')
 const RuleTester = require('eslint').RuleTester
 
 const commonValidCode = [
+  'new Meteor.foo()',
   'Meteor.call("foo")',
   'Meteor.call("foo", true)',
   'Meteor.apply("foo", [], function () {})',