add --fix to no-identical-tests.

README.md

@@ -56,7 +56,7 @@ Name | ✔️ | 🛠 | Description
 [prefer-placeholders](https://github.com/not-an-aardvark/eslint-plugin-eslint-plugin/blob/master/docs/rules/prefer-placeholders.md) | | | Disallows template literals as report messages
 [no-useless-token-range](https://github.com/not-an-aardvark/eslint-plugin-eslint-plugin/blob/master/docs/rules/no-useless-token-range.md) | ✔️ | 🛠 | Disallows unnecessary calls to sourceCode.getFirstToken and sourceCode.getLastToken
 [consistent-output](https://github.com/not-an-aardvark/eslint-plugin-eslint-plugin/blob/master/docs/rules/consistent-output.md) | | | Enforces consistent use of output assertions in rule tests
-[no-identical-tests](https://github.com/not-an-aardvark/eslint-plugin-eslint-plugin/blob/master/docs/rules/no-identical-tests.md) | | | Disallows identical tests
+[no-identical-tests](https://github.com/not-an-aardvark/eslint-plugin-eslint-plugin/blob/master/docs/rules/no-identical-tests.md) | | 🛠 | Disallows identical tests
 
 ## Supported Presets
 

docs/rules/no-identical-tests.md

@@ -1,5 +1,7 @@
 # Disallow identical tests (no-identical-tests)
 
+(fixable) The `--fix` option on the [command line](../user-guide/command-line-interface#fix) automatically fixes problems reported by this rule.
+
 When a rule has a lot of tests, it's sometimes difficult to tell if any tests are duplicates. This rule would warn if any test cases have the same properties.
 
 ## Rule Details

lib/rules/no-identical-tests.js

@@ -18,7 +18,7 @@ module.exports = {
       category: 'Tests',
       recommended: false,
     },
-    fixable: null,  // or "code" or "whitespace"
+    fixable: 'code',
     schema: [],
   },
 
@@ -41,6 +41,14 @@ module.exports = {
                 context.report({
                   node: test,
                   message,
+                  fix (fixer) {
+                    const start = sourceCode.getTokenBefore(test);
+                    const end = sourceCode.getTokenAfter(test);
+                    return fixer.replaceTextRange(
+                      // should remove test's trailing comma
+                      [start.range[1], end.range[end.value === ',' ? 1 : 0]]
+                      , '');
+                  },
                 });
               } else {
                 cache[testCode] = true;

tests/lib/rules/no-identical-tests.js

@@ -25,7 +25,7 @@ ruleTester.run('no-identical-tests', rule, {
       new RuleTester().run('foo', bar, {
         valid: [
           { code: 'foo' },
-          { code: 'bar' }
+          { code: 'bar' },
         ],
         invalid: []
       });
@@ -38,12 +38,20 @@ ruleTester.run('no-identical-tests', rule, {
         new RuleTester().run('foo', bar, {
           valid: [
             { code: 'foo' },
-            { code: 'foo' }
+            { code: 'foo' },
           ],
           invalid: []
         });
       `,
       errors: [ERROR],
+      output: `
+        new RuleTester().run('foo', bar, {
+          valid: [
+            { code: 'foo' },
+          ],
+          invalid: []
+        });
+      `,
     },
     {
       code: `
@@ -59,6 +67,16 @@ ruleTester.run('no-identical-tests', rule, {
         });
       `,
       errors: [ERROR, ERROR],
+      output: `
+        new RuleTester().run('foo', bar, {
+          valid: [
+            { code: 'foo' },
+          ],
+          invalid: [
+            { code: 'foo', errors: ['bar'] },
+          ]
+        });
+      `,
     },
   ],
 });