fix(expect-expect): include numbers when matching assert function nam…

…es with wildcards (#1134)
jest-community · May 29, 2022 · 2d0ef11 · 2d0ef11
1 parent a19bcf4
commit 2d0ef11
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/rules/__tests__/expect-expect.test.ts b/src/rules/__tests__/expect-expect.test.ts
@@ -187,6 +187,14 @@ ruleTester.run('expect-expect', rule, {
 
 ruleTester.run('wildcards', rule, {
   valid: [
+    {
+      code: "test('should pass *', () => expect404ToBeLoaded());",
+      options: [{ assertFunctionNames: ['expect*'] }],
+    },
+    {
+      code: "test('should pass *', () => expect.toHaveStatus404());",
+      options: [{ assertFunctionNames: ['expect.**'] }],
+    },
     {
       code: "test('should pass', () => tester.foo().expect(123));",
       options: [{ assertFunctionNames: ['tester.*.expect'] }],

diff --git a/src/rules/expect-expect.ts b/src/rules/expect-expect.ts
@@ -28,9 +28,9 @@ function matchesAssertFunctionName(
       `^${p
         .split('.')
         .map(x => {
-          if (x === '**') return '[a-z\\.]*';
+          if (x === '**') return '[a-z\\d\\.]*';
 
-          return x.replace(/\*/gu, '[a-z]*');
+          return x.replace(/\*/gu, '[a-z\\d]*');
         })
         .join('\\.')}(\\.|$)`,
       'ui',