From 1abf1dc90d86f91d9196b1d3fe0dcf9476c04e2a Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Tue, 24 Jun 2014 12:57:15 -0300
Subject: [PATCH 1/2] small refactor on rootScope

---
 js/controllers/addresses.js     |  17 ++--
 js/models/core/PublicKeyRing.js | 134 +++++++++++++++++---------------
 js/services/controllerUtils.js  |  52 ++++++++++---
 js/services/socket.js           |  14 +++-
 4 files changed, 130 insertions(+), 87 deletions(-)

diff --git a/js/controllers/addresses.js b/js/controllers/addresses.js
index 8e9e0a213..3a0f7bacd 100644
--- a/js/controllers/addresses.js
+++ b/js/controllers/addresses.js
@@ -12,27 +12,27 @@ angular.module('copayApp.controllers').controller('AddressesController',
           controllerUtils.setSocketHandlers();
           controllerUtils.updateAddressList();
           $scope.loading = false;
-        },1);
+        }, 1);
       });
     };
 
-    $scope.selectAddress = function (addr) {
+    $scope.selectAddress = function(addr) {
       $scope.selectedAddr = addr;
     };
 
     $rootScope.$watch('addrInfos', function() {
       $scope.addressList();
-    });   
+    });
 
-    $scope.addressList = function () {
+    $scope.addressList = function() {
       $scope.addresses = [];
       var addrInfos = $rootScope.addrInfos;
       if (addrInfos) {
-        for(var i=0;i<addrInfos.length;i++) {
+        for (var i = 0; i < addrInfos.length; i++) {
           var addrinfo = addrInfos[i];
-          $scope.addresses.push({ 
-            'address' : addrinfo.address.toString(),
-            'balance' : $rootScope.balanceByAddr ? $rootScope.balanceByAddr[addrinfo.address.toString()] : 0,
+          $scope.addresses.push({
+            'address': addrinfo.addressStr,
+            'balance': $rootScope.balanceByAddr ? $rootScope.balanceByAddr[addrinfo.addressStr] : 0,
             'isChange': addrinfo.isChange
           });
         }
@@ -41,5 +41,4 @@ angular.module('copayApp.controllers').controller('AddressesController',
         $rootScope.receivedFund = null;
       }
     }
-
   });
diff --git a/js/models/core/PublicKeyRing.js b/js/models/core/PublicKeyRing.js
index 90f1a199e..92db93e53 100644
--- a/js/models/core/PublicKeyRing.js
+++ b/js/models/core/PublicKeyRing.js
@@ -1,24 +1,23 @@
-
 'use strict';
 
 
-var imports     = require('soop').imports();
+var imports = require('soop').imports();
 var preconditions = require('preconditions').instance();
-var bitcore     = require('bitcore');
-var HK          = bitcore.HierarchicalKey;
-var PrivateKey  = require('./PrivateKey');
-var Structure   = require('./Structure');
-var AddressIndex= require('./AddressIndex');
-var Address     = bitcore.Address;
-var Script      = bitcore.Script;
+var bitcore = require('bitcore');
+var HK = bitcore.HierarchicalKey;
+var PrivateKey = require('./PrivateKey');
+var Structure = require('./Structure');
+var AddressIndex = require('./AddressIndex');
+var Address = bitcore.Address;
+var Script = bitcore.Script;
 
 function PublicKeyRing(opts) {
   opts = opts || {};
 
   this.walletId = opts.walletId;
 
-  this.network = opts.networkName === 'livenet' ? 
-      bitcore.networks.livenet : bitcore.networks.testnet;
+  this.network = opts.networkName === 'livenet' ?
+    bitcore.networks.livenet : bitcore.networks.testnet;
 
   this.requiredCopayers = opts.requiredCopayers || 3;
   this.totalCopayers = opts.totalCopayers || 5;
@@ -33,11 +32,11 @@ function PublicKeyRing(opts) {
   this.addressToPath = {};
 }
 
-PublicKeyRing.fromObj = function (data) {
+PublicKeyRing.fromObj = function(data) {
   if (data instanceof PublicKeyRing) {
     throw new Error('bad data format: Did you use .toObj()?');
   }
-  var ret =  new PublicKeyRing(data);
+  var ret = new PublicKeyRing(data);
 
   for (var k in data.copayersExtPubKeys) {
     ret.addCopayer(data.copayersExtPubKeys[k]);
@@ -54,8 +53,8 @@ PublicKeyRing.prototype.toObj = function() {
     totalCopayers: this.totalCopayers,
     indexes: this.indexes.toObj(),
 
-    copayersExtPubKeys: this.copayersHK.map( function (b) { 
-      return b.extendedPublicKeyString(); 
+    copayersExtPubKeys: this.copayersHK.map(function(b) {
+      return b.extendedPublicKeyString();
     }),
     nicknameFor: this.nicknameFor,
     publicKeysCache: this.publicKeysCache
@@ -67,11 +66,11 @@ PublicKeyRing.prototype.getCopayerId = function(i) {
   return this.copayerIds[i];
 };
 
-PublicKeyRing.prototype.registeredCopayers = function () {
+PublicKeyRing.prototype.registeredCopayers = function() {
   return this.copayersHK.length;
 };
 
-PublicKeyRing.prototype.isComplete = function () {
+PublicKeyRing.prototype.isComplete = function() {
   return this.registeredCopayers() === this.totalCopayers;
 };
 
@@ -86,21 +85,23 @@ PublicKeyRing.prototype.myCopayerId = function(i) {
 PublicKeyRing.prototype._checkKeys = function() {
 
   if (!this.isComplete())
-      throw new Error('dont have required keys yet');
+    throw new Error('dont have required keys yet');
 };
 
-PublicKeyRing.prototype._newExtendedPublicKey = function () {
-  return new PrivateKey({networkName: this.network.name})
+PublicKeyRing.prototype._newExtendedPublicKey = function() {
+  return new PrivateKey({
+      networkName: this.network.name
+    })
     .deriveBIP45Branch()
     .extendedPublicKeyString();
 };
 
-PublicKeyRing.prototype._updateBip = function (index) {
+PublicKeyRing.prototype._updateBip = function(index) {
   var hk = this.copayersHK[index].derive(Structure.IdBranch);
-  this.copayerIds[index]= hk.eckey.public.toString('hex');
+  this.copayerIds[index] = hk.eckey.public.toString('hex');
 };
 
-PublicKeyRing.prototype._setNicknameForIndex = function (index, nickname) {
+PublicKeyRing.prototype._setNicknameForIndex = function(index, nickname) {
   this.nicknameFor[this.copayerIds[index]] = nickname;
 };
 
@@ -114,9 +115,9 @@ PublicKeyRing.prototype.nicknameForCopayer = function(copayerId) {
 
 PublicKeyRing.prototype.addCopayer = function(newEpk, nickname) {
   if (this.isComplete())
-      throw new Error('PKR already has all required key:' + this.totalCopayers);
+    throw new Error('PKR already has all required key:' + this.totalCopayers);
 
-  this.copayersHK.forEach(function(b){
+  this.copayersHK.forEach(function(b) {
     if (b.extendedPublicKeyString() === newEpk)
       throw new Error('PKR already has that key');
   });
@@ -129,7 +130,7 @@ PublicKeyRing.prototype.addCopayer = function(newEpk, nickname) {
   var bip = new HK(newEpk);
   this.copayersHK.push(bip);
   this._updateBip(i);
-  if (nickname) { 
+  if (nickname) {
     this._setNicknameForIndex(i, nickname);
   }
   return newEpk;
@@ -138,19 +139,22 @@ PublicKeyRing.prototype.addCopayer = function(newEpk, nickname) {
 PublicKeyRing.prototype.getPubKeys = function(index, isChange) {
   this._checkKeys();
 
-  var path = Structure.Branch(index, isChange); 
+  var path = Structure.Branch(index, isChange);
   var pubKeys = this.publicKeysCache[path];
   if (!pubKeys) {
     pubKeys = [];
     var l = this.copayersHK.length;
-    for(var i=0; i<l; i++) {
+    for (var i = 0; i < l; i++) {
       var hk = this.copayersHK[i].derive(path);
       pubKeys[i] = hk.eckey.public;
     }
-    this.publicKeysCache[path] = pubKeys.map(function(pk){return pk.toString('hex');});
-  } 
-  else {
-    pubKeys = pubKeys.map(function(s){return new Buffer(s,'hex');}); 
+    this.publicKeysCache[path] = pubKeys.map(function(pk) {
+      return pk.toString('hex');
+    });
+  } else {
+    pubKeys = pubKeys.map(function(s) {
+      return new Buffer(s, 'hex');
+    });
   }
 
 
@@ -158,15 +162,15 @@ PublicKeyRing.prototype.getPubKeys = function(index, isChange) {
 };
 
 // TODO this could be cached
-PublicKeyRing.prototype.getRedeemScript = function (index, isChange) {
+PublicKeyRing.prototype.getRedeemScript = function(index, isChange) {
   var pubKeys = this.getPubKeys(index, isChange);
-  var script  = Script.createMultisig(this.requiredCopayers, pubKeys);
+  var script = Script.createMultisig(this.requiredCopayers, pubKeys);
   return script;
 };
 
 // TODO this could be cached
-PublicKeyRing.prototype.getAddress = function (index, isChange) {
-  var script  = this.getRedeemScript(index,isChange);
+PublicKeyRing.prototype.getAddress = function(index, isChange) {
+  var script = this.getRedeemScript(index, isChange);
   var address = Address.fromScript(script, this.network.name);
   this.addressToPath[address.toString()] = Structure.FullBranch(index, isChange);
   return address;
@@ -174,13 +178,13 @@ PublicKeyRing.prototype.getAddress = function (index, isChange) {
 
 PublicKeyRing.prototype.pathForAddress = function(address) {
   var path = this.addressToPath[address];
-  if (!path) throw new Error('Couldn\'t find path for address '+address);
+  if (!path) throw new Error('Couldn\'t find path for address ' + address);
   return path;
 };
 
 // TODO this could be cached
-PublicKeyRing.prototype.getScriptPubKeyHex = function (index, isChange) {
-  var addr  = this.getAddress(index,isChange);
+PublicKeyRing.prototype.getScriptPubKeyHex = function(index, isChange) {
+  var addr = this.getAddress(index, isChange);
   return Script.createP2SH(addr.payload()).getBuffer().toString('hex');
 };
 
@@ -204,18 +208,22 @@ PublicKeyRing.prototype.getAddressesInfo = function(opts) {
 
   var ret = [];
   if (!opts.excludeChange) {
-    for (var i=0; i<this.indexes.getChangeIndex(); i++) {
+    for (var i = 0; i < this.indexes.getChangeIndex(); i++) {
+      var a = this.getAddress(i, true);
       ret.unshift({
-        address: this.getAddress(i,true),
+        address: this.getAddress(i, true),
+        addressStr: a.toString(),
         isChange: true
       });
     }
   }
 
   if (!opts.excludeMain) {
-    for (var i=0; i<this.indexes.getReceiveIndex(); i++) {
+    for (var i = 0; i < this.indexes.getReceiveIndex(); i++) {
+      var a = this.getAddress(i, false);
       ret.unshift({
-        address: this.getAddress(i,false),
+        address: a,
+        addressStr: a.toString(),
         isChange: false
       });
     }
@@ -225,59 +233,59 @@ PublicKeyRing.prototype.getAddressesInfo = function(opts) {
 };
 
 // TODO this could be cached
-PublicKeyRing.prototype._addScriptMap = function (map, index, isChange) {
-  var script  = this.getRedeemScript(index,isChange);
+PublicKeyRing.prototype._addScriptMap = function(map, index, isChange) {
+  var script = this.getRedeemScript(index, isChange);
   map[Address.fromScript(script, this.network.name).toString()] = script.getBuffer().toString('hex');
 };
 
-PublicKeyRing.prototype.getRedeemScriptMap = function () {
+PublicKeyRing.prototype.getRedeemScriptMap = function() {
   var ret = {};
 
-  for (var i=0; i<this.indexes.getChangeIndex(); i++) {
-    this._addScriptMap(ret,i,true);
+  for (var i = 0; i < this.indexes.getChangeIndex(); i++) {
+    this._addScriptMap(ret, i, true);
   }
-  for (var i=0; i<this.indexes.getReceiveIndex(); i++) {
-    this._addScriptMap(ret,i,false);
+  for (var i = 0; i < this.indexes.getReceiveIndex(); i++) {
+    this._addScriptMap(ret, i, false);
   }
   return ret;
 };
 
 PublicKeyRing.prototype._checkInPKR = function(inPKR, ignoreId) {
 
-  if (!ignoreId  && this.walletId !== inPKR.walletId) {
+  if (!ignoreId && this.walletId !== inPKR.walletId) {
     throw new Error('inPKR walletId mismatch');
   }
 
   if (this.network.name !== inPKR.network.name) {
-    throw new Error('Network mismatch. Should be '+this.network.name +
-        ' and found '+inPKR.network.name);
+    throw new Error('Network mismatch. Should be ' + this.network.name +
+      ' and found ' + inPKR.network.name);
   }
 
   if (
     this.requiredCopayers && inPKR.requiredCopayers &&
     (this.requiredCopayers !== inPKR.requiredCopayers))
-    throw new Error('inPKR requiredCopayers mismatch '+this.requiredCopayers+'!='+inPKR.requiredCopayers);
+    throw new Error('inPKR requiredCopayers mismatch ' + this.requiredCopayers + '!=' + inPKR.requiredCopayers);
 
   if (
     this.totalCopayers && inPKR.totalCopayers &&
     (this.totalCopayers !== inPKR.totalCopayers))
-    throw new Error('inPKR totalCopayers mismatch'+this.totalCopayers+'!='+inPKR.requiredCopayers);
+    throw new Error('inPKR totalCopayers mismatch' + this.totalCopayers + '!=' + inPKR.requiredCopayers);
 };
 
 
 PublicKeyRing.prototype._mergePubkeys = function(inPKR) {
   var self = this;
   var hasChanged = false;
-  var l= self.copayersHK.length;
-  if (self.isComplete()) 
+  var l = self.copayersHK.length;
+  if (self.isComplete())
     return;
 
-  inPKR.copayersHK.forEach( function(b) {
+  inPKR.copayersHK.forEach(function(b) {
     var haveIt = false;
-    var epk = b.extendedPublicKeyString(); 
-    for(var j=0; j<l; j++) {
+    var epk = b.extendedPublicKeyString();
+    for (var j = 0; j < l; j++) {
       if (self.copayersHK[j].extendedPublicKeyString() === epk) {
-        haveIt=true;
+        haveIt = true;
         break;
       }
     }
@@ -289,8 +297,8 @@ PublicKeyRing.prototype._mergePubkeys = function(inPKR) {
       self.copayersHK.push(new HK(epk));
       self._updateBip(l2);
       if (inPKR.nicknameFor[self.getCopayerId(l2)])
-        self._setNicknameForIndex(l2,inPKR.nicknameFor[self.getCopayerId(l2)]);
-      hasChanged=true;
+        self._setNicknameForIndex(l2, inPKR.nicknameFor[self.getCopayerId(l2)]);
+      hasChanged = true;
     }
   });
   return hasChanged;
diff --git a/js/services/controllerUtils.js b/js/services/controllerUtils.js
index 4cfc731f5..19774c43d 100644
--- a/js/services/controllerUtils.js
+++ b/js/services/controllerUtils.js
@@ -14,6 +14,8 @@ angular.module('copayApp.services')
     };
 
     root.logout = function() {
+      Socket.removeAllListeners();
+
       $rootScope.wallet = null;
       delete $rootScope['wallet'];
       video.close();
@@ -70,10 +72,35 @@ angular.module('copayApp.services')
       });
     };
 
+    root.setupRootVariables = function() {
+      $rootScope.unitName = config.unitName;
+      $rootScope.txAlertCount = 0;
+      $rootScope.insightError = 0;
+      $rootScope.isCollapsed = true;
+      $rootScope.$watch('insightError', function(status) {
+        if (status === -1) {
+          $rootScope.$flashMessage = {
+            type: 'success',
+            message: 'Networking Restored :)',
+          };
+          $rootScope.insightError = 0;
+        }
+      });
+
+      $rootScope.$watch('txAlertCount', function(txAlertCount) {
+        if (txAlertCount && txAlertCount > 0) {
+          notification.info('New Transaction', ($rootScope.txAlertCount == 1) ? 'You have a pending transaction proposal' : 'You have ' + $rootScope.txAlertCount + ' pending transaction proposals', txAlertCount);
+        }
+      });
+    };
+
 
     root.startNetwork = function(w, $scope) {
+      Socket.removeAllListeners();
 
+      root.setupRootVariables();
       root.installStartupHandlers(w, $scope);
+      root.setSocketHandlers();
 
       var handlePeerVideo = function(err, peerID, url) {
         if (err) {
@@ -103,8 +130,8 @@ angular.module('copayApp.services')
       });
 
       w.on('publicKeyRingUpdated', function(dontDigest) {
-        root.setSocketHandlers();
         root.updateAddressList();
+        root.setSocketHandlers();
         if (!dontDigest) {
           $rootScope.$digest();
         }
@@ -122,7 +149,7 @@ angular.module('copayApp.services')
           });
         }, 3000);
       });
-      w.on('txProposalEvent', function(e){
+      w.on('txProposalEvent', function(e) {
         switch (e.type) {
           case 'signed':
             var user = w.publicKeyRing.nicknameForCopayer(e.cId);
@@ -273,20 +300,23 @@ angular.module('copayApp.services')
       if (!$rootScope.wallet) return;
 
       var currentAddrs = Socket.getListeners();
-      var addrs = $rootScope.wallet.getAddressesStr();
+      var allAddrs = $rootScope.addrInfos;
 
       var newAddrs = [];
-      for (var i in addrs) {
-        var a = addrs[i];
-        if (!currentAddrs[a])
+      for (var i in allAddrs) {
+        var a = allAddrs[i];
+        if (!currentAddrs[a.addressStr])
           newAddrs.push(a);
       }
       for (var i = 0; i < newAddrs.length; i++) {
-        Socket.emit('subscribe', newAddrs[i]);
+        Socket.emit('subscribe', newAddrs[i].addressStr);
       }
-      newAddrs.forEach(function(addr) {
-        Socket.on(addr, function(txid) {
-          $rootScope.receivedFund = [txid, addr];
+      newAddrs.forEach(function(a) {
+        Socket.on(a.addressStr, function(txid) {
+
+          if (!a.isChange)
+            notification.funds('Received fund', a.addressStr);
+
           root.updateBalance(function() {
             $rootScope.$digest();
           });
@@ -297,7 +327,7 @@ angular.module('copayApp.services')
         Socket.emit('subscribe', 'inv');
         Socket.on('block', function(block) {
           root.updateBalance(function() {
-              $rootScope.$digest();
+            $rootScope.$digest();
           });
         });
       }
diff --git a/js/services/socket.js b/js/services/socket.js
index 10f843b6f..99dce34b4 100644
--- a/js/services/socket.js
+++ b/js/services/socket.js
@@ -38,8 +38,12 @@ angular.module('copayApp.services').factory('Socket',
         var ret = {};
 
         var addrList = listeners
-          .filter(function(i) { return i.event != 'block'; })
-          .map(function(i) {return i.event;});
+          .filter(function(i) {
+            return i.event != 'block';
+          })
+          .map(function(i) {
+            return i.event;
+          });
 
         for (var i in addrList) {
           ret[addrList[i]] = 1;
@@ -47,7 +51,9 @@ angular.module('copayApp.services').factory('Socket',
         return ret;
       },
       isListeningBlocks: function() {
-        return listeners.filter(function(i) { return i.event == 'block'; }).length > 0;
+        return listeners.filter(function(i) {
+          return i.event == 'block';
+        }).length > 0;
       },
       emit: function(event, data, callback) {
         socket.emit(event, data, function() {
@@ -62,7 +68,7 @@ angular.module('copayApp.services').factory('Socket',
       removeAllListeners: function() {
         for (var i = 0; i < listeners.length; i++) {
           var details = listeners[i];
-          socket.removeListener(details.event, details.fn);
+          socket.removeAllListeners(details.event);
         }
 
         listeners = [];

From 6b7b00769d7b57752c69d64f495a55d15c5fa41c Mon Sep 17 00:00:00 2001
From: Matias Alejo Garcia <ematiu@gmail.com>
Date: Tue, 24 Jun 2014 13:18:13 -0300
Subject: [PATCH 2/2] add tests

---
 test/test.PublicKeyRing.js         |   6 +-
 test/unit/services/servicesSpec.js | 117 +++++++++++++++--------------
 2 files changed, 66 insertions(+), 57 deletions(-)

diff --git a/test/test.PublicKeyRing.js b/test/test.PublicKeyRing.js
index ea9811443..36d84ee15 100644
--- a/test/test.PublicKeyRing.js
+++ b/test/test.PublicKeyRing.js
@@ -147,11 +147,13 @@ describe('PublicKeyRing model', function() {
       for (var i = 0; i < 2; i++)
         w.generateAddress(isChange);
 
-    var as = w.getAddresses();
+    var as = w.getAddressesInfo();
     as.length.should.equal(4);
     for (var j in as) {
       var a = as[j];
-      a.isValid().should.equal(true);
+      a.address.isValid().should.equal(true);
+      a.addressStr.should.equal(a.address.toString());
+      a.isChange.should.equal([false, false, true, true][j]);
     }
   });
 
diff --git a/test/unit/services/servicesSpec.js b/test/unit/services/servicesSpec.js
index 4bb1e959b..eae710e15 100644
--- a/test/unit/services/servicesSpec.js
+++ b/test/unit/services/servicesSpec.js
@@ -70,65 +70,72 @@ describe("Unit: Walletfactory Service", function() {
 });
 
 describe("Unit: controllerUtils", function() {
-  beforeEach(angular.mock.module('copayApp.services'));
+    beforeEach(angular.mock.module('copayApp.services'));
 
-  it('should updateBalance in bits', inject(function(controllerUtils, $rootScope) {
-    expect(controllerUtils.updateBalance).not.to.equal(null);
-    scope = $rootScope.$new();
+    it('should updateBalance in bits', inject(function(controllerUtils, $rootScope) {
+        expect(controllerUtils.updateBalance).not.to.equal(null);
+        scope = $rootScope.$new();
 
-    $rootScope.wallet = new FakeWallet();
-    var addr = '1CjPR7Z5ZSyWk6WtXvSFgkptmpoi4UM9BC';
-    var a = {};
-    a[addr] = 100;
-    //SATs 
-    $rootScope.wallet.set(100000001, 90000002, a);
+        $rootScope.wallet = new FakeWallet();
+        var addr = '1CjPR7Z5ZSyWk6WtXvSFgkptmpoi4UM9BC';
+        var a = {};
+        a[addr] = 100;
+        //SATs 
+        $rootScope.wallet.set(100000001, 90000002, a);
 
-    //retuns values in DEFAULT UNIT(bits)
-    controllerUtils.updateBalance(function() {
-      expect($rootScope.totalBalanceBTC).to.be.equal('1.0000');
-      expect($rootScope.availableBalanceBTC).to.be.equal('0.9000');
-      expect($rootScope.totalBalance).to.be.equal(1000000.01);
-      expect($rootScope.availableBalance).to.be.equal(900000.02);
-      expect($rootScope.addrInfos).not.to.equal(null);
-      expect($rootScope.addrInfos[0].address).to.equal(addr);
+        //retuns values in DEFAULT UNIT(bits)
+        controllerUtils.updateBalance(function() {
+          expect($rootScope.totalBalanceBTC).to.be.equal('1.0000');
+          expect($rootScope.availableBalanceBTC).to.be.equal('0.9000');
+          expect($rootScope.totalBalance).to.be.equal(1000000.01);
+          expect($rootScope.availableBalance).to.be.equal(900000.02);
+          expect($rootScope.addrInfos).not.to.equal(null);
+          expect($rootScope.addrInfos[0].address).to.equal(addr);
+        });
+      }));
+
+      it('should set the rootScope', inject(function(controllerUtils, $rootScope) {
+        controllerUtils.setupRootVariables(function() {
+          expect($rootScope.txAlertCount).to.be.equal(0);
+          expect($rootScope.insightError).to.be.equal(0);
+          expect($rootScope.isCollapsed).to.be.equal(0);
+          expect($rootScope.unitName).to.be.equal('bits');
+        });
+      }));
     });
-  }));
 
+  describe("Unit: Notification Service", function() {
+    beforeEach(angular.mock.module('copayApp.services'));
+    it('should contain a notification service', inject(function(notification) {
+      expect(notification).not.to.equal(null);
+    }));
+  });
 
-});
+  describe("Unit: Backup Service", function() {
+    beforeEach(angular.mock.module('copayApp.services'));
+    it('should contain a backup service', inject(function(backupService) {
+      expect(backupService).not.to.equal(null);
+    }));
+    it('should backup in file', inject(function(backupService) {
+      var mock = sinon.mock(window);
+      var expectation = mock.expects('saveAs');
+      backupService.download(new FakeWallet());
+      expectation.once();
+    }));
+  });
 
-describe("Unit: Notification Service", function() {
-  beforeEach(angular.mock.module('copayApp.services'));
-  it('should contain a notification service', inject(function(notification) {
-    expect(notification).not.to.equal(null);
-  }));
-});
-
-describe("Unit: Backup Service", function() {
-  beforeEach(angular.mock.module('copayApp.services'));
-  it('should contain a backup service', inject(function(backupService) {
-    expect(backupService).not.to.equal(null);
-  }));
-  it('should backup in file', inject(function(backupService) {
-    var mock = sinon.mock(window);
-    var expectation = mock.expects('saveAs');
-    backupService.download(new FakeWallet());
-    expectation.once();
-  }));
-});
-
-describe("Unit: isMobile Service", function() {
-  beforeEach(angular.mock.module('copayApp.services'));
-  it('should contain a isMobile service', inject(function(isMobile) {
-    expect(isMobile).not.to.equal(null);
-  }));
-  it('should not detect mobile by default', inject(function(isMobile) {
-    isMobile.any().should.equal(false);
-  }));
-  it('should detect mobile if user agent is Android', inject(function(isMobile) {
-    navigator.__defineGetter__('userAgent', function() {
-      return 'Android 2.2.3';
-    });
-    isMobile.any().should.equal(true);
-  }));
-});
+  describe("Unit: isMobile Service", function() {
+    beforeEach(angular.mock.module('copayApp.services'));
+    it('should contain a isMobile service', inject(function(isMobile) {
+      expect(isMobile).not.to.equal(null);
+    }));
+    it('should not detect mobile by default', inject(function(isMobile) {
+      isMobile.any().should.equal(false);
+    }));
+    it('should detect mobile if user agent is Android', inject(function(isMobile) {
+      navigator.__defineGetter__('userAgent', function() {
+        return 'Android 2.2.3';
+      });
+      isMobile.any().should.equal(true);
+    }));
+  });