JavaScript interface breaks if options object not provided

[mpj]

When using wrapup from JavaScript (i.e. not the command line) one must provide an object (empty object works) to the options method, or one will get an error from Uglify.

Example 1: WORKS!

var wrup = require("wrapup")(),
    fs = require('fs');

wrup.require("chai", "./node_modules/beautiful-lies")
    .options({
      compress: true
    })
    .up(function(err, js){
       fs.writeFile("./__requirements-wrapped-up.js", js)
    });

Example 2: ALSO WORKS!

var wrup = require("wrapup")(),
    fs = require('fs');

wrup.require("chai", "./node_modules/beautiful-lies")
    .options({
      compress: false
    })
    .up(function(err, js){
       fs.writeFile("./__requirements-wrapped-up.js", js)
    });

Example 3: THIS WORKS TOO!!!

var wrup = require("wrapup")(),
    fs = require('fs');

wrup.require("chai", "./node_modules/beautiful-lies")
    .options({
      //compress: false
    })
    .up(function(err, js){
       fs.writeFile("./__requirements-wrapped-up.js", js)
    });

Example 4: BREAKS HORRIBLY!

var wrup = require("wrapup")(),
    fs = require('fs');

wrup.require("chai", "./node_modules/beautiful-lies")
    //.options({
      //compress: false
    //})
    .up(function(err, js){
       fs.writeFile("./__requirements-wrapped-up.js", js)
    });

The error output!

        /Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:110
        name = name.toString();
                    ^
TypeError: Cannot call method 'toString' of undefined
    at make_name (/Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:110:21)
    at Object.OutputStream.print_name (/Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:314:50)
    at /Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:1032:16
    at AST_Node.print (/Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:355:17)
    at /Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:944:25
    at AST_Node.print (/Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:355:17)
    at /Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:961:19
    at AST_Node.print (/Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:355:17)
    at /Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:572:19
    at AST_Node.print (/Users/mpj/code/library/test/node_modules/wrapup/node_modules/uglify-js/lib/output.js:355:17)

​

[arian]

Yes, forgot to set the default options correctly, so when using a named module, it didn't know which global variable it had to use (default is window).