Rule fix: Add fixer for no-error (#272)

Also fix copy/paste error in valid cases.

Part of #115

docs/rules/no-error.md

@@ -4,18 +4,25 @@ Disallows the [`$.error`](https://api.jquery.com/jQuery.error/) utility. Prefer
 
 ⚙️ This rule is enabled in `plugin:no-jquery/all`.
 
+🔧 The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
 ## Rule details
 
 ❌ Examples of **incorrect** code:
 ```js
-$.error();
+$.error( msg );
 ```
 
 ✔️ Examples of **correct** code:
 ```js
-nodeName();
-myClass.nodeName();
-$div.nodeName();
+error( msg );
+myClass.error( msg );
+$div.error( msg );
+```
+
+🔧 Examples of code **fixed** by this rule:
+```js
+$.error( msg ); /* → */ throw new Error( msg );
 ```
 
 ## Resources

src/rules/no-error.js

@@ -4,5 +4,11 @@ const utils = require( '../utils.js' );
 
 module.exports = utils.createUtilMethodRule(
 	'error',
-	'Prefer `throw` to `$.error`'
+	'Prefer `throw` to `$.error`',
+	{
+		fixable: 'code',
+		fix: function ( node, fixer ) {
+			return fixer.replaceText( node.callee, 'throw new Error' );
+		}
+	}
 );

tests/rules/no-error.js

@@ -7,11 +7,12 @@ const error = 'Prefer throw to $.error';
 
 const ruleTester = new RuleTester();
 ruleTester.run( 'no-error', rule, {
-	valid: [ 'nodeName()', 'myClass.nodeName()', '$div.nodeName()' ],
+	valid: [ 'error(msg)', 'myClass.error(msg)', '$div.error(msg)' ],
 	invalid: [
 		{
-			code: '$.error()',
-			errors: [ error ]
+			code: '$.error(msg)',
+			errors: [ error ],
+			output: 'throw new Error(msg)'
 		}
 	]
 } );