Add fixer for no-now

See #115
wikimedia · Oct 3, 2019 · fed4173 · fed4173
1 parent c315d71
commit fed4173
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/docs/no-now.md b/docs/no-now.md
@@ -4,6 +4,8 @@ Disallows the $.now utility. Prefer (new Date).getTime() to $.now.
 
 This rule is enabled in `plugin:no-jquery/deprecated-3.3`.
 
+🔧 The `--fix` option can be used to fix problems reported by this rule.
+
 ## Rule details
 
 ❌ The following patterns are considered errors:

diff --git a/rules/no-now.js b/rules/no-now.js
@@ -4,5 +4,9 @@ const utils = require( './utils.js' );
 
 module.exports = utils.createUtilMethodRule(
 	'now',
-	'Prefer (new Date).getTime() to $.now'
+	'Prefer (new Date).getTime() to $.now',
+	'code',
+	function ( node, fixer ) {
+		return fixer.replaceText( node.callee, 'Date.now' );
+	}
 );
diff --git a/tests/no-now.js b/tests/no-now.js
@@ -11,7 +11,8 @@ ruleTester.run( 'no-now', rule, {
 	invalid: [
 		{
 			code: '$.now()',
-			errors: [ { message: error, type: 'CallExpression' } ]
+			errors: [ { message: error, type: 'CallExpression' } ],
+			output: 'Date.now()'
 		}
 	]
 } );