test(to-have-length): add optional chaining case

[hekystyle]

Hi,

I tried to implement optional chaning operator (?.) for to-have-length rule.

[G-Rath]

thanks but this isn't quite equivalent since toHaveLength doesn't support undefined - in the test cases you've added this will result in a matcher error which is still arguably ok because the test fails like you'd expect, but it causes an incorrect result when combined with not i.e.

it('is not the expected length', () => {
  // this will pass even if x is undefined
  expect(x?.length).not.toBe(1);

  // this will fail if x is undefined
  expect(x).not.toHaveLength(1);
});

Given that I don't think it's worth supporting optional chaining here

[hekystyle]

Oh, I see. At least let me add valid test case for expect(users[0]?.permissions?.length).toBe(1);.

[github-actions]

🎉 This PR is included in version 27.2.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀