diff --git a/spec/controllerSpec.js b/spec/controllerSpec.js
index 1119628..d5d621a 100644
--- a/spec/controllerSpec.js
+++ b/spec/controllerSpec.js
@@ -23,6 +23,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $scope: {},
                 focaAbmVendedorCobradorService: {},
                 $location: {},
+                $localStorage: {},
                 $filter: {},
                 focaBotoneraLateralService: focaBotoneraLateralService,
                 focaModalService: {}
@@ -41,6 +42,7 @@ describe('Controladores abm vendedor cobrador', function() {
                     $scope: scope,
                     focaAbmVendedorCobradorService: {},
                     $location: $location,
+                    $localStorage: {},
                     $filter: {},
                     focaBotoneraLateralService: focaBotoneraLateralService,
                     focaModalService: {}
@@ -66,6 +68,7 @@ describe('Controladores abm vendedor cobrador', function() {
                     $scope: scope,
                     focaAbmVendedorCobradorService: {},
                     $location: $location,
+                    $localStorage: {},
                     $filter: {},
                     focaBotoneraLateralService: focaBotoneraLateralService,
                     focaModalService: focaModalService
@@ -96,6 +99,7 @@ describe('Controladores abm vendedor cobrador', function() {
                     $scope: scope,
                     focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
                     $location: $location,
+                    $localStorage: {},
                     $filter: {},
                     focaBotoneraLateralService: focaBotoneraLateralService,
                     focaModalService: focaModalService
@@ -126,6 +130,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $scope: scope,
                 focaAbmVendedorCobradorService: {},
                 $location: {},
+                $localStorage: {},
                 $filter: {},
                 focaBotoneraLateralService: focaBotoneraLateralService,
                 focaModalService: {}
@@ -150,6 +155,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $scope: scope,
                 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
                 $location: {},
+                $localStorage: {},
                 $filter: {},
                 focaBotoneraLateralService: focaBotoneraLateralService,
                 focaModalService: {}
@@ -175,6 +181,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $scope: scope,
                 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
                 $location: {},
+                $localStorage: {},
                 $filter: {},
                 focaBotoneraLateralService: focaBotoneraLateralService,
                 focaModalService: {}
@@ -210,7 +217,9 @@ describe('Controladores abm vendedor cobrador', function() {
 
             //act
             var controlador = $controller('focaAbmVendedorCobradorController', {
-                $scope: {},
+                $scope: {
+                    $watch: function() {}
+                },
                 focaAbmVendedorCobradorService: {
                     getCodigoSiguiente: function() {
                         return {
@@ -228,7 +237,10 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
-                focaModalService: {}
+                $localStorage: {},
+                focaModalService: {
+                    
+                }
             });
 
             //assert
@@ -237,7 +249,10 @@ describe('Controladores abm vendedor cobrador', function() {
 
         it('la función $scope.guardar llama a guardaVendedorCobrador del servicio', function() {
             //arrange
-            var scope= {};
+            var scope = {
+                $watch: function() {},
+ 
+            };  
             var focaAbmVendedorCobradorService = {
                 getCodigoSiguiente: function() {
                     return {
@@ -252,14 +267,18 @@ describe('Controladores abm vendedor cobrador', function() {
                 guardarVendedorCobrador: function() { }
             };
             $controller('focaAbmVendedorCobradorController', {
-                $scope: scope,
+                $scope: scope,                
                 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
                 $routeParams: {},
                 focaBotoneraLateralService: {},
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
-                focaModalService: {}
+                $localStorage: {},
+                guardarVendedorCobrador: {},
+                focaModalService: {
+                    alert: function() {}
+                }
             });
             var responseFake = {
                 then: function() {
@@ -268,6 +287,19 @@ describe('Controladores abm vendedor cobrador', function() {
                     };
                 }
             };
+            scope.vendedorCobrador = {
+                NOM: true,
+                domicilio: true,
+                codigoPostal: true,
+                dni: true,
+                rol: true,
+                provincia: {
+                    NOMBRE: true
+                },
+                localidad: {
+                    NOMBRE: true
+                },
+            };
 
             //act
             spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
@@ -278,13 +310,15 @@ describe('Controladores abm vendedor cobrador', function() {
             expect(focaAbmVendedorCobradorService.guardarVendedorCobrador).toHaveBeenCalled();
         });
 
-        it('la función $scope.guardar llama a guardaVendedorCobrador y edicion true' +
+        it('la función $scope.guardar llama a guardaVendedorCobrador y edicion true ' +
             'llama a $location.path', function(done)
         {
             inject(function($location) {
                 
                 //arrange
-                var scope= {};
+                var scope= {
+                    $watch: function(){},
+                };
                 var focaAbmVendedorCobradorService = {
                     getCodigoSiguiente: function() {
                         return {
@@ -306,10 +340,27 @@ describe('Controladores abm vendedor cobrador', function() {
                     $timeout: timeout,
                     $uibModal: {},
                     $location: $location,
-                    focaModalService: {}
+                    $localStorage: {},
+                    focaModalService: {
+                        alert: function() {}
+                    }
                 });
                 var promiseFake = Promise.resolve(true, false);
 
+                scope.vendedorCobrador = {
+                    NOM: true,
+                    domicilio: true,
+                    codigoPostal: true,
+                    dni: true,
+                    rol: true,
+                    provincia: {
+                        NOMBRE: true
+                    },
+                    localidad: {
+                        NOMBRE: true
+                    },
+                };
+
                 //act
                 spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
                     .and.returnValue(promiseFake);
@@ -324,11 +375,13 @@ describe('Controladores abm vendedor cobrador', function() {
             });
         });
 
-        it('la función $scope.guardar llama a guardaVendedorCobrador y scope.edicion false' +
+        it('la función $scope.guardar llama a guardaVendedorCobrador y scope.edicion false ' +
             'llama a scope.cambiarContraseña', function(done)
         {
             //arrange
-            var scope= {};
+            var scope= {
+                $watch: function() {}
+            };
             var focaAbmVendedorCobradorService = {
                 getCodigoSiguiente: function() {
                     return {
@@ -350,10 +403,27 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
-                focaModalService: {}
+                $localStorage: {},
+                focaModalService: {
+                    alert: function(){}
+                }
             });
             var promiseFake = Promise.resolve(true, false);
 
+            scope.vendedorCobrador = {
+                NOM: true,
+                domicilio: true,
+                codigoPostal: true,
+                dni: true,
+                rol: true,
+                provincia: {
+                    NOMBRE: true
+                },
+                localidad: {
+                    NOMBRE: true
+                },
+            };
+
             //act
             spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
                 .and.returnValue(promiseFake);
@@ -369,10 +439,12 @@ describe('Controladores abm vendedor cobrador', function() {
         });
 
         it('la función $scope.guardar llama a guardaVendedorCobrador y llama foca modal alert' +
-            'cuando la promsa es rechazada y el error es 409', function(done)
+            'cuando la promesa es rechazada y el error es 409', function(done)
         {
             //arrange
-            var scope= {};
+            var scope= {
+                $watch: function() {}
+            };
             var focaAbmVendedorCobradorService = {
                 getCodigoSiguiente: function() {
                     return {
@@ -397,6 +469,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
+                $localStorage: {},
                 focaModalService: focaModalService
             });
             var promiseFake = Promise.reject({ status: 409 });
@@ -418,7 +491,9 @@ describe('Controladores abm vendedor cobrador', function() {
 
         it('la función $scope.seleccionarProvincia llama a foca Modal', function() {
             //arrange
-            var scope= {};
+            var scope= {
+                $watch: function() {}
+            };
             var focaAbmVendedorCobradorService = {
                 getCodigoSiguiente: function() {
                     return {
@@ -442,6 +517,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
+                $localStorage: {},
                 focaModalService: focaModalService
             });
 
@@ -455,7 +531,9 @@ describe('Controladores abm vendedor cobrador', function() {
 
         it('función $scope.seleccionarLocalidad llama foca alert si no hay provincia', function() {
             //arrange
-            var scope= {};
+            var scope = {
+                $watch: function() {}
+            };
             var focaAbmVendedorCobradorService = {
                 getCodigoSiguiente: function() {
                     return {
@@ -479,6 +557,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
+                $localStorage: {},
                 focaModalService: focaModalService
             });
 
@@ -493,7 +572,9 @@ describe('Controladores abm vendedor cobrador', function() {
 
         it('función $scope.seleccionarLocalidad llama foca modal', function() {
             //arrange
-            var scope= {};
+            var scope= {
+                $watch: function() {}
+            };
             var focaAbmVendedorCobradorService = {
                 getCodigoSiguiente: function() {
                     return {
@@ -517,6 +598,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
+                $localStorage: {},
                 focaModalService: focaModalService
             });
 
@@ -533,7 +615,9 @@ describe('Controladores abm vendedor cobrador', function() {
 
             inject(function($location) {
                 //arrange
-                var scope = {};
+                var scope = {
+                    $watch: function() {}
+                };
                 $controller('focaAbmVendedorCobradorController', {
                     $scope: scope,
                     focaAbmVendedorCobradorService: {
@@ -553,6 +637,7 @@ describe('Controladores abm vendedor cobrador', function() {
                     $timeout: timeout,
                     $uibModal: {},
                     $location: $location,
+                    $localStorage: {},
                     focaModalService: {}
                 });
 
@@ -570,7 +655,9 @@ describe('Controladores abm vendedor cobrador', function() {
         it('la funcion scope.next suma uno a scope.focused', function() {
 
             //arrange
-            var scope = {};
+            var scope = {
+                $watch: function() {}
+            };
             $controller('focaAbmVendedorCobradorController', {
                 $scope: scope,
                 focaAbmVendedorCobradorService: {
@@ -590,6 +677,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: {},
                 $location: {},
+                $localStorage:  {},
                 focaModalService: {}
             });
 
@@ -604,7 +692,9 @@ describe('Controladores abm vendedor cobrador', function() {
         it('la funcion scope.cambiarPassword abre modal', function() {
 
             //arrange
-            var scope = {};
+            var scope = {
+                $watch: function() {}
+            };
             var uibModal = {
                 open: function() { }
             };
@@ -627,6 +717,7 @@ describe('Controladores abm vendedor cobrador', function() {
                 $timeout: timeout,
                 $uibModal: uibModal,
                 $location: {},
+                $localStorage: {},
                 focaModalService: {}
             });
             var responseFake = {
@@ -649,7 +740,9 @@ describe('Controladores abm vendedor cobrador', function() {
             inject(function($location) {
 
                 //arrange
-                var scope = {};
+                var scope = {
+                    $watch: function() {}
+                };
                 var uibModal = {
                     open: function() { }
                 };
@@ -672,6 +765,7 @@ describe('Controladores abm vendedor cobrador', function() {
                     $timeout: timeout,
                     $uibModal: uibModal,
                     $location: $location,
+                    $localStorage: {},
                     focaModalService: {}
                 });
                 var responseFake = {
diff --git a/src/js/controllerItem.js b/src/js/controllerItem.js
index b49022a..ee0b392 100644
--- a/src/js/controllerItem.js
+++ b/src/js/controllerItem.js
@@ -86,7 +86,10 @@ angular.module('focaAbmVendedorCobrador')
                 } else if (!$scope.vendedorCobrador.localidad.NOMBRE) {
                     focaModalService.alert('Seleccione una Localidad');
                     return;
-                }            
+                } else if ($scope.vendedorCobrador.comisionCobrador < 0 || $scope.vendedorCobrador.comisionVendedor < 0){
+                    focaModalService.alert('Ingrese un numero superior a 0 en porcentaje de comision');
+                    return;
+                }
 
                 if ($scope.guardado) {
                     $scope.cambiarPassword();