[New] jsx-no-literals: print node value in warning message

Author: jlgonzalezdev

lib/rules/jsx-no-literals.js

@@ -33,6 +33,7 @@ module.exports = {
 
   create: function(context) {
     const isNoStrings = context.options[0] ? context.options[0].noStrings : false;
+    const sourceCode = context.getSourceCode();
 
     const message = isNoStrings ?
       'Strings not allowed in JSX files' :
@@ -41,7 +42,7 @@ module.exports = {
     function reportLiteralNode(node) {
       context.report({
         node: node,
-        message: message
+        message: `${message}: “${sourceCode.getText(node).trim()}”`
       });
     }
 

tests/lib/rules/jsx-no-literals.js

@@ -201,7 +201,7 @@ ruleTester.run('jsx-no-literals', rule, {
         }
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:test'}]
     }, {
       code: `
         class Comp1 extends Component {
@@ -211,7 +211,7 @@ ruleTester.run('jsx-no-literals', rule, {
         }
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:test'}]
     }, {
       code: `
         class Comp1 extends Component {
@@ -222,7 +222,7 @@ ruleTester.run('jsx-no-literals', rule, {
         }
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:test'}]
     }, {
       code: `
         class Comp1 extends Component {
@@ -233,7 +233,7 @@ ruleTester.run('jsx-no-literals', rule, {
         }
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:test'}]
     }, {
       code: `
         var Hello = createReactClass({
@@ -244,7 +244,7 @@ ruleTester.run('jsx-no-literals', rule, {
         });
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:hello'}]
     }, {
       code: `
         class Comp1 extends Component {
@@ -258,7 +258,7 @@ ruleTester.run('jsx-no-literals', rule, {
         }
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:asdjfl'}]
     }, {
       code: `
         class Comp1 extends Component {
@@ -274,7 +274,7 @@ ruleTester.run('jsx-no-literals', rule, {
         }
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:asdjfl\n                test\n                foo'}]
     }, {
       code: `
         class Comp1 extends Component {
@@ -290,7 +290,7 @@ ruleTester.run('jsx-no-literals', rule, {
         }
       `,
       parser: 'babel-eslint',
-      errors: [{message: 'Missing JSX expression container around literal string'}]
+      errors: [{message: 'Missing JSX expression container around literal string:test'}]
     }, {
       code: `
         <Foo bar="test">
@@ -299,23 +299,23 @@ ruleTester.run('jsx-no-literals', rule, {
       `,
       parser: 'babel-eslint',
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:\'Test\''}]
     }, {
       code: `
         <Foo bar="test">
           {'Test'}
         </Foo>
       `,
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:\'Test\''}]
     }, {
       code: `
         <Foo bar="test">
           {'Test' + name}
         </Foo>
       `,
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:\'Test\''}]
     }, {
       code: `
         <Foo bar="test">
@@ -324,55 +324,55 @@ ruleTester.run('jsx-no-literals', rule, {
       `,
       parser: 'babel-eslint',
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:Test'}]
     }, {
       code: `
         <Foo bar="test">
           Test
         </Foo>
       `,
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:Test'}]
     }, {
       code: `
         <Foo>
           {\`Test\`}
         </Foo>
       `,
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:`Test`'}]
     }, {
       code: '<Foo bar={`Test`} />',
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:`Test`'}]
     }, {
       code: '<Foo bar={`${baz}`} />',
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:`${baz}`'}]
     }, {
       code: '<Foo bar={`Test ${baz}`} />',
       options: [{noStrings: true}],
-      errors: [{message: 'Strings not allowed in JSX files'}]
+      errors: [{message: 'Strings not allowed in JSX files:`Test ${baz}`'}]
     }, {
       code: '<Foo bar={`foo` + \'bar\'} />',
       options: [{noStrings: true}],
       errors: [
-        {message: 'Strings not allowed in JSX files'},
-        {message: 'Strings not allowed in JSX files'}
+        {message: 'Strings not allowed in JSX files:`foo`'},
+        {message: 'Strings not allowed in JSX files:\'bar\''}
       ]
     }, {
       code: '<Foo bar={`foo` + `bar`} />',
       options: [{noStrings: true}],
       errors: [
-        {message: 'Strings not allowed in JSX files'},
-        {message: 'Strings not allowed in JSX files'}
+        {message: 'Strings not allowed in JSX files:`foo`'},
+        {message: 'Strings not allowed in JSX files:`bar`'}
       ]
     }, {
       code: '<Foo bar={\'foo\' + `bar`} />',
       options: [{noStrings: true}],
       errors: [
-        {message: 'Strings not allowed in JSX files'},
-        {message: 'Strings not allowed in JSX files'}
+        {message: 'Strings not allowed in JSX files:\'foo\''},
+        {message: 'Strings not allowed in JSX files:`bar`'}
       ]
     }
   ]