fix: addressed feedback and made necessary corrections

jest-community · Apr 29, 2024 · b030cc7 · b030cc7
1 parent 2c43334
commit b030cc7
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 25 deletions.
diff --git a/src/rules/__tests__/unbound-method.test.ts b/src/rules/__tests__/unbound-method.test.ts
@@ -113,37 +113,33 @@ const invalidTestCases: Array<TSESLint.InvalidTestCase<MessageIds, Options>> = [
     ],
   },
   // toThrow matchers call the expected value (which is expected to be a function)
-  ...toThrowMatchers.map<TSESLint.InvalidTestCase<MessageIds, Options>>(
-    matcher => ({
-      code: dedent`
+  ...toThrowMatchers.map(matcher => ({
+    code: dedent`
       ${ConsoleClassAndVariableCode}
 
       expect(console.log).${matcher}();
     `,
-      errors: [
-        {
-          line: 9,
-          messageId: 'unboundWithoutThisAnnotation',
-        },
-      ],
-    }),
-  ),
+    errors: [
+      {
+        line: 9,
+        messageId: 'unboundWithoutThisAnnotation' as const,
+      },
+    ],
+  })),
   // toThrow matchers call the expected value (which is expected to be a function)
-  ...toThrowMatchers.map<TSESLint.InvalidTestCase<MessageIds, Options>>(
-    matcher => ({
-      code: dedent`
+  ...toThrowMatchers.map(matcher => ({
+    code: dedent`
       ${ConsoleClassAndVariableCode}
 
       expect(console.log).not.${matcher}();
     `,
-      errors: [
-        {
-          line: 9,
-          messageId: 'unboundWithoutThisAnnotation',
-        },
-      ],
-    }),
-  ),
+    errors: [
+      {
+        line: 9,
+        messageId: 'unboundWithoutThisAnnotation' as const,
+      },
+    ],
+  })),
 ];
 
 const requireRule = (throwWhenRequiring: boolean) => {

diff --git a/src/rules/valid-expect-in-promise.ts b/src/rules/valid-expect-in-promise.ts
@@ -98,7 +98,7 @@ const isPromiseMethodThatUsesValue = (
   ) {
     const nodeName = getNodeName(node.argument);
 
-    if (nodeName && ['Promise.all', 'Promise.allSettled'].includes(nodeName)) {
+    if (['Promise.all', 'Promise.allSettled'].includes(nodeName as string)) {
       const [firstArg] = node.argument.arguments;
 
       if (
@@ -110,8 +110,7 @@ const isPromiseMethodThatUsesValue = (
     }
 
     if (
-      nodeName &&
-      ['Promise.resolve', 'Promise.reject'].includes(nodeName) &&
+      ['Promise.resolve', 'Promise.reject'].includes(nodeName as string) &&
       node.argument.arguments.length === 1
     ) {
       return isIdentifier(node.argument.arguments[0], name);