JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

make tests compatible with Node.js 12 (#3304)

Browse Source 
In Node.js 12, the formatting of console arguments will change slightly.
Previously, a string other than the first argument was formatted using
single quotes if the first argument was non-string. Now, quotes are
never added regardless of position of a string argument.

To make test compatible in all Node.js versions, I work around by
ensuring the first argument to console.log is a string which prevents
the quotes from being added on older versions of Node.js.

Ref: https://github.com/nodejs/node/pull/23162
This commit is contained in:
silverwind
2019-03-12 21:55:04 +01:00
committed by Alex Lam S.L
parent 008c236137
commit 9aae4f2424
2 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
test/compress/evaluate.js
View File

@@ -1245,12 +1245,12 @@ self_comparison_1: {
} }
input: { input: {
var o = { n: NaN }; var o = { n: NaN };
console.log(o.n == o.n, o.n === o.n, o.n != o.n, o.n !== o.n, typeof o.n); console.log(typeof o.n, o.n == o.n, o.n === o.n, o.n != o.n, o.n !== o.n);
} }
expect: { expect: {
console.log(false, false, true, true, "number"); console.log("number", false, false, true, true);
} }
expect_stdout: "false false true true 'number'" expect_stdout: "number false false true true"
} }
self_comparison_2: { self_comparison_2: {
@@ -1265,12 +1265,12 @@ self_comparison_2: {
} }
input: { input: {
var o = { n: NaN }; var o = { n: NaN };
console.log(o.n == o.n, o.n === o.n, o.n != o.n, o.n !== o.n, typeof o.n); console.log(typeof o.n, o.n == o.n, o.n === o.n, o.n != o.n, o.n !== o.n);
} }
expect: { expect: {
console.log(false, false, true, true, "number"); console.log("number", false, false, true, true);
} }
expect_stdout: "false false true true 'number'" expect_stdout: "number false false true true"
} }
issue_2535_1: { issue_2535_1: {

6
test/compress/reduce_vars.js
View File

@@ -2740,18 +2740,18 @@ issue_1814_2: {
!function() { !function() {
var b = a + 1; var b = a + 1;
!function(a) { !function(a) {
console.log(a++, b); console.log(b, a++);
}(0); }(0);
}(); }();
} }
expect: { expect: {
!function() { !function() {
!function(a) { !function(a) {
console.log(a++, "321"); console.log("321", a++);
}(0); }(0);
}(); }();
} }
expect_stdout: "0 '321'" expect_stdout: "321 0"
} }
try_abort: { try_abort: {