Account for template literal with no expression

Author: jackyho112

lib/rules/no-unused-state.js

@@ -25,10 +25,14 @@ function uncast(node) {
 // method definitions, ObjectExpression property keys).
 function getName(node) {
   node = uncast(node);
-  if (node.type === 'Identifier') {
+  const type = node.type;
+
+  if (type === 'Identifier') {
     return node.name;
-  } else if (node.type === 'Literal') {
+  } else if (type === 'Literal') {
     return String(node.value);
+  } else if (type === 'TemplateLiteral' && node.expressions.length === 0) {
+    return node.quasis[0].value.raw;
   }
   return null;
 }
@@ -97,6 +101,7 @@ module.exports = {
         if (
           prop.type === 'Property' &&
           (key.type === 'Literal' ||
+          (key.type === 'TemplateLiteral' && key.expressions.length === 0) ||
           (prop.computed === false && key.type === 'Identifier')) &&
           getName(prop.key) !== null
         ) {

tests/lib/rules/no-unused-state.js

@@ -102,6 +102,14 @@ eslintTester.run('no-unused-state', rule, {
         return <SomeComponent />;
       }
     });`,
+    `var ComputedKeyFromTemplateLiteralTest = createReactClass({
+      getInitialState: function() {
+        return { [\`foo\`]: 0 };
+      },
+      render: function() {
+        return <SomeComponent foo={this.state['foo']} />;
+      }
+    });`,
     `var GetInitialStateMethodTest = createReactClass({
       getInitialState() {
         return { foo: 0 };
@@ -198,6 +206,14 @@ eslintTester.run('no-unused-state', rule, {
         return <SomeComponent />;
       }
     }`,
+    `class ComputedKeyFromTemplateLiteralTest extends React.Component {
+      constructor() {
+        this.state = { [\`foo\`]: 0 };
+      }
+      render() {
+        return <SomeComponent foo={this.state.foo} />;
+      }
+    }`,
     `class SetStateTest extends React.Component {
         onFooChange(newFoo) {
           this.setState({ foo: newFoo });
@@ -407,6 +423,17 @@ eslintTester.run('no-unused-state', rule, {
         })`,
       errors: getErrorMessages(['foo'])
     },
+    {
+      code: `var UnusedComputedTemplateLiteralKeyStateTest = createReactClass({
+          getInitialState: function() {
+            return { [\`foo\`]: 0 };
+          },
+          render: function() {
+            return <SomeComponent />;
+          }
+        })`,
+      errors: getErrorMessages(['foo'])
+    },
     {
       code: `var UnusedComputedNumberLiteralKeyStateTest = createReactClass({
           getInitialState: function() {
@@ -493,6 +520,26 @@ eslintTester.run('no-unused-state', rule, {
       errors: getErrorMessages(['foo']),
       parser: 'babel-eslint'
     },
+    {
+      code: `class UnusedComputedTemplateLiteralKeyStateTest extends React.Component {
+          state = { [\`foo\`]: 0 };
+          render() {
+            return <SomeComponent />;
+          }
+        }`,
+      errors: getErrorMessages(['foo']),
+      parser: 'babel-eslint'
+    },
+    {
+      code: `class UnusedComputedTemplateLiteralKeyStateTest extends React.Component {
+          state = { [\`foo \\n bar\`]: 0 };
+          render() {
+            return <SomeComponent />;
+          }
+        }`,
+      errors: getErrorMessages(['foo \\n bar']),
+      parser: 'babel-eslint'
+    },
     {
       code: `class UnusedComputedBooleanLiteralKeyStateTest extends React.Component {
           state = { [true]: 0 };