From c822b48d4f7eedf4db803d158cfe7bcde24c25ea Mon Sep 17 00:00:00 2001
From: leaysgur <6259812+leaysgur@users.noreply.github.com>
Date: Thu, 10 Oct 2024 05:00:45 +0000
Subject: [PATCH] fix(regular_expression): Fix CharacterClass negative codegen
 (#6415)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Part of #6413 , fixes these mismatch.

```
  × Regular Expression content mismatch for `/[^]a/m`: `[]a` == `[]a`
  × Regular Expression content mismatch for `/a[^]/`: `a[]` == `a[]`
  × Regular Expression content mismatch for `/[^]/`: `[]` == `[]`
  × Regular Expression content mismatch for `/[^]/`: `[]` == `[]`
```
---
 .../oxc_regular_expression/src/ast_impl/display.rs |  7 +++----
 tasks/coverage/snapshots/parser_test262.snap       | 13 +------------
 tasks/coverage/snapshots/semantic_test262.snap     | 14 ++------------
 3 files changed, 6 insertions(+), 28 deletions(-)

diff --git a/crates/oxc_regular_expression/src/ast_impl/display.rs b/crates/oxc_regular_expression/src/ast_impl/display.rs
index 4e90b689de726..774edae814840 100644
--- a/crates/oxc_regular_expression/src/ast_impl/display.rs
+++ b/crates/oxc_regular_expression/src/ast_impl/display.rs
@@ -181,12 +181,11 @@ impl<'a> Display for CharacterClass<'a> {
         }
 
         write!(f, "[")?;
+        if self.negative {
+            write!(f, "^")?;
+        }
 
         if !self.body.is_empty() {
-            if self.negative {
-                write!(f, "^")?;
-            }
-
             let sep = match self.kind {
                 CharacterClassContentsKind::Union => "",
                 CharacterClassContentsKind::Subtraction => "--",
diff --git a/tasks/coverage/snapshots/parser_test262.snap b/tasks/coverage/snapshots/parser_test262.snap
index ff608d77a2e8b..9b2d57a1f1abb 100644
--- a/tasks/coverage/snapshots/parser_test262.snap
+++ b/tasks/coverage/snapshots/parser_test262.snap
@@ -2,23 +2,12 @@ commit: d62fa93c
 
 parser_test262 Summary:
 AST Parsed     : 43776/43776 (100.00%)
-Positive Passed: 43772/43776 (99.99%)
+Positive Passed: 43775/43776 (100.00%)
 Negative Passed: 4237/4239 (99.95%)
 Expect Syntax Error: tasks/coverage/test262/test/language/import/import-attributes/json-invalid.js
 Expect Syntax Error: tasks/coverage/test262/test/language/import/import-attributes/json-named-bindings.js
-Expect to Parse: tasks/coverage/test262/test/built-ins/RegExp/S15.10.2.13_A2_T1.js
-
-  × Regular Expression content mismatch for `/[^]a/m`: `[]a` == `[]a`
-Expect to Parse: tasks/coverage/test262/test/built-ins/RegExp/S15.10.2.13_A2_T2.js
-
-  × Regular Expression content mismatch for `/a[^]/`: `a[]` == `a[]`
-Expect to Parse: tasks/coverage/test262/test/built-ins/RegExp/S15.10.2.13_A2_T8.js
-
-  × Regular Expression content mismatch for `/[^]/`: `[]` == `[]`
 Expect to Parse: tasks/coverage/test262/test/built-ins/String/prototype/split/separator-regexp.js
 
-  × Regular Expression content mismatch for `/[^]/`: `[]` == `[]`
-
   × Regular Expression content mismatch for `/\cY/`: `\c` == `\c`
 
   × '0'-prefixed octal literals and octal escape sequences are deprecated
diff --git a/tasks/coverage/snapshots/semantic_test262.snap b/tasks/coverage/snapshots/semantic_test262.snap
index abb913f1d97d0..39d3e10bb0563 100644
--- a/tasks/coverage/snapshots/semantic_test262.snap
+++ b/tasks/coverage/snapshots/semantic_test262.snap
@@ -2,7 +2,7 @@ commit: d62fa93c
 
 semantic_test262 Summary:
 AST Parsed     : 43776/43776 (100.00%)
-Positive Passed: 43572/43776 (99.53%)
+Positive Passed: 43575/43776 (99.54%)
 tasks/coverage/test262/test/annexB/language/function-code/if-decl-else-decl-a-func-block-scoping.js
 semantic error: Symbol scope ID mismatch for "f":
 after transform: SymbolId(3): ScopeId(4294967294)
@@ -1119,18 +1119,8 @@ semantic error: Symbol scope ID mismatch for "f":
 after transform: SymbolId(0): ScopeId(4294967294)
 rebuilt        : SymbolId(0): ScopeId(4294967294)
 
-tasks/coverage/test262/test/built-ins/RegExp/S15.10.2.13_A2_T1.js
-semantic error: Regular Expression content mismatch for `/[^]a/m`: `[]a` == `[]a`
-
-tasks/coverage/test262/test/built-ins/RegExp/S15.10.2.13_A2_T2.js
-semantic error: Regular Expression content mismatch for `/a[^]/`: `a[]` == `a[]`
-
-tasks/coverage/test262/test/built-ins/RegExp/S15.10.2.13_A2_T8.js
-semantic error: Regular Expression content mismatch for `/[^]/`: `[]` == `[]`
-
 tasks/coverage/test262/test/built-ins/String/prototype/split/separator-regexp.js
-semantic error: Regular Expression content mismatch for `/[^]/`: `[]` == `[]`
-Regular Expression content mismatch for `/\cY/`: `\c` == `\c`
+semantic error: Regular Expression content mismatch for `/\cY/`: `\c` == `\c`
 
 tasks/coverage/test262/test/language/module-code/eval-rqstd-once.js
 semantic error: Bindings mismatch: