fix corner case in --reduce-test (#3985)
This commit is contained in:
parent
1dbf7d4a3a
commit
ed69adedcd
5
test/input/reduce/diff_error.js
Normal file
5
test/input/reduce/diff_error.js
Normal file
|
|
@ -0,0 +1,5 @@
|
||||||
|
(function f(a) {
|
||||||
|
do {
|
||||||
|
console.log(f.length);
|
||||||
|
} while (console.log(f += 0));
|
||||||
|
})();
|
||||||
19
test/input/reduce/diff_error.reduced.js
Normal file
19
test/input/reduce/diff_error.reduced.js
Normal file
|
|
@ -0,0 +1,19 @@
|
||||||
|
// (beautified)
|
||||||
|
(function f(a) {
|
||||||
|
do {
|
||||||
|
console.log(f.length);
|
||||||
|
} while (console.log(f += 0));
|
||||||
|
})();
|
||||||
|
// output: 1
|
||||||
|
// function(){}0
|
||||||
|
//
|
||||||
|
// minify: 0
|
||||||
|
// function(){}0
|
||||||
|
//
|
||||||
|
// options: {
|
||||||
|
// "compress": {
|
||||||
|
// "keep_fargs": false,
|
||||||
|
// "unsafe": true
|
||||||
|
// },
|
||||||
|
// "mangle": false
|
||||||
|
// }
|
||||||
|
|
@ -296,4 +296,18 @@ describe("test/reduce.js", function() {
|
||||||
"// }",
|
"// }",
|
||||||
]).join("\n"));
|
]).join("\n"));
|
||||||
});
|
});
|
||||||
|
it("Should handle corner cases when intermediate case differs only in Error.message", function() {
|
||||||
|
if (semver.satisfies(process.version, "<=0.10")) return;
|
||||||
|
var result = reduce_test(read("test/input/reduce/diff_error.js"), {
|
||||||
|
compress: {
|
||||||
|
keep_fargs: false,
|
||||||
|
unsafe: true,
|
||||||
|
},
|
||||||
|
mangle: false,
|
||||||
|
}, {
|
||||||
|
verbose: false,
|
||||||
|
});
|
||||||
|
if (result.error) throw result.error;
|
||||||
|
assert.strictEqual(result.code, read("test/input/reduce/diff_error.reduced.js"));
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
|
||||||
|
|
@ -438,7 +438,8 @@ module.exports = function reduce_test(testcase, minify_options, reduce_options)
|
||||||
}
|
}
|
||||||
}));
|
}));
|
||||||
var code = testcase_ast.print_to_string();
|
var code = testcase_ast.print_to_string();
|
||||||
if (diff = test_for_diff(code, minify_options, result_cache, max_timeout)) {
|
var diff = test_for_diff(code, minify_options, result_cache, max_timeout);
|
||||||
|
if (diff && !diff.timed_out && !diff.error) {
|
||||||
testcase = code;
|
testcase = code;
|
||||||
differs = diff;
|
differs = diff;
|
||||||
} else {
|
} else {
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user