chore: add tests with computed expression

rome · Nov 21, 2022 · 5186531 · 5186531
1 parent 03c4570
commit 5186531
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 0 deletions.
diff --git a/crates/rome_js_analyze/tests/specs/nursery/noExtraNonNullAssertion/invalid.ts b/crates/rome_js_analyze/tests/specs/nursery/noExtraNonNullAssertion/invalid.ts
@@ -34,3 +34,7 @@ class Case9 {
 }
 
 case10!!.prop = null;
+
+case11!?.[computedField];
+
+case12!?.[a.b!!];
diff --git a/crates/rome_js_analyze/tests/specs/nursery/noExtraNonNullAssertion/invalid.ts.snap b/crates/rome_js_analyze/tests/specs/nursery/noExtraNonNullAssertion/invalid.ts.snap
@@ -42,6 +42,10 @@ class Case9 {
 
 case10!!.prop = null;
 
+case11!?.[computedField];
+
+case12!?.[a.b!!];
+
 ```
 
 # Diagnostics
@@ -237,6 +241,7 @@ invalid.ts:36:1 lint/nursery/noExtraNonNullAssertion  FIXABLE  ━━━━━
   > 36 │ case10!!.prop = null;
        │ ^^^^^^^
     37 │ 
+    38 │ case11!?.[computedField];
 
   i Safe fix: Remove extra non-null assertion.
 
@@ -245,4 +250,22 @@ invalid.ts:36:1 lint/nursery/noExtraNonNullAssertion  FIXABLE  ━━━━━
 
 ```
 
+```
+invalid.ts:40:11 lint/nursery/noExtraNonNullAssertion  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ! Forbidden extra non-null assertion.
+
+    38 │ case11!?.[computedField];
+    39 │ 
+  > 40 │ case12!?.[a.b!!];
+       │           ^^^^
+    41 │ 
+
+  i Safe fix: Remove extra non-null assertion.
+
+    40 │ case12!?.[a.b!!];
+       │               -  
+
+```
+