equipo programa service final

src/equipo-programa/entity/equipo-programa.entity.ts

@@ -16,7 +16,7 @@ export class EquipoPrograma {
   @Column({ type: Number, nullable: true })
   id_equipo: number;
 
-  @Column({ type: Number, nullable: true })
+  @Column({ type: Number, nullable: true, default: 1 })
   id_programa: number;
 
   @ManyToOne(() => Equipo, (equipo) => equipo.programas)

src/equipo-programa/equipo-programa.service.ts

@@ -30,18 +30,22 @@ export class EquipoProgramaService {
     private institucionProgramaService: InstitucionProgramaService,
   ) {}
 
-  async asignar(operador: Operador, id_equipo: number, id_programa: number) {
+  async asignar(
+    operador: Operador,
+    id_equipo: number,
+    id_programa: number,
+  ): Promise<{ message: string }> {
     const equipo = await this.equipoService.findById(id_equipo);
     const programa = await this.institucionProgramaService.findProgramaById(
       id_programa,
     );
 
-    // valido que el programa seleccionado no sea "Sin programa"
+    // Validamos que el programa seleccionado no sea "Sin programa"
     if (programa.id_programa === 1)
       throw new ConflictException(
-        'No se puede asignar manualmente este programa.',
+        'No se puede asignar manualmente sin programa.',
       );
-    // Valida que el equipo pertenezca a la institución del operador
+    // Validamos que el equipo pertenezca a la institución del operador
     if (
       operador.tipoUsuario.id_tipo_usuario > 2 &&
       operador.institucion.id_institucion !=
@@ -50,47 +54,47 @@ export class EquipoProgramaService {
       throw new ForbiddenException(
         'No puedes modificar la información este equipo porque no pertenece a tu institución.',
       );
-    // Verifica que no eixsta un registro con estos ids
-    return this.findByEquipoPrograma(equipo, programa).then(
-      (existeEquipoPrograma) => {
-        // Error si existe
+    // Buscamos un registro con los datos mandados del client
+    return this.findByEquipoPrograma(equipo, programa)
+      .then((existeEquipoPrograma) => {
+        // Si encuentro uno saco error
         if (existeEquipoPrograma)
           throw new ConflictException(
             'Este software ya fue asignado a este equipo.',
           );
         return this.create(equipo, programa);
+      })
+      .then((_) => ({
+        message: 'Se asignó correctamente el software a este equipo.',
+      }));
+  }
+
+  create(equipo: Equipo, programa: Programa): Promise<EquipoPrograma> {
+    // Verificamos si tiene programa "Sin programa"
+    return this.findByEquipoPrograma(equipo, { id_programa: 1 }).then(
+      async (existeEquipoSinPrograma) => {
+        // Si lo tiene lo eliminamos
+        if (existeEquipoSinPrograma)
+          await this.repository.remove(existeEquipoSinPrograma);
+        // Creamos y guardamos el registro
+        return this.repository.save(
+          this.repository.create({ equipo, programa }),
+        );
       },
     );
   }
 
-  create(equipo: Equipo, programa: Programa) {
-    // Ver si tiene programa "Sin programa"
-    return this.findByEquipoPrograma(equipo, { id_programa: 1 })
-      .then(async (existeEquipoSinPrograma) => {
-        // Si tiene lo elimino
-        if (existeEquipoSinPrograma)
-          await this.repository.remove(existeEquipoSinPrograma);
-        // Creo el registro
-        return this.repository.save(
-          this.repository.create({ equipo, programa }),
-        );
-      })
-      .then((equipoPrograma) => ({
-        message: `Se asignó el software: ${equipoPrograma.programa.programa}, al equipo con número de inventario: ${equipo.numero_inventario}.`,
-        equipoPrograma,
-      }));
+  createSinPrograma(equipo: Equipo): Promise<EquipoPrograma> {
+    return this.repository.save(this.repository.create({ equipo }));
   }
 
-  createSinPrograma(equipo: Equipo) {
-    return this.repository.save(
-      this.repository.create({ equipo, programa: { id_programa: 1 } }),
-    );
-  }
-
-  async delete(operador: Operador, id_equipo_programa: number) {
+  async delete(
+    operador: Operador,
+    id_equipo_programa: number,
+  ): Promise<{ message: string }> {
     const equipoPrograma = await this.findById(id_equipo_programa);
 
-    // Valida que el equipo pertenezca a la institución del operador
+    // Validamos que el equipo pertenezca a la institución del operador
     if (
       operador.institucion.id_institucion !=
       equipoPrograma.equipo.carrito.modulo.institucion.id_institucion
@@ -100,14 +104,16 @@ export class EquipoProgramaService {
       );
     // No se puede eliminar "Sin programa" manualmente
     if (equipoPrograma.programa.id_programa === 1)
-      throw new ConflictException('No se puede eliminar esta opción.');
-    // Ver cuantos programas tiene el equipo
+      throw new ConflictException(
+        'No se puede eliminar manualmente sin programa.',
+      );
+    // Buscamos cuantos programas tiene el equipo
     return this.repository
       .count({ where: { equipo: equipoPrograma.equipo } })
       .then(async (n) => {
         // Asignar "Sin programa" si es que solo tiene 1, el que se va a eliminar
         if (n === 1) await this.createSinPrograma(equipoPrograma.equipo);
-        // Eliminar registro
+        // Eliminamos registro
         return this.repository.remove(equipoPrograma);
       })
       .then((_) => ({
@@ -115,7 +121,7 @@ export class EquipoProgramaService {
       }));
   }
 
-  findFullInfoAllByIdEquipo(id_equipo: number) {
+  findFullInfoAllByIdEquipo(id_equipo: number): Promise<EquipoPrograma[]> {
     return this.fullInformacionEquipoProgramaView
       .find({ where: { id_equipo } })
       .then((infoEquipoProgramas) => {
@@ -135,7 +141,7 @@ export class EquipoProgramaService {
       });
   }
 
-  findById(id_equipo_programa: number) {
+  findById(id_equipo_programa: number): Promise<EquipoPrograma> {
     return this.informacionEquipoProgramaView
       .findOne({ where: { id_equipo_programa } })
       .then((infoEquipoPrograma) => {
@@ -163,7 +169,10 @@ export class EquipoProgramaService {
       });
   }
 
-  findByEquipoPrograma(equipo: Equipo, programa: Programa | Partial<Programa>) {
+  findByEquipoPrograma(
+    equipo: Equipo,
+    programa: Programa | Partial<Programa>,
+  ): Promise<EquipoPrograma> {
     return this.repository.findOne({ where: { equipo, programa } });
   }
 }