feat(rome_analyze): noNegationElse

[IWANABETHATGUY]

Summary

1. finish noNegationElse rule
2. /~https://github.com/rome/tools/blob/archived-js/website/src/docs/lint/rules/js/noNegationElse.md
3. Resolved 📎 noNegationElse  #2656

Test Plan

1. cargo test

[IWANABETHATGUY]

Found there has some differ from our test case, here is I println result

// valid
if (!true) {consequent;};
true ? consequent : alternate;
// invalid
if (true) {alternate} else {consequent}
!condition ? consequent : alternate;
-----------------------
// valid
if (!true) {consequent;};
true ? consequent : alternate;
// invalid
if (!true) {consequent} else {alternate}
condition ? alternate : consequent;

here is the snapshot result:

Input

// valid
if (!true) {consequent;};
true ? consequent : alternate;
// invalid
if (!true) {consequent} else {alternate}
!condition ? consequent : alternate;

Diagnostics

error[noNegationElse]: Invert blocks when performing a negation test.
  ┌─ noNegationElse.js:5:1
  │
5 │ if (!true) {consequent} else {alternate}
  │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Invert blocks when performing a negation test.

Replace with strict equality

You could see here, the replacement of if (!true) {consequent} else {alternate} is empty,
but the println result is

if (true) {alternate} else {consequent}

error[noNegationElse]: Invert blocks when performing a negation test.
┌─ noNegationElse.js:6:1
│
6 │ !condition ? consequent : alternate;
│ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Invert blocks when performing a negation test.

Replace with strict equality
| @@ -3,4 +3,4 @@
2 2 | true ? consequent : alternate;
3 3 | // invalid
4 4 | if (!true) {consequent} else {alternate}
5 | - !condition ? consequent : alternate;
5 | + condition ? alternate : consequent;

[IWANABETHATGUY]

@ematipico , @leops this pull request is ready for reviewing

[leops]

Once the formatting issues are fixed (just running cargo fmt should take care of it) it should be good to merge

[IWANABETHATGUY]

@leops , format done.