From a077dd51651579522bcf21e8960ba51a82afcca1 Mon Sep 17 00:00:00 2001
From: Joan Sanchez <joan@verdnatura.es>
Date: Fri, 22 Nov 2019 14:34:20 +0100
Subject: [PATCH] test unitario

---
 db/dump/fixtures.sql                          |  5 ++++
 .../worker-dms/specs/removeFile.spec.js       |  6 ++--
 modules/worker/front/dms/create/index.spec.js | 30 ++++++++++---------
 3 files changed, 24 insertions(+), 17 deletions(-)

diff --git a/db/dump/fixtures.sql b/db/dump/fixtures.sql
index 86ee0a6120..0d81f0a3d3 100644
--- a/db/dump/fixtures.sql
+++ b/db/dump/fixtures.sql
@@ -1880,6 +1880,11 @@ INSERT INTO `vn`.`clientDms`(`clientFk`, `dmsFk`)
         (104, 2),
         (104, 3);
 
+INSERT INTO `vn`.`workerDocument`(`worker`, `document`)
+    VALUES
+        (104, 2),
+        (104, 3);
+
 INSERT INTO `vn`.`device` (`sn`, `model`, `userFk`) 
     VALUES 
         ('aaa', 'android', '9');
diff --git a/modules/worker/back/methods/worker-dms/specs/removeFile.spec.js b/modules/worker/back/methods/worker-dms/specs/removeFile.spec.js
index 01cf1977b1..f8fe36625d 100644
--- a/modules/worker/back/methods/worker-dms/specs/removeFile.spec.js
+++ b/modules/worker/back/methods/worker-dms/specs/removeFile.spec.js
@@ -1,13 +1,13 @@
 const app = require('vn-loopback/server/server');
 
-describe('ClientDms removeFile()', () => {
-    const clientDmsFk = 3;
+fdescribe('WorkerDms removeFile()', () => {
+    const workerDmsFk = 1;
     it(`should return an error for a user without enough privileges`, async() => {
         let clientId = 101;
         let ctx = {req: {accessToken: {userId: clientId}}};
 
         let error;
-        await app.models.ClientDms.removeFile(ctx, clientDmsFk).catch(e => {
+        await app.models.WorkerDms.removeFile(ctx, workerDmsFk).catch(e => {
             error = e;
         }).finally(() => {
             expect(error.message).toEqual(`You don't have enough privileges`);
diff --git a/modules/worker/front/dms/create/index.spec.js b/modules/worker/front/dms/create/index.spec.js
index 4ff8ce1229..41fe0e0cac 100644
--- a/modules/worker/front/dms/create/index.spec.js
+++ b/modules/worker/front/dms/create/index.spec.js
@@ -1,32 +1,34 @@
 import './index';
 
 describe('Client', () => {
-    describe('Component vnClientDmsCreate', () => {
+    describe('Component vnWorkerDmsCreate', () => {
+        let $element;
         let controller;
         let $scope;
         let $httpBackend;
         let $httpParamSerializer;
 
-        beforeEach(ngModule('client'));
+        beforeEach(ngModule('worker'));
 
-        beforeEach(angular.mock.inject(($componentController, $rootScope, _$httpBackend_, _$httpParamSerializer_) => {
+        beforeEach(angular.mock.inject(($compile, $rootScope, _$httpBackend_, _$httpParamSerializer_) => {
             $scope = $rootScope.$new();
             $httpBackend = _$httpBackend_;
             $httpParamSerializer = _$httpParamSerializer_;
-            controller = $componentController('vnClientDmsCreate', {$scope});
-            controller._client = {id: 101, name: 'Bruce wayne'};
+            $element = $compile(`<vn-worker-dms-create></vn-worker-dms-create>`)($rootScope);
+            controller = $element.controller('vnWorkerDmsCreate');
+            controller._worker = {id: 101, name: 'Bruce wayne'};
         }));
 
-        describe('client() setter', () => {
-            it('should set the client data and then call setDefaultParams() and getAllowedContentTypes()', () => {
+        describe('worker() setter', () => {
+            it('should set the worker data and then call setDefaultParams() and getAllowedContentTypes()', () => {
                 spyOn(controller, 'setDefaultParams');
                 spyOn(controller, 'getAllowedContentTypes');
-                controller.client = {
+                controller.worker = {
                     id: 15,
                     name: 'Bruce wayne'
                 };
 
-                expect(controller.client).toBeDefined();
+                expect(controller.worker).toBeDefined();
                 expect(controller.setDefaultParams).toHaveBeenCalledWith();
                 expect(controller.getAllowedContentTypes).toHaveBeenCalledWith();
             });
@@ -34,12 +36,12 @@ describe('Client', () => {
 
         describe('setDefaultParams()', () => {
             it('should perform a GET query and define the dms property on controller', () => {
+                $httpBackend.whenRoute('GET', `DmsTypes`).respond({id: 12, code: 'hhrrData'});
                 const params = {filter: {
-                    where: {code: 'paymentsLaw'}
+                    where: {code: 'hhrrData'}
                 }};
                 let serializedParams = $httpParamSerializer(params);
-                $httpBackend.when('GET', `DmsTypes/findOne?${serializedParams}`).respond({id: 12, code: 'paymentsLaw'});
-                $httpBackend.expect('GET', `DmsTypes/findOne?${serializedParams}`);
+                $httpBackend.when('GET', `DmsTypes/findOne?${serializedParams}`).respond({id: 12, code: 'hhrrData'});
                 controller.setDefaultParams();
                 $httpBackend.flush();
 
@@ -62,8 +64,8 @@ describe('Client', () => {
         describe('getAllowedContentTypes()', () => {
             it('should make an HTTP GET request to get the allowed content types', () => {
                 const expectedResponse = ['image/png', 'image/jpg'];
-                $httpBackend.when('GET', `clientDms/allowedContentTypes`).respond(expectedResponse);
-                $httpBackend.expect('GET', `clientDms/allowedContentTypes`);
+                $httpBackend.when('GET', `workerDms/allowedContentTypes`).respond(expectedResponse);
+                $httpBackend.expect('GET', `workerDms/allowedContentTypes`);
                 controller.getAllowedContentTypes();
                 $httpBackend.flush();