From f9cdd741c288bee59aa9120b0a5c6f7ca284bd31 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Donny/=EA=B0=95=EB=8F=99=EC=9C=A4?= Date: Tue, 16 May 2023 12:21:08 +0900 Subject: [PATCH] test(es/minifier): Enable more terser tests (#7396) --- crates/swc_ecma_minifier/tests/TODO.txt | 6 ----- crates/swc_ecma_minifier/tests/passing.txt | 6 +++++ .../compress/issue_281/issue_1758/output.js | 9 +++----- .../issue_640/negate_iife_1/output.js | 4 ++-- .../issue_640/negate_iife_3/output.js | 6 ++--- .../recursive_inlining_3/output.js | 23 ++++++++----------- .../compress/typeof/issue_2728_3/output.js | 3 +-- .../compress/typeof/issue_2728_4/output.js | 1 - 8 files changed, 24 insertions(+), 34 deletions(-) diff --git a/crates/swc_ecma_minifier/tests/TODO.txt b/crates/swc_ecma_minifier/tests/TODO.txt index 2ae718e4a8e..ee6a2301b95 100644 --- a/crates/swc_ecma_minifier/tests/TODO.txt +++ b/crates/swc_ecma_minifier/tests/TODO.txt @@ -251,15 +251,12 @@ issue_203/compress_new_function_with_destruct/input.js issue_203/compress_new_function_with_destruct_arrows/input.js issue_22/return_with_no_value_in_if_body/input.js issue_281/collapse_vars_constants/input.js -issue_281/issue_1758/input.js issue_2871/comparison_with_undefined/input.js issue_417/test_unexpected_crash/input.js issue_417/test_unexpected_crash_2/input.js issue_44/issue_44_valid_ast_1/input.js issue_44/issue_44_valid_ast_2/input.js issue_640/conditional/input.js -issue_640/negate_iife_1/input.js -issue_640/negate_iife_3/input.js issue_747/dont_reuse_prop/input.js issue_747/unmangleable_props_should_always_be_reserved/input.js issue_751/negate_booleans_1/input.js @@ -349,7 +346,6 @@ reduce_vars/perf_3/input.js reduce_vars/perf_7/input.js reduce_vars/pure_getters_2/input.js reduce_vars/pure_getters_3/input.js -reduce_vars/recursive_inlining_3/input.js reduce_vars/redefine_farg_1/input.js reduce_vars/redefine_farg_2/input.js reduce_vars/redefine_farg_3/input.js @@ -366,6 +362,4 @@ template_string/regex_2/input.js template_string/side_effects/input.js template_string/template_evaluate_undefined/input.js transform/label_if_break/input.js -typeof/issue_2728_3/input.js -typeof/issue_2728_4/input.js typeof/typeof_defun_1/input.js diff --git a/crates/swc_ecma_minifier/tests/passing.txt b/crates/swc_ecma_minifier/tests/passing.txt index d9f29fc460a..bd5e69ccf69 100644 --- a/crates/swc_ecma_minifier/tests/passing.txt +++ b/crates/swc_ecma_minifier/tests/passing.txt @@ -925,6 +925,7 @@ issue_281/issue_1254_negate_iife_nested/input.js issue_281/issue_1254_negate_iife_true/input.js issue_281/issue_1288_side_effects/input.js issue_281/issue_1595_3/input.js +issue_281/issue_1758/input.js issue_281/keep_fargs/input.js issue_281/modified/input.js issue_281/negate_iife_3/input.js @@ -966,6 +967,8 @@ issue_640/iife/input.js issue_640/issue_1254_negate_iife_nested/input.js issue_640/issue_1254_negate_iife_true/input.js issue_640/limit_1/input.js +issue_640/negate_iife_1/input.js +issue_640/negate_iife_3/input.js issue_640/negate_iife_3_off/input.js issue_640/negate_iife_4/input.js issue_640/negate_iife_5/input.js @@ -1365,6 +1368,7 @@ reduce_vars/perf_5/input.js reduce_vars/pure_getters_1/input.js reduce_vars/recursive_inlining_1/input.js reduce_vars/recursive_inlining_2/input.js +reduce_vars/recursive_inlining_3/input.js reduce_vars/recursive_inlining_4/input.js reduce_vars/recursive_inlining_5/input.js reduce_vars/redefine_arguments_1/input.js @@ -1592,6 +1596,8 @@ typeof/duplicate_lambda_arg_name/input.js typeof/issue_1668/input.js typeof/issue_2728_1/input.js typeof/issue_2728_2/input.js +typeof/issue_2728_3/input.js +typeof/issue_2728_4/input.js typeof/issue_2728_5/input.js typeof/issue_2728_6/input.js typeof/typeof_defun_2/input.js diff --git a/crates/swc_ecma_minifier/tests/terser/compress/issue_281/issue_1758/output.js b/crates/swc_ecma_minifier/tests/terser/compress/issue_281/issue_1758/output.js index dada8aa5c7a..ffd0348d650 100644 --- a/crates/swc_ecma_minifier/tests/terser/compress/issue_281/issue_1758/output.js +++ b/crates/swc_ecma_minifier/tests/terser/compress/issue_281/issue_1758/output.js @@ -1,6 +1,3 @@ -console.log( - (function (c) { - var undefined = 42; - return c--, c--, void c.toString(); - })() -); +console.log(function(c) { + return c--, c--, void c.toString(); +}()); diff --git a/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_1/output.js b/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_1/output.js index 60208bbabfc..cd52dbe41c4 100644 --- a/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_1/output.js +++ b/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_1/output.js @@ -1,3 +1,3 @@ -(function () { +!function() { stuff(); -})(); +}(); diff --git a/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_3/output.js b/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_3/output.js index 17bfa1ce761..72642b3c393 100644 --- a/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_3/output.js +++ b/crates/swc_ecma_minifier/tests/terser/compress/issue_640/negate_iife_3/output.js @@ -1,5 +1,3 @@ -(function () { +!function() { return t; -})() - ? console.log(true) - : console.log(false); +}() ? console.log(false) : console.log(true); diff --git a/crates/swc_ecma_minifier/tests/terser/compress/reduce_vars/recursive_inlining_3/output.js b/crates/swc_ecma_minifier/tests/terser/compress/reduce_vars/recursive_inlining_3/output.js index e9f4ab90dec..f524703ccee 100644 --- a/crates/swc_ecma_minifier/tests/terser/compress/reduce_vars/recursive_inlining_3/output.js +++ b/crates/swc_ecma_minifier/tests/terser/compress/reduce_vars/recursive_inlining_3/output.js @@ -1,15 +1,12 @@ -!(function () { - function qux(x) { +!function() { + (function qux(x) { console.log("qux", x); - if (x) - (function (x) { - console.log("foo", x); - if (x) - (function (x) { - console.log("bar", x); - if (x) qux(x - 1); - })(x - 1); + if (x) (function(x) { + console.log("foo", x); + if (x) (function(x) { + console.log("bar", x); + if (x) qux(x - 1); })(x - 1); - } - qux(4); -})(); + })(x - 1); + })(4); +}(); diff --git a/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_3/output.js b/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_3/output.js index 32d86b95db5..0d5aac40120 100644 --- a/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_3/output.js +++ b/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_3/output.js @@ -1,4 +1,3 @@ -(function () { - function arguments() {} +(function() { console.log("function"); })(); diff --git a/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_4/output.js b/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_4/output.js index ebdaa8ac5a2..2272a540453 100644 --- a/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_4/output.js +++ b/crates/swc_ecma_minifier/tests/terser/compress/typeof/issue_2728_4/output.js @@ -1,2 +1 @@ -function arguments() {} console.log("function");