fix: forbid_const_assign_when_lowering_using test (#1413)

rolldown · Jun 19, 2024 · 0d15f0e · 0d15f0e
1 parent 949c819
commit 0d15f0e
Show file tree

Hide file tree

Showing 5 changed files with 60 additions and 19 deletions.
diff --git a/...es/rolldown/tests/esbuild/default/.forbid_const_assign_when_lowering_using/artifacts.snap b/...es/rolldown/tests/esbuild/default/.forbid_const_assign_when_lowering_using/artifacts.snap
diff --git a/...sign_when_lowering_using/test.config.json → ...t_assign_when_lowering_using/_config.json b/...sign_when_lowering_using/test.config.json → ...t_assign_when_lowering_using/_config.json
@@ -1,5 +1,6 @@
 {
-  "input": {
+  "expectExecuted": false,
+  "config": {
     "input": [
       {
         "name": "entry_js",

diff --git a/crates/rolldown/tests/esbuild/default/forbid_const_assign_when_lowering_using/artifacts.snap b/crates/rolldown/tests/esbuild/default/forbid_const_assign_when_lowering_using/artifacts.snap
@@ -0,0 +1,53 @@
+---
+source: crates/rolldown/tests/common/case.rs
+assertion_line: 171
+expression: content
+input_file: crates/rolldown/tests/esbuild/default/forbid_const_assign_when_lowering_using
+---
+# warnings
+
+## ILLEGAL_REASSIGNMENT
+
+```text
+[ILLEGAL_REASSIGNMENT] Warning: Unexpected re-assignment of const variable `x`
+   ╭─[entry.js:3:1]
+   │
+ 1 │ const x = 1
+   │       ┬  
+   │       ╰── `x` is declared here as const
+   │ 
+ 3 │ x = 3
+   │ ┬  
+   │ ╰── `x` is re-assigned here
+───╯
+
+```
+## ILLEGAL_REASSIGNMENT
+
+```text
+[ILLEGAL_REASSIGNMENT] Warning: Unexpected re-assignment of const variable `y`
+   ╭─[entry.js:7:2]
+   │
+ 5 │     const y = 1
+   │           ┬  
+   │           ╰── `y` is declared here as const
+   │ 
+ 7 │     y = 3
+   │     ┬  
+   │     ╰── `y` is re-assigned here
+───╯
+
+```
+# Assets
+
+## entry_js.mjs
+
+```js
+
+//#region entry.js
+const x = 1;
+using x2 = 2;
+x = 3;
+
+//#endregion
+```
diff --git a/...const_assign_when_lowering_using/entry.js → ...const_assign_when_lowering_using/entry.js b/...const_assign_when_lowering_using/entry.js → ...const_assign_when_lowering_using/entry.js
diff --git a/crates/rolldown/tests/snapshots/fixtures__filename_with_hash.snap b/crates/rolldown/tests/snapshots/fixtures__filename_with_hash.snap
@@ -1,5 +1,6 @@
 ---
 source: crates/rolldown/tests/fixtures.rs
+assertion_line: 54
 expression: "snapshot_outputs.join(\"\\n\")"
 ---
 # tests/esbuild/dce/base64_loader_remove_unused
@@ -237,6 +238,10 @@ expression: "snapshot_outputs.join(\"\\n\")"
 
 - entry_js-!~{000}~.mjs => entry_js-Xnvegfxd.mjs
 
+# tests/esbuild/default/forbid_const_assign_when_lowering_using
+
+- entry_js-!~{000}~.mjs => entry_js-idsVRs5M.mjs
+
 # tests/esbuild/default/import_fs_node_common_js
 
 - entry_js-!~{000}~.cjs => entry_js-aITEk-gU.cjs