catch-error-name - Support optional catch binding (#187)

Author: MrHen

rules/catch-error-name.js

@@ -85,6 +85,12 @@ const create = context => {
 			}
 		},
 		CatchClause: node => {
+			// Optional catch binding
+			if (!node || !node.param) {
+				push(true);
+				return;
+			}
+
 			if (node.param.name === '_') {
 				push(!astUtils.someContainIdentifier('_', node.body.body));
 				return;

test/catch-error-name.js

@@ -120,7 +120,15 @@ ruleTester.run('catch-error-name', rule, {
 					caughtErrorsIgnorePattern: '^skip'
 				}
 			]
-		}
+		},
+		// TODO: Uncomment once test runner supports optional-catch-binding https://github.com/tc39/proposal-optional-catch-binding
+		// testCase(`
+		// 	try {
+		// 		throw new Error('message');
+		// 	} catch {
+		// 		console.log('failed');
+		// 	}
+		// `),
 	],
 	invalid: [
 		testCase('try {} catch (err) {}', null, true),