Luis Suarez / foca-abm-vendedor-cobrador

Download as
Browse Code ยป

Commit d01a61a08ee192fc176547b790e0913cdb632aa1

Authored by Jose Pinto
2 parents 134e815d02 faf6ae477e
Exists in master and in 2 other branches develop, lab

Merge branch 'master' into 'develop' 

Master

See merge request !25
Showing 2 changed files   Show diff stats
spec/controllerSpec.js
Diff comments   View file @ d01a61a
... ... @@ -23,6 +23,7 @@ describe('Controladores abm vendedor cobrador', function() {
23 23 $scope: {},
24 24 focaAbmVendedorCobradorService: {},
25 25 $location: {},
  26 + $localStorage: {},
26 27 $filter: {},
27 28 focaBotoneraLateralService: focaBotoneraLateralService,
28 29 focaModalService: {}
... ... @@ -41,6 +42,7 @@ describe('Controladores abm vendedor cobrador', function() {
41 42 $scope: scope,
42 43 focaAbmVendedorCobradorService: {},
43 44 $location: $location,
  45 + $localStorage: {},
44 46 $filter: {},
45 47 focaBotoneraLateralService: focaBotoneraLateralService,
46 48 focaModalService: {}
... ... @@ -66,6 +68,7 @@ describe('Controladores abm vendedor cobrador', function() {
66 68 $scope: scope,
67 69 focaAbmVendedorCobradorService: {},
68 70 $location: $location,
  71 + $localStorage: {},
69 72 $filter: {},
70 73 focaBotoneraLateralService: focaBotoneraLateralService,
71 74 focaModalService: focaModalService
... ... @@ -96,6 +99,7 @@ describe('Controladores abm vendedor cobrador', function() {
96 99 $scope: scope,
97 100 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
98 101 $location: $location,
  102 + $localStorage: {},
99 103 $filter: {},
100 104 focaBotoneraLateralService: focaBotoneraLateralService,
101 105 focaModalService: focaModalService
... ... @@ -126,6 +130,7 @@ describe('Controladores abm vendedor cobrador', function() {
126 130 $scope: scope,
127 131 focaAbmVendedorCobradorService: {},
128 132 $location: {},
  133 + $localStorage: {},
129 134 $filter: {},
130 135 focaBotoneraLateralService: focaBotoneraLateralService,
131 136 focaModalService: {}
... ... @@ -150,6 +155,7 @@ describe('Controladores abm vendedor cobrador', function() {
150 155 $scope: scope,
151 156 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
152 157 $location: {},
  158 + $localStorage: {},
153 159 $filter: {},
154 160 focaBotoneraLateralService: focaBotoneraLateralService,
155 161 focaModalService: {}
... ... @@ -175,6 +181,7 @@ describe('Controladores abm vendedor cobrador', function() {
175 181 $scope: scope,
176 182 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
177 183 $location: {},
  184 + $localStorage: {},
178 185 $filter: {},
179 186 focaBotoneraLateralService: focaBotoneraLateralService,
180 187 focaModalService: {}
... ... @@ -210,7 +217,9 @@ describe('Controladores abm vendedor cobrador', function() {
210 217  
211 218 //act
212 219 var controlador = $controller('focaAbmVendedorCobradorController', {
213   - $scope: {},
  220 + $scope: {
  221 + $watch: function() {}
  222 + },
214 223 focaAbmVendedorCobradorService: {
215 224 getCodigoSiguiente: function() {
216 225 return {
... ... @@ -228,7 +237,9 @@ describe('Controladores abm vendedor cobrador', function() {
228 237 $timeout: timeout,
229 238 $uibModal: {},
230 239 $location: {},
231   - focaModalService: {}
  240 + $localStorage: {},
  241 + focaModalService: {
  242 + }
232 243 });
233 244  
234 245 //assert
... ... @@ -237,7 +248,9 @@ describe('Controladores abm vendedor cobrador', function() {
237 248  
238 249 it('la funciรณn $scope.guardar llama a guardaVendedorCobrador del servicio', function() {
239 250 //arrange
240   - var scope= {};
  251 + var scope = {
  252 + $watch: function() {},
  253 + };
241 254 var focaAbmVendedorCobradorService = {
242 255 getCodigoSiguiente: function() {
243 256 return {
... ... @@ -259,7 +272,11 @@ describe('Controladores abm vendedor cobrador', function() {
259 272 $timeout: timeout,
260 273 $uibModal: {},
261 274 $location: {},
262   - focaModalService: {}
  275 + $localStorage: {},
  276 + guardarVendedorCobrador: {},
  277 + focaModalService: {
  278 + alert: function() {}
  279 + }
263 280 });
264 281 var responseFake = {
265 282 then: function() {
... ... @@ -268,6 +285,19 @@ describe('Controladores abm vendedor cobrador', function() {
268 285 };
269 286 }
270 287 };
  288 + scope.vendedorCobrador = {
  289 + NOM: true,
  290 + domicilio: true,
  291 + codigoPostal: true,
  292 + dni: true,
  293 + rol: true,
  294 + provincia: {
  295 + NOMBRE: true
  296 + },
  297 + localidad: {
  298 + NOMBRE: true
  299 + },
  300 + };
271 301  
272 302 //act
273 303 spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
... ... @@ -278,13 +308,15 @@ describe('Controladores abm vendedor cobrador', function() {
278 308 expect(focaAbmVendedorCobradorService.guardarVendedorCobrador).toHaveBeenCalled();
279 309 });
280 310  
281   - it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y edicion true' +
  311 + it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y edicion true ' +
282 312 'llama a $location.path', function(done)
283 313 {
284 314 inject(function($location) {
285 315  
286 316 //arrange
287   - var scope= {};
  317 + var scope= {
  318 + $watch: function(){},
  319 + };
288 320 var focaAbmVendedorCobradorService = {
289 321 getCodigoSiguiente: function() {
290 322 return {
... ... @@ -306,10 +338,27 @@ describe('Controladores abm vendedor cobrador', function() {
306 338 $timeout: timeout,
307 339 $uibModal: {},
308 340 $location: $location,
309   - focaModalService: {}
  341 + $localStorage: {},
  342 + focaModalService: {
  343 + alert: function() {}
  344 + }
310 345 });
311 346 var promiseFake = Promise.resolve(true, false);
312 347  
  348 + scope.vendedorCobrador = {
  349 + NOM: true,
  350 + domicilio: true,
  351 + codigoPostal: true,
  352 + dni: true,
  353 + rol: true,
  354 + provincia: {
  355 + NOMBRE: true
  356 + },
  357 + localidad: {
  358 + NOMBRE: true
  359 + },
  360 + };
  361 +
313 362 //act
314 363 spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
315 364 .and.returnValue(promiseFake);
... ... @@ -324,11 +373,13 @@ describe('Controladores abm vendedor cobrador', function() {
324 373 });
325 374 });
326 375  
327   - it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y scope.edicion false' +
  376 + it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y scope.edicion false ' +
328 377 'llama a scope.cambiarContraseรฑa', function(done)
329 378 {
330 379 //arrange
331   - var scope= {};
  380 + var scope= {
  381 + $watch: function() {}
  382 + };
332 383 var focaAbmVendedorCobradorService = {
333 384 getCodigoSiguiente: function() {
334 385 return {
... ... @@ -350,10 +401,27 @@ describe('Controladores abm vendedor cobrador', function() {
350 401 $timeout: timeout,
351 402 $uibModal: {},
352 403 $location: {},
353   - focaModalService: {}
  404 + $localStorage: {},
  405 + focaModalService: {
  406 + alert: function() {}
  407 + }
354 408 });
355 409 var promiseFake = Promise.resolve(true, false);
356 410  
  411 + scope.vendedorCobrador = {
  412 + NOM: true,
  413 + domicilio: true,
  414 + codigoPostal: true,
  415 + dni: true,
  416 + rol: true,
  417 + provincia: {
  418 + NOMBRE: true
  419 + },
  420 + localidad: {
  421 + NOMBRE: true
  422 + },
  423 + };
  424 +
357 425 //act
358 426 spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
359 427 .and.returnValue(promiseFake);
... ... @@ -369,10 +437,12 @@ describe('Controladores abm vendedor cobrador', function() {
369 437 });
370 438  
371 439 it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y llama foca modal alert' +
372   - 'cuando la promsa es rechazada y el error es 409', function(done)
  440 + 'cuando la promesa es rechazada y el error es 409', function(done)
373 441 {
374 442 //arrange
375   - var scope= {};
  443 + var scope= {
  444 + $watch: function() {}
  445 + };
376 446 var focaAbmVendedorCobradorService = {
377 447 getCodigoSiguiente: function() {
378 448 return {
... ... @@ -397,6 +467,7 @@ describe('Controladores abm vendedor cobrador', function() {
397 467 $timeout: timeout,
398 468 $uibModal: {},
399 469 $location: {},
  470 + $localStorage: {},
400 471 focaModalService: focaModalService
401 472 });
402 473 var promiseFake = Promise.reject({ status: 409 });
... ... @@ -418,7 +489,9 @@ describe('Controladores abm vendedor cobrador', function() {
418 489  
419 490 it('la funciรณn $scope.seleccionarProvincia llama a foca Modal', function() {
420 491 //arrange
421   - var scope= {};
  492 + var scope= {
  493 + $watch: function() {}
  494 + };
422 495 var focaAbmVendedorCobradorService = {
423 496 getCodigoSiguiente: function() {
424 497 return {
... ... @@ -442,6 +515,7 @@ describe('Controladores abm vendedor cobrador', function() {
442 515 $timeout: timeout,
443 516 $uibModal: {},
444 517 $location: {},
  518 + $localStorage: {},
445 519 focaModalService: focaModalService
446 520 });
447 521  
... ... @@ -455,7 +529,9 @@ describe('Controladores abm vendedor cobrador', function() {
455 529  
456 530 it('funciรณn $scope.seleccionarLocalidad llama foca alert si no hay provincia', function() {
457 531 //arrange
458   - var scope= {};
  532 + var scope = {
  533 + $watch: function() {}
  534 + };
459 535 var focaAbmVendedorCobradorService = {
460 536 getCodigoSiguiente: function() {
461 537 return {
... ... @@ -479,6 +555,7 @@ describe('Controladores abm vendedor cobrador', function() {
479 555 $timeout: timeout,
480 556 $uibModal: {},
481 557 $location: {},
  558 + $localStorage: {},
482 559 focaModalService: focaModalService
483 560 });
484 561  
... ... @@ -493,7 +570,9 @@ describe('Controladores abm vendedor cobrador', function() {
493 570  
494 571 it('funciรณn $scope.seleccionarLocalidad llama foca modal', function() {
495 572 //arrange
496   - var scope= {};
  573 + var scope= {
  574 + $watch: function() {}
  575 + };
497 576 var focaAbmVendedorCobradorService = {
498 577 getCodigoSiguiente: function() {
499 578 return {
... ... @@ -517,6 +596,7 @@ describe('Controladores abm vendedor cobrador', function() {
517 596 $timeout: timeout,
518 597 $uibModal: {},
519 598 $location: {},
  599 + $localStorage: {},
520 600 focaModalService: focaModalService
521 601 });
522 602  
... ... @@ -533,7 +613,9 @@ describe('Controladores abm vendedor cobrador', function() {
533 613  
534 614 inject(function($location) {
535 615 //arrange
536   - var scope = {};
  616 + var scope = {
  617 + $watch: function() {}
  618 + };
537 619 $controller('focaAbmVendedorCobradorController', {
538 620 $scope: scope,
539 621 focaAbmVendedorCobradorService: {
... ... @@ -553,6 +635,7 @@ describe('Controladores abm vendedor cobrador', function() {
553 635 $timeout: timeout,
554 636 $uibModal: {},
555 637 $location: $location,
  638 + $localStorage: {},
556 639 focaModalService: {}
557 640 });
558 641  
... ... @@ -570,7 +653,9 @@ describe('Controladores abm vendedor cobrador', function() {
570 653 it('la funcion scope.next suma uno a scope.focused', function() {
571 654  
572 655 //arrange
573   - var scope = {};
  656 + var scope = {
  657 + $watch: function() {}
  658 + };
574 659 $controller('focaAbmVendedorCobradorController', {
575 660 $scope: scope,
576 661 focaAbmVendedorCobradorService: {
... ... @@ -590,6 +675,7 @@ describe('Controladores abm vendedor cobrador', function() {
590 675 $timeout: timeout,
591 676 $uibModal: {},
592 677 $location: {},
  678 + $localStorage: {},
593 679 focaModalService: {}
594 680 });
595 681  
... ... @@ -604,7 +690,9 @@ describe('Controladores abm vendedor cobrador', function() {
604 690 it('la funcion scope.cambiarPassword abre modal', function() {
605 691  
606 692 //arrange
607   - var scope = {};
  693 + var scope = {
  694 + $watch: function() {}
  695 + };
608 696 var uibModal = {
609 697 open: function() { }
610 698 };
... ... @@ -627,6 +715,7 @@ describe('Controladores abm vendedor cobrador', function() {
627 715 $timeout: timeout,
628 716 $uibModal: uibModal,
629 717 $location: {},
  718 + $localStorage: {},
630 719 focaModalService: {}
631 720 });
632 721 var responseFake = {
... ... @@ -649,7 +738,9 @@ describe('Controladores abm vendedor cobrador', function() {
649 738 inject(function($location) {
650 739  
651 740 //arrange
652   - var scope = {};
  741 + var scope = {
  742 + $watch: function() {}
  743 + };
653 744 var uibModal = {
654 745 open: function() { }
655 746 };
... ... @@ -672,6 +763,7 @@ describe('Controladores abm vendedor cobrador', function() {
672 763 $timeout: timeout,
673 764 $uibModal: uibModal,
674 765 $location: $location,
  766 + $localStorage: {},
675 767 focaModalService: {}
676 768 });
677 769 var responseFake = {
src/js/controllerItem.js
Diff comments   View file @ d01a61a
... ... @@ -86,7 +86,10 @@ angular.module('focaAbmVendedorCobrador')
86 86 } else if (!$scope.vendedorCobrador.localidad.NOMBRE) {
87 87 focaModalService.alert('Seleccione una Localidad');
88 88 return;
89   - }
  89 + } else if ($scope.vendedorCobrador.comisionCobrador < 0 || $scope.vendedorCobrador.comisionVendedor < 0){
  90 + focaModalService.alert('Ingrese un numero superior a 0 en porcentaje de comision');
  91 + return;
  92 + }
90 93  
91 94 if ($scope.guardado) {
92 95 $scope.cambiarPassword();