Add Symbol to builtins
This commit is contained in:
parent
f63803e3e3
commit
df00507fb3
|
|
@ -47,7 +47,7 @@ function find_builtins() {
|
||||||
var a = [];
|
var a = [];
|
||||||
[ Object, Array, Function, Number,
|
[ Object, Array, Function, Number,
|
||||||
String, Boolean, Error, Math,
|
String, Boolean, Error, Math,
|
||||||
Date, RegExp
|
Date, RegExp, Symbol
|
||||||
].forEach(function(ctor){
|
].forEach(function(ctor){
|
||||||
Object.getOwnPropertyNames(ctor).map(add);
|
Object.getOwnPropertyNames(ctor).map(add);
|
||||||
if (ctor.prototype) {
|
if (ctor.prototype) {
|
||||||
|
|
|
||||||
26
test/mocha/builtins.js
Normal file
26
test/mocha/builtins.js
Normal file
|
|
@ -0,0 +1,26 @@
|
||||||
|
var UglifyJS = require("../../");
|
||||||
|
var assert = require("assert");
|
||||||
|
|
||||||
|
describe("builtins", function() {
|
||||||
|
it ("Should not mangle builtins", function() {
|
||||||
|
var test = "function foo(something){\n" +
|
||||||
|
" return [Object,Array,Function,Number,String,Boolean,Error,Math,Date,RegExp,Symbol,something];\n" +
|
||||||
|
"};";
|
||||||
|
|
||||||
|
var result = UglifyJS.minify(test, {fromString: true, parse: {bare_returns: true}}).code;
|
||||||
|
|
||||||
|
assert.strictEqual(result.indexOf("something"), -1);
|
||||||
|
|
||||||
|
assert.notEqual(result.indexOf("Object"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Array"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Function"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Number"), -1);
|
||||||
|
assert.notEqual(result.indexOf("String"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Boolean"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Error"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Math"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Date"), -1);
|
||||||
|
assert.notEqual(result.indexOf("RegExp"), -1);
|
||||||
|
assert.notEqual(result.indexOf("Symbol"), -1);
|
||||||
|
});
|
||||||
|
});
|
||||||
Loading…
Reference in New Issue
Block a user