Catch error when creating new TXP

js/controllers/send.js

@@ -117,6 +117,14 @@ angular.module('copayApp.controllers').controller('SendController',
       var w = $rootScope.wallet;
 
       function done(err, ntxid, merchantData) {
+        if (err) {
+          var message = 'The transaction' + (w.isShared() ? ' proposal' : '') + ' could not be created';
+          notification.error('Error', message);
+          $scope.loading = false;
+          $scope.loadTxs();
+          return;
+        }
+
         // If user is granted the privilege of choosing
         // their own amount, add it to the tx.
         if (merchantData && +merchantData.total === 0) {

js/models/core/Wallet.js

@@ -2045,7 +2045,7 @@ Wallet.prototype.createTx = function(toAddress, amountSatStr, comment, opts, cb)
   }
 
   this.getUnspent(function(err, safeUnspent) {
-    if (err) return cb(err);
+    if (err) return cb(new Error('Could not get list of UTXOs'));
 
     var ntxid = self.createTxSync(toAddress, amountSatStr, comment, safeUnspent, opts);
     if (!ntxid) {

test/mocks/FakeWallet.js

@@ -46,7 +46,7 @@ var FakeWallet = function() {
 
 FakeWallet.prototype.createTx = function(toAddress, amountSatStr, comment, opts, cb) {
   var callback = cb || opts;
-  callback({});
+  callback(null, {});
 }
 
 FakeWallet.prototype.sendTx = function(ntxid, cb) {

test/test.Wallet.js

@@ -792,6 +792,19 @@ describe('Wallet model', function() {
     });
   });
 
+  describe('#createTx', function () {
+    it('should fail if insight server is down', function (done) {
+      var w = cachedCreateW2();
+      var utxo = createUTXO(w);
+      w.blockchain.fixUnspent(utxo);
+      sinon.stub(w, 'getUnspent').yields('error', null);
+      w.createTx(toAddress, amountSatStr, null, function(err, ntxid) { 
+        chai.expect(err.message).to.equal('Could not get list of UTXOs');
+        done(); 
+      });
+    });
+  });
+
   describe('#createTxSync', function() {
     it('should fail if amount below min value', function() {
       var w = cachedCreateW2();

test/unit/controllers/controllersSpec.js

@@ -285,6 +285,18 @@ describe("Unit: Controllers", function() {
       sinon.assert.callCount(scope.loadTxs, 1);
     });
 
+    it('should not send txp when there is an error at creation', function() {
+      sendForm.address.$setViewValue('mkfTyEk7tfgV611Z4ESwDDSZwhsZdbMpVy');
+      sendForm.amount.$setViewValue(1000);
+      scope.wallet.totalCopayers = scope.wallet.requiredCopayers = 1;
+      sinon.stub(scope.wallet, 'createTx').yields('error');
+      var spySendTx = sinon.spy(scope.wallet, 'sendTx');
+      scope.loadTxs = sinon.spy();
+
+      scope.submitForm(sendForm);
+      sinon.assert.callCount(spySendTx, 0);
+      sinon.assert.callCount(scope.loadTxs, 1);
+    });
   });
 
   describe("Unit: Version Controller", function() {