fix(valid-title): ignore string addition

jest-community · Oct 28, 2019 · 5337caa · 5337caa
1 parent a7b91f7
commit 5337caa
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/src/rules/__tests__/valid-title.test.ts b/src/rules/__tests__/valid-title.test.ts
@@ -12,6 +12,7 @@ const ruleTester = new TSESLint.RuleTester({
 ruleTester.run('title-must-be-string', rule, {
   valid: [
     'it("is a string", () => {});',
+    'it("is" + " a " + " string", () => {});',
     'test("is a string", () => {});',
     'xtest("is a string", () => {});',
     'xtest(`${myFunc} is a string`, () => {});',

diff --git a/src/rules/valid-title.ts b/src/rules/valid-title.ts
@@ -16,6 +16,20 @@ import {
 const trimFXprefix = (word: string) =>
   ['f', 'x'].includes(word.charAt(0)) ? word.substr(1) : word;
 
+const doesBinaryExpressionContainStringNode = (
+  binaryExp: TSESTree.BinaryExpression,
+): boolean => {
+  if (isStringNode(binaryExp.right)) {
+    return true;
+  }
+
+  if (binaryExp.left.type === AST_NODE_TYPES.BinaryExpression) {
+    return doesBinaryExpressionContainStringNode(binaryExp.left);
+  }
+
+  return isStringNode(binaryExp.left);
+};
+
 export default createRule({
   name: __filename,
   meta: {
@@ -56,6 +70,13 @@ export default createRule({
         const [argument] = node.arguments;
 
         if (!isStringNode(argument)) {
+          if (
+            argument.type === AST_NODE_TYPES.BinaryExpression &&
+            doesBinaryExpressionContainStringNode(argument)
+          ) {
+            return;
+          }
+
           if (
             argument.type !== AST_NODE_TYPES.TemplateLiteral &&
             !(ignoreTypeOfDescribeName && isDescribe(node))