Skip to content

Commit b01bda8

Browse files
committedJun 13, 2011
[fix test] Update to ensure optimist is aware of default booleans. Associated tests included
1 parent aa753e7 commit b01bda8

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed
 

‎index.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,7 @@ function Argv (args, cwd) {
303303
function parseArgs () {
304304
var argv = { _ : [], $0 : self.$0 };
305305
Object.keys(flags.bools).forEach(function (key) {
306-
setArg(key, false);
306+
setArg(key, defaults[key] || false);
307307
});
308308

309309
function setArg (key, val) {

‎test/parse.js

+22
Original file line numberDiff line numberDiff line change
@@ -228,3 +228,25 @@ exports.multiAlias = function () {
228228
assert.equal(argv.z, argv.zm);
229229
assert.equal(argv.f, 11);
230230
};
231+
232+
exports['boolean default true'] = function () {
233+
var argv = optimist.options({
234+
sometrue: {
235+
boolean: true,
236+
default: true
237+
}
238+
}).argv;
239+
240+
assert.equal(argv.sometrue, true);
241+
};
242+
243+
exports['boolean default false'] = function () {
244+
var argv = optimist.options({
245+
somefalse: {
246+
boolean: true,
247+
default: false
248+
}
249+
}).argv;
250+
251+
assert.equal(argv.somefalse, false);
252+
};

0 commit comments

Comments
 (0)
Please sign in to comment.