fix unit testing

Author: antonio06

src/modules/users/edit-user-sheet/edit-user-sheet.mappers.spec.ts

@@ -4,6 +4,28 @@ import { mapUserFromApiToVm, mapUserFromVmToApi } from './edit-user-sheet.mapper
 
 describe('edit mapper', () => {
   describe('mapUserFromApiToVm', () => {
+    it('should return a empty user when feed a null value', () => {
+      // Arrange
+      const user: apiModel.Usuario = null!;
+
+      // Act
+      const result = mapUserFromApiToVm(user);
+
+      // Assert
+      expect(result).toEqual(viewModel.createEmptyUsuario());
+    });
+
+    it('should return a empty user when feed a undefined value', () => {
+      // Arrange
+      const user: apiModel.Usuario = undefined!;
+
+      // Act
+      const result = mapUserFromApiToVm(user);
+
+      // Assert
+      expect(result).toEqual(viewModel.createEmptyUsuario());
+    });
+
     it('should return a mapped vm user, when feed a user api', () => {
       // Arrange
       const user: apiModel.Usuario = {
@@ -44,6 +66,28 @@ describe('edit mapper', () => {
   });
 
   describe('mapUserFromVmToApi', () => {
+    it('should return a null user when feed a null value', () => {
+      // Arrange
+      const user: viewModel.Usuario = null!;
+
+      // Act
+      const result = mapUserFromVmToApi(user);
+
+      // Assert
+      expect(result).toEqual(null);
+    });
+
+    it('should return a null user when feed a undefined value', () => {
+      // Arrange
+      const user: viewModel.Usuario = undefined!;
+
+      // Act
+      const result = mapUserFromVmToApi(user);
+
+      // Assert
+      expect(result).toEqual(null);
+    });
+
     it('should return a mapped vm user, when feed a user api', () => {
       // Arrange
       const user: viewModel.Usuario = {

src/modules/users/edit-user-sheet/edit-user-sheet.mappers.ts

@@ -1,30 +1,36 @@
 import * as apiModel from './api';
 import * as viewModel from './edit-user-sheet.vm';
 
-export const mapUserFromApiToVm = (user: apiModel.Usuario): viewModel.Usuario => ({
-  id: user.id,
-  nombre: user.nombre,
-  apellido: user.apellido,
-  telefono: user.telefono,
-  movil: user.movil,
-  email: user.email,
-  rol: user.rol,
-  unidad: user.unidad,
-  esResponsable: user.esResponsable,
-  esProponente: user.esProponente,
-  esAutorizante: user.esAutorizante,
-});
+export const mapUserFromApiToVm = (user: apiModel.Usuario): viewModel.Usuario =>
+  user
+    ? {
+        id: user.id,
+        nombre: user.nombre,
+        apellido: user.apellido,
+        telefono: user.telefono,
+        movil: user.movil,
+        email: user.email,
+        rol: user.rol,
+        unidad: user.unidad,
+        esResponsable: user.esResponsable,
+        esProponente: user.esProponente,
+        esAutorizante: user.esAutorizante,
+      }
+    : viewModel.createEmptyUsuario();
 
-export const mapUserFromVmToApi = (usuario: viewModel.Usuario): apiModel.Usuario => ({
-  id: usuario.id,
-  nombre: usuario.nombre,
-  apellido: usuario.apellido,
-  telefono: usuario.telefono,
-  movil: usuario.movil,
-  email: usuario.email,
-  rol: usuario.rol,
-  unidad: usuario.unidad,
-  esResponsable: usuario.esResponsable,
-  esProponente: usuario.esProponente,
-  esAutorizante: usuario.esAutorizante,
-});
+export const mapUserFromVmToApi = (user: viewModel.Usuario): apiModel.Usuario =>
+  user
+    ? {
+        id: user.id,
+        nombre: user.nombre,
+        apellido: user.apellido,
+        telefono: user.telefono,
+        movil: user.movil,
+        email: user.email,
+        rol: user.rol,
+        unidad: user.unidad,
+        esResponsable: user.esResponsable,
+        esProponente: user.esProponente,
+        esAutorizante: user.esAutorizante,
+      }
+    : null!;