Merge pull request #182 from lo1tuma/fix-test

Fix invalid syntax in test case
lo1tuma · Oct 26, 2018 · 5b7f988 · 5b7f988
2 parents 8ad5267 + 798efa0
commit 5b7f988
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 16 deletions.
diff --git a/lib/rules/prefer-arrow-callback.js b/lib/rules/prefer-arrow-callback.js
@@ -27,17 +27,6 @@ function isFunctionName(variable) {
     return variable && variable.defs[0].type === 'FunctionName';
 }
 
-/**
- * Checks whether or not a given MetaProperty node equals to a given value.
- * @param {ASTNode} node - A MetaProperty node to check.
- * @param {string} metaName - The name of `MetaProperty.meta`.
- * @param {string} propertyName - The name of `MetaProperty.property`.
- * @returns {boolean} `true` if the node is the specific value.
- */
-function checkMetaProperty(node, metaName, propertyName) {
-    return node.meta.name === metaName && node.property.name === propertyName;
-}
-
 /**
  * Gets the variable object of `arguments` which is defined implicitly.
  * @param {eslint-scope.Scope} scope - A scope to get.
@@ -245,12 +234,10 @@ module.exports = {
                 }
             },
 
-            MetaProperty(node) {
+            MetaProperty() {
                 const info = stack[stack.length - 1];
 
-                if (info && checkMetaProperty(node, 'new', 'target')) {
-                    info.meta = true;
-                }
+                info.meta = true;
             },
 
             // To skip nested scopes.

diff --git a/test/rules/prefer-arrow-callback.js b/test/rules/prefer-arrow-callback.js
@@ -45,7 +45,6 @@ ruleTester.run('prefer-arrow-callback', rules['prefer-arrow-callback'], {
         '() => super()',
         'foo(function bar() { new.target; });',
         'foo(function bar() { new.target; }.bind(this));',
-        '() => new.target',
         'foo(function bar() { this; }.bind(this, somethingElse));',
         // mocha-specific valid test cases
         'before(function bar() {});',