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

speed up compress tests (#4737)

Browse Source
This commit is contained in:
Alex Lam S.L
2021-03-05 17:11:05 +00:00
committed by GitHub
parent dd30ed6a9b
commit 2db1a141ab
2 changed files with 22 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
test/compress.js
View File

@@ -10,33 +10,38 @@ var sandbox = require("./sandbox");
var semver = require("semver"); var semver = require("semver");
var U = require("./node"); var U = require("./node");
var file = process.argv[2]; var batch = 50;
var dir = path.resolve(path.dirname(module.filename), "compress"); var dir = path.resolve(path.dirname(module.filename), "compress");
if (file) { if (process.argv.length > 3) {
var file = process.argv[2];
var start = process.argv[3] | 0;
var minify_options = require("./ufuzz/options.json").map(JSON.stringify); var minify_options = require("./ufuzz/options.json").map(JSON.stringify);
log("--- {file}", { file: file });
var tests = parse_test(path.resolve(dir, file)); var tests = parse_test(path.resolve(dir, file));
process.exit(Object.keys(tests).filter(function(name) { process.exit(Object.keys(tests).slice(start, start + batch).filter(function(name) {
return !test_case(tests[name]); return !test_case(tests[name]);
}).length); }).length);
} else { } else {
var files = fs.readdirSync(dir).filter(function(name) { var files = process.argv.length == 3 ? [ process.argv[2] ] : fs.readdirSync(dir).filter(function(name) {
return /\.js$/i.test(name); return /\.js$/i.test(name);
}); });
var failures = 0; var failures = 0;
var failed_files = Object.create(null); var failed_files = Object.create(null);
(function next() { (function next(file, start, length) {
var file = files.shift(); if (start < length) {
if (file) { child_process.spawn(process.argv[0], [ process.argv[1], file, start, batch ], {
child_process.spawn(process.argv[0], [ process.argv[1], file ], {
stdio: [ "ignore", 1, 2 ] stdio: [ "ignore", 1, 2 ]
}).on("exit", function(code) { }).on("exit", function(code) {
if (code) { if (code) {
failures += code; failures += code;
failed_files[file] = code; failed_files[file] = true;
} }
next(); next(file, start + batch, length);
}); });
} else if (file = files.shift()) {
log("--- {file}", { file: file });
start = 0;
length = Object.keys(parse_test(path.resolve(dir, file))).length;
next(file, start, length);
} else if (failures) { } else if (failures) {
console.error(); console.error();
console.error("!!! Failed " + failures + " test case(s)."); console.error("!!! Failed " + failures + " test case(s).");

13
test/reduce.js
View File

@@ -137,6 +137,12 @@ module.exports = function reduce_test(testcase, minify_options, reduce_options)
if (parent instanceof U.AST_ExportDefault) return; if (parent instanceof U.AST_ExportDefault) return;
if (parent instanceof U.AST_ExportForeign) return; if (parent instanceof U.AST_ExportForeign) return;
if (parent instanceof U.AST_ExportReferences) return; if (parent instanceof U.AST_ExportReferences) return;
// preserve sole definition of an export statement
if (node instanceof U.AST_VarDef
&& parent.definitions.length == 1
&& tt.parent(1) instanceof U.AST_ExportDeclaration) {
return;
}
// preserve for (var xxx; ...) // preserve for (var xxx; ...)
if (parent instanceof U.AST_For && parent.init === node && node instanceof U.AST_Definitions) return node; if (parent instanceof U.AST_For && parent.init === node && node instanceof U.AST_Definitions) return node;
// preserve for (xxx in/of ...) // preserve for (xxx in/of ...)
@@ -460,13 +466,6 @@ module.exports = function reduce_test(testcase, minify_options, reduce_options)
return List.skip; return List.skip;
} }
// preserve sole definition of an export statement
if (node instanceof U.AST_VarDef
&& parent.definitions.length == 1
&& tt.parent(1) instanceof U.AST_ExportDeclaration) {
return node;
}
// remove this node unless its the sole element of a (transient) sequence // remove this node unless its the sole element of a (transient) sequence
if (!(parent instanceof U.AST_Sequence) || parent.expressions.length > 1) { if (!(parent instanceof U.AST_Sequence) || parent.expressions.length > 1) {
node.start._permute++; node.start._permute++;