marca service final

src/marca/marca.service.ts

@@ -7,9 +7,9 @@ import { Marca } from './entity/marca.entity';
 export class MarcaService {
   constructor(@InjectRepository(Marca) private repository: Repository<Marca>) {}
 
-  create(marca: string, tipo: string) {
-    // Buscamos una marca con ese nombre
-    return this.findByMarca(marca, tipo, false)
+  create(marca: string, tipo: string): Promise<{ message: string }> {
+    // Buscamos un registro con ese nombre
+    return this.findByMarca(marca, tipo)
       .then((existeMarca) => {
         // Sacamos error si existe
         if (existeMarca) throw new ConflictException('Ya existe esta marca.');
@@ -19,25 +19,20 @@ export class MarcaService {
       .then((_) => ({ message: 'Se creó correctamente una nueva marca.' }));
   }
 
-  findAll(tipo: string) {
+  findAll(tipo: string): Promise<Marca[]> {
     return this.repository.find({ where: { tipo }, order: { tipo: 'ASC' } });
   }
 
-  findById(id_marca: number, tipo: string, validarExistencia = true) {
+  findById(id_marca: number, tipo: string): Promise<Marca> {
     return this.repository
       .findOne({ where: { id_marca, tipo } })
       .then((marca) => {
-        if (validarExistencia && !marca)
-          throw new ConflictException('No existe esta marca.');
+        if (!marca) throw new ConflictException('No existe este id marca.');
         return marca;
       });
   }
 
-  findByMarca(marca: string, tipo: string, validarExistencia = true) {
-    return this.repository.findOne({ where: { marca, tipo } }).then((marca) => {
-      if (validarExistencia && !marca)
-        throw new ConflictException('No existe esta marca.');
-      return marca;
-    });
+  findByMarca(marca: string, tipo: string): Promise<Marca> {
+    return this.repository.findOne({ where: { marca, tipo } });
   }
 }

src/upload-file/upload-file.service.ts

@@ -190,11 +190,7 @@ export class UploadFileService {
       institucion,
       dataEquipo.modulo,
     );
-    const marca = await this.marcaService.findByMarca(
-      dataEquipo.marca,
-      'e',
-      false,
-    );
+    const marca = await this.marcaService.findByMarca(dataEquipo.marca, 'e');
     const modelo = await this.modeloService.findByModelo(
       dataEquipo.modelo,
       'e',