clarify no-module-extensions message

Author: acdvs

lib/rules/no-module-extensions.js

@@ -21,9 +21,9 @@ module.exports = {
   meta: {
     type: 'problem',
     messages: {
-      invalidModuleExtension: 'Module name should not include file extension \'{{ module }}\''
+      invalidModuleExtension: 'Module name should not include file extension'
     },
-    fixable: true
+    fixable: 'code'
   },
 
   create: function(context) {
@@ -39,9 +39,6 @@ module.exports = {
           context.report({
             node: invalidModuleNode,
             messageId: 'invalidModuleExtension',
-            data: {
-              module: invalidModuleNode.value
-            },
             fix: function(fixer) {
               const fixedModuleName = invalidModuleNode.raw.replace(INVALID_MODULE_REGEX, '');
               return fixer.replaceText(invalidModuleNode, fixedModuleName);

tests/rules/no-module-extensions.js

@@ -41,22 +41,22 @@ ruleTester.run('no-module-extensions', rule, {
   invalid: [
     {
       code: 'define(["./lib.js"], function(lib) {});',
-      errors: [{ messageId: 'invalidModuleExtension', data: { module: './lib.js' }}],
+      errors: [{ messageId: 'invalidModuleExtension' }],
       output: 'define(["./lib"], function(lib) {});'
     },
     {
       code: 'define([\'./lib.js\'], function(lib) {});',
-      errors: [{ messageId: 'invalidModuleExtension', data: { module: './lib.js' }}],
+      errors: [{ messageId: 'invalidModuleExtension' }],
       output: 'define([\'./lib\'], function(lib) {});'
     },
     {
       code: 'define(["./lib1", "./lib2.js"], function(lib1, lib2) {});',
-      errors: [{ messageId: 'invalidModuleExtension', data: { module: './lib2.js' }}],
+      errors: [{ messageId: 'invalidModuleExtension' }],
       output: 'define(["./lib1", "./lib2"], function(lib1, lib2) {});'
     },
     {
       code: 'define(["./lib1", "./lib2.js"], (lib1, lib2) => {});',
-      errors: [{ messageId: 'invalidModuleExtension', data: { module: './lib2.js' }}],
+      errors: [{ messageId: 'invalidModuleExtension' }],
       output: 'define(["./lib1", "./lib2"], (lib1, lib2) => {});'
     }
   ]