more tests fixes

js/models/core/WalletFactory.js

@@ -91,6 +91,8 @@ WalletFactory.prototype.fromObj = function(obj, skipFields) {
   preconditions.checkArgument(obj);
 
 
+  // not stored options
+  obj.opts = obj.opts || {};
   obj.opts.reconnectDelay = this.walletDefaults.reconnectDelay;
 
   // this is only used if private key or public key ring is skipped
@@ -231,7 +233,6 @@ WalletFactory.prototype.create = function(opts, cb) {
   opts.totalCopayers = totalCopayers;
   opts.version = opts.version || this.version;
 
-console.log('[WalletFactory.js.165]'); //TODO
   var w = new Wallet(opts);
   var self = this;
   w.store(function() {

test/mocks/FakeStorage.js

@@ -20,6 +20,19 @@ FakeStorage.prototype.getGlobal = function(id, cb) {
   return cb(this.storage[id]);
 };
 
+FakeStorage.prototype.getMany = function(wid, fields, cb) {
+  var self= this;
+  var ret = [];
+  for(var ii in fields){
+    var k = fields[ii];
+    ret[k] = this.storage[wid + '::' + k];
+  }
+
+  return cb(ret);
+};
+
+
+
 FakeStorage.prototype.setLastOpened = function(val, cb) {
   this.storage['lastOpened'] = val;
   return cb();

test/test.WalletFactory.js

@@ -85,6 +85,7 @@ function assertObjectEqual(a, b, msg) {
 
 
 describe('WalletFactory model', function() {
+
   var config = {
     Network: FakeNetwork,
     Blockchain: FakeBlockchain,
@@ -121,42 +122,46 @@ describe('WalletFactory model', function() {
   it('should be able to create wallets', function(done) {
     var wf = new WalletFactory(config, '0.0.1');
     wf.create(null, function(err, w) {
-
-console.log('[test.WalletFactory.js.123]'); //TODO
-      should.not.exist(err);
       should.exist(w);
-      w.should.be.instanceof('WalletFactory');
+      should.not.exist(err);
       done();
     });
   });
 
-  it('should be able to create wallets with given pk', function() {
+  it('should be able to create wallets with given pk', function(done) {
     var wf = new WalletFactory(config, '0.0.1');
     var priv = 'tprv8ZgxMBicQKsPdEqHcA7RjJTayxA3gSSqeRTttS1JjVbgmNDZdSk9EHZK5pc52GY5xFmwcakmUeKWUDzGoMLGAhrfr5b3MovMUZUTPqisL2m';
-    var w = wf.create({
+    wf.create({
       privateKeyHex: priv,
-    });
+    }, function(err, w) {
       w.privateKey.toObj().extendedPrivateKeyString.should.equal(priv);
+      should.not.exist(err);
+      done();
+    });
   });
 
-  it('should be able to create wallets with random pk', function() {
+  it('should be able to create wallets with random pk', function(done) {
     var wf = new WalletFactory(config, '0.0.1');
-    var priv = 'tprv8ZgxMBicQKsPdEqHcA7RjJTayxA3gSSqeRTttS1JjVbgmNDZdSk9EHZK5pc52GY5xFmwcakmUeKWUDzGoMLGAhrfr5b3MovMUZUTPqisL2m';
+    wf.create(null, function(err, w1) {
-    var w1 = wf.create();
+      wf.create(null, function(err, w2) {
-    var w2 = wf.create();
         w1.privateKey.toObj().extendedPrivateKeyString.should.not.equal(
           w2.privateKey.toObj().extendedPrivateKeyString
         );
+        done();
+      });
+    });
   });
 
 
-  it('should be able to get wallets', function() {
+  it.only('should be able to get wallets', function(done) {
     var wf = new WalletFactory(config, '0.0.1');
-    var w = wf.create();
+    wf.create(null, function(err,w){
-
+      wf.read(w.id, [], function(err, w2){
-    var w2 = wf.read(w.id);
         should.exist(w2);
         w2.id.should.equal(w.id);
+        done();
+      });
+    });
   });