Merge f3e61ed into ac7a43c

veged · May 15, 2018 · 622ef12 · 622ef12
2 parents ac7a43c + f3e61ed
commit 622ef12
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/lib/cmd.js b/lib/cmd.js
@@ -370,7 +370,7 @@ class Cmd extends CoaObject {
 
     _setDefaults(params, desc) {
         for(const item of desc)
-            item._def &&
+            item._def !== undefined &&
                 !params.hasOwnProperty(item._name) &&
                 item._saveVal(params, item._def);
 

diff --git a/test/coa.js b/test/coa.js
@@ -128,14 +128,28 @@ describe('Opt', function() {
                 .short('a')
                 .def('aaa')
                 .end()
+            .opt()
+                .name('b')
+                .short('b')
+                .def(false)
+                .end()
+            .opt()
+                .name('c')
+                .short('c')
+                .def(0)
+                .end()
             .act(function(opts) {
                 return opts;
             });
 
         it('should return default value if not specified', function() {
             return cmd.do()
                 .then(function(opts) {
-                    assert.equal(opts.a, 'aaa');
+                    assert.deepEqual(opts, {
+                        a : 'aaa',
+                        b : false,
+                        c : 0
+                    });
                 });
         });