Fix: incorrect no-deprecated-report-api autofix if > 5 arguments passed

Author: not-an-aardvark

lib/rules/no-deprecated-report-api.js

@@ -47,13 +47,13 @@ module.exports = {
             fix (fixer) {
               const openingParen = sourceCode.getTokenBefore(node.arguments[0]);
               const closingParen = sourceCode.getLastToken(node);
-              const keyNames = node.arguments.length === 5
+              const keys = node.arguments.length >= 5
                 ? ['node', 'loc', 'message', 'data', 'fix']
                 : ['node', 'message', 'data', 'fix'];
 
               return fixer.replaceTextRange(
                 [openingParen.range[1], closingParen.range[0]],
-                '{' + node.arguments.map((arg, index) => `${keyNames[index]}: ${sourceCode.getText(arg)}`).join(', ') + '}'
+                `{${node.arguments.slice(0, 5).map((arg, index) => `${keys[index]}: ${sourceCode.getText(arg)}`).join(', ')}}`
               );
             },
           });

tests/lib/rules/no-deprecated-report-api.js

@@ -214,5 +214,22 @@ ruleTester.run('no-deprecated-report-api', rule, {
       `,
       errors: [ERROR],
     },
+    {
+      code: `
+        module.exports = {
+          create(context) {
+            context.report(theNode, theLocation, theMessage, theData, theFix, somethingElse, somethingElse, somethingElse);
+          }
+        };
+      `,
+      output: `
+        module.exports = {
+          create(context) {
+            context.report({node: theNode, loc: theLocation, message: theMessage, data: theData, fix: theFix});
+          }
+        };
+      `,
+      errors: [ERROR],
+    },
   ],
 });