diff --git a/js/models/core/Wallet.js b/js/models/core/Wallet.js index 61e81b137..ecd80db24 100644 --- a/js/models/core/Wallet.js +++ b/js/models/core/Wallet.js @@ -520,7 +520,6 @@ Wallet.prototype._onData = function(senderId, data, ts) { preconditions.checkArgument(data.type); preconditions.checkArgument(ts); preconditions.checkArgument(_.isNumber(ts)); - log.debug('RECV', senderId, data); if (data.type !== 'walletId' && this.id !== data.walletId) { @@ -529,7 +528,6 @@ Wallet.prototype._onData = function(senderId, data, ts) { return; } - switch (data.type) { // This handler is repeaded on WalletFactory (#join). TODO case 'walletId': diff --git a/test/test.Wallet.js b/test/test.Wallet.js index c8ae349e5..7e4090a55 100644 --- a/test/test.Wallet.js +++ b/test/test.Wallet.js @@ -402,6 +402,37 @@ describe('Wallet model', function() { }); + describe('#_onData', function() { + var w = cachedCreateW(); + var sender = '025c046aaf505a6d23203edd343132e9d4d21818b962d1e9a9c98573cc2031bfc9'; + var ts = 1410810974778246; + it('should fail on message unknown', function() { + var data = { + type: "xxx", + walletId: w.id + }; + + (function() { + w._onData(sender, data, ts); + }).should. + throw('unknown message type received: xxx from: 025c046aaf505a6d23203edd343132e9d4d21818b962d1e9a9c98573cc2031bfc9'); + + }); + + it('should call sendWalletReady', function() { + var data = { + type: "walletId", + walletId: w.id + }; + + var spy = sinon.spy(w, 'sendWalletReady'); + w._onData(sender, data, ts); + sinon.assert.callCount(spy, 1); + }); + + }); + + describe('#purgeTxProposals', function() { it('should delete all', function() { var w = cachedCreateW();