fix: handle string-based import names when resolving Jest functions (#1761)

Author: G-Rath

src/rules/__tests__/no-hooks.test.ts

@@ -49,6 +49,17 @@ ruleTester.run('no-hooks', rule, {
         { messageId: 'unexpectedHook', data: { hookName: HookName.afterEach } },
       ],
     },
+    {
+      code: dedent`
+        import { 'afterEach' as afterEachTest } from '@jest/globals';
+
+        afterEachTest(() => {})
+      `,
+      parserOptions: { sourceType: 'module', ecmaVersion: 2022 },
+      errors: [
+        { messageId: 'unexpectedHook', data: { hookName: HookName.afterEach } },
+      ],
+    },
     {
       code: 'beforeEach(() => {}); afterEach(() => { jest.resetModules() });',
       options: [{ allow: [HookName.afterEach] }],

src/rules/__tests__/require-top-level-describe.test.ts

@@ -108,6 +108,16 @@ ruleTester.run('require-top-level-describe', rule, {
       parserOptions: { sourceType: 'module' },
       errors: [{ messageId: 'unexpectedHook' }],
     },
+    {
+      code: dedent`
+        import { 'describe' as describe, afterAll as onceEverythingIsDone } from '@jest/globals';
+
+        describe("test suite", () => {});
+        onceEverythingIsDone("my test", () => {})
+      `,
+      parserOptions: { sourceType: 'module', ecmaVersion: 2022 },
+      errors: [{ messageId: 'unexpectedHook' }],
+    },
     {
       code: "it.skip('test', () => {});",
       errors: [{ messageId: 'unexpectedTestCase' }],

src/rules/utils/__tests__/parseJestFnCall.test.ts

@@ -441,7 +441,34 @@ ruleTester.run('esm', rule, {
       parserOptions: { sourceType: 'module', ecmaVersion: 2017 },
     },
   ],
-  invalid: [],
+  invalid: [
+    {
+      code: dedent`
+        import { 'describe' as it } from '@jest/globals';
+
+        it('is a jest function', () => {});
+      `,
+      parserOptions: { sourceType: 'module', ecmaVersion: 2022 },
+      errors: [
+        {
+          messageId: 'details',
+          data: expectedParsedJestFnCallResultData({
+            name: 'describe',
+            type: 'describe',
+            head: {
+              original: 'describe',
+              local: 'it',
+              type: 'import',
+              node: 'it',
+            },
+            members: [],
+          }),
+          column: 1,
+          line: 3,
+        },
+      ],
+    },
+  ],
 });
 
 ruleTester.run('esm (dynamic)', rule, {

src/rules/utils/parseJestFnCall.ts

@@ -452,9 +452,15 @@ const describeImportDefAsImport = (
     return null;
   }
 
+  let imported = def.node.imported.name ?? '';
+
+  if ('value' in def.node.imported) {
+    imported = def.node.imported.value as string;
+  }
+
   return {
     source: def.parent.source.value,
-    imported: def.node.imported.name,
+    imported,
     local: def.node.local.name,
   };
 };