ssr: Test cases for let statement trailing semi insertion/omission

Author: MarijnS95

crates/ssr/src/tests.rs

@@ -198,12 +198,11 @@ fn ssr_block_expr_match() {
 
 #[test]
 fn ssr_let_stmt_replace() {
-    // FIXME: See remark above about trailing semicolons
-    // assert_ssr_transform(
-    //     "let $a = $b ==>> let $a = 11",
-    //     "fn main() { let x = 10; x }",
-    //     expect![["fn main() { let x = 11; x }"]],
-    // );
+    assert_ssr_transform(
+        "let $a = $b ==>> let $a = 11",
+        "fn main() { let x = 10; x }",
+        expect![["fn main() { let x = 11; x }"]],
+    );
 
     // Pattern without trailing semicolon
     // FIXME: This should fail, the expected value must be `let x = 11;;`
@@ -219,6 +218,30 @@ fn ssr_let_stmt_replace() {
         "fn main() { let x = 10; x }",
         expect![["fn main() { let x = 11; x }"]],
     );
+
+    // Trailing semicolon should be dropped from the new expression
+    assert_ssr_transform(
+        "let $a = $b; ==>> let $a = 11",
+        "fn main() { let x = 10; x }",
+        expect![["fn main() { let x = 11 x }"]],
+    );
+}
+
+#[test]
+fn ssr_let_stmt_replace_expr() {
+    // Trailing semicolon should be dropped from the new expression
+    assert_ssr_transform(
+        "let $a = $b; ==>> $b",
+        "fn main() { let x = 10; }",
+        expect![["fn main() { 10 }"]],
+    );
+
+    // Trailing semicolon should _not_ be dropped from the new expression
+    assert_ssr_transform(
+        "let $a = $b ==>> $b",
+        "fn main() { let x = 10; }",
+        expect![["fn main() { 10; }"]],
+    );
 }
 
 #[test]