fix(expect-expert): report on test function identifier rather than body (#337)

* fix(expect-expert): report on test function identifier rather than body

* Prettier

Author: manzoorwanijk

src/rules/expect-expect.test.ts

@@ -5,33 +5,33 @@ runRuleTester('expect-expect', rule, {
   invalid: [
     {
       code: 'test("should fail", () => {});',
-      errors: [{ messageId: 'noAssertions' }],
+      errors: [{ messageId: 'noAssertions', type: 'Identifier' }],
     },
     {
       code: 'test.skip("should fail", () => {});',
-      errors: [{ messageId: 'noAssertions' }],
+      errors: [{ messageId: 'noAssertions', type: 'MemberExpression' }],
     },
     {
       code: javascript`
         test('should fail', async ({ page }) => {
           await assertCustomCondition(page)
         })
       `,
-      errors: [{ messageId: 'noAssertions' }],
+      errors: [{ messageId: 'noAssertions', type: 'Identifier' }],
     },
     {
       code: javascript`
         test('should fail', async ({ page }) => {
           await assertCustomCondition(page)
         })
       `,
-      errors: [{ messageId: 'noAssertions' }],
+      errors: [{ messageId: 'noAssertions', type: 'Identifier' }],
       name: 'Custom assert function',
       options: [{ assertFunctionNames: ['wayComplexCustomCondition'] }],
     },
     {
       code: 'it("should pass", () => hi(true).toBeDefined())',
-      errors: [{ messageId: 'noAssertions' }],
+      errors: [{ messageId: 'noAssertions', type: 'Identifier' }],
       name: 'Global aliases',
       settings: {
         playwright: {

src/rules/expect-expect.ts

@@ -40,7 +40,7 @@ export default createRule({
       },
       'Program:exit'() {
         unchecked.forEach((node) => {
-          context.report({ messageId: 'noAssertions', node })
+          context.report({ messageId: 'noAssertions', node: node.callee })
         })
       },
     }