proposal: optional chaining is conditionality and should violate no-conditional-in-test

[jroru]

This code currently violates no-conditional-in-test

describe("suite", () => {
  it("clicking button does something", () => {
    const button: Button | null = findButton();
    if (button) {
      button.click();
    }
    expect(someCallback).toHaveBeenCalled();
  });
});

Whereas this code does not:

describe("suite", () => {
  it("clicking button does something", () => {
    const button: Button | null = findButton();
    button?.click();
    expect(someCallback).toHaveBeenCalled();
  });
});

Happy to help on a PR if there is consensus.

[veritem]

i agree, a pr is welcome!