Merge pull request #99 from Marsup/no-argument-setter

Fix: bad error in parsing invalid class setter (fixes #98)
eslint · Mar 21, 2015 · 12ebb32 · 12ebb32
2 parents 969aecf + 03b1fa4
commit 12ebb32
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/espree.js b/espree.js
@@ -2435,7 +2435,7 @@ function tryParseMethodDefinition(token, key, computed, marker) {
                 rest: null
             };
             if (match(")")) {
-                throwErrorTolerant(lookahead, Messages.UnexpectedToken);
+                throwErrorTolerant(lookahead, Messages.UnexpectedToken, lookahead.value);
             } else {
                 parseParam(options);
                 if (options.defaultCount === 0) {

diff --git a/tests/fixtures/ecma-features/classes/invalid-class-setter-declaration.result.js b/tests/fixtures/ecma-features/classes/invalid-class-setter-declaration.result.js
@@ -0,0 +1,7 @@
+module.exports = {
+    "index": 18,
+    "lineNumber": 1,
+    "column": 19,
+    "message": "Line 1: Unexpected token )",
+    "description": "Unexpected token )"
+};
diff --git a/tests/fixtures/ecma-features/classes/invalid-class-setter-declaration.src.js b/tests/fixtures/ecma-features/classes/invalid-class-setter-declaration.src.js
@@ -0,0 +1 @@
+class A { set foo() {}};