jasmine/no-focused-tests' endLine includes too much of the test

[DamienCassou]

Describe the bug

jasmine/no-focused-tests returns an endLine value that represents the whole test. This makes it particularly inconvenient to code on a fit test. As you can see, everything is underlined:

[
  {
    "filePath": "…",
    "messages": [
      {
        "ruleId": "jasmine/no-focused-tests",
        "severity": 2,
        "message": "Unexpected fit.",
        "line": 2,
        "column": 2,
        "nodeType": "CallExpression",
        "endLine": 9,
        "endColumn": 4
      }
    ],
    "..."
  }
]

To Reproduce

1. Write this code:

   describe("my class", () => {
   	fit("does the work", () => {
   		expect(1 + 1).toBe(2);
   		expect(1 + 2).toBe(3);
   		expect(1 + 3).toBe(4);
   		expect(1 + 4).toBe(5);
   		expect(1 + 5).toBe(6);
   		expect(1 + 6).toBe(7);
   	});
   });

2. Enable jasmine/no-focused-tests

3. Run npx eslint --format=json .

4. The endLine is 9.

Expected behaviour

That endLine is the same as the line, i.e., 2.

The rule mocha/no-exclusive-tests does what I expect by only returning the only part as being wrong:

Context

• Node.js version: 10.17.0
• eslint-plugin-jasmine version: 4.1.0
• eslint version: 6.8.0
• Operating system: Fedora 31

[DamienCassou]

Please see #231 for a fix.

[tlvince]

Closed by 796dddf