feat: adding conflict validation and verify if start_date is greater than end_date

Author: renanjava

backend/src/modules/time-trackers/__tests__/time-trackers.service.spec.ts

@@ -40,12 +40,17 @@ describe('TimeTrackersService', () => {
   it('should create timetrackers correctly', async () => {
     const timeTracker = new TimeTrackersStub();
     timeTrackersRepository.create.mockResolvedValue(timeTracker);
+    timeTrackersRepository.verifyTimeConflict.mockResolvedValue([]);
     const result = await service.create(timeTracker as CreateTimeTrackerDto);
 
     expect(result).toEqual(timeTracker);
     expect(timeTrackersRepository.create).toHaveBeenCalledWith(
       timeTracker as CreateTimeTrackerDto,
     );
+    expect(timeTrackersRepository.verifyTimeConflict).toHaveBeenCalledWith(
+      timeTracker.end_date,
+      timeTracker.start_date,
+    );
   });
 
   it('should find all timetrackers correctly', async () => {

backend/src/modules/time-trackers/dtos/create-time-tracker.dto.ts

@@ -1,17 +1,15 @@
-import {
-  IsDateString,
-  IsNotEmpty,
-  IsOptional,
-  IsString,
-} from 'class-validator';
+import { Type } from 'class-transformer';
+import { IsDate, IsNotEmpty, IsOptional, IsString } from 'class-validator';
 
 export class CreateTimeTrackerDto {
-  @IsDateString()
+  @IsDate()
   @IsNotEmpty()
+  @Type(() => Date)
   start_date: Date;
 
-  @IsDateString()
+  @IsDate()
   @IsNotEmpty()
+  @Type(() => Date)
   end_date: Date;
 
   @IsString()

backend/src/modules/time-trackers/mocks/time-trackers-repository.mock.ts

@@ -3,4 +3,5 @@ export const mockTimeTrackersRepository = () => ({
   findAll: jest.fn(),
   findOne: jest.fn(),
   update: jest.fn(),
+  verifyTimeConflict: jest.fn(),
 });

backend/src/modules/time-trackers/repositories/prisma-time-trackers.repository.ts

@@ -31,6 +31,17 @@ export class PrismaTimeTrackersRepository implements TimeTrackersRepository {
       include: { collaborators: true, tasks: true },
     });
   }
+  async verifyTimeConflict(
+    newTimeTrackerEndDate: Date,
+    newTimeTrackerStartDate: Date,
+  ): Promise<ResponseTimeTrackerDto[]> {
+    return await this.prismaClient.$queryRaw`
+  SELECT *
+  FROM "TimeTrackers" t
+  WHERE t."StartDate" < ${newTimeTrackerEndDate}
+    AND t."EndDate" > ${newTimeTrackerStartDate}
+`;
+  }
   async update(
     id: string,
     updateTimeTrackerDto: UpdateTimeTrackerDto,

backend/src/modules/time-trackers/time-trackers.service.ts

@@ -1,5 +1,9 @@
 import { PrismaTimeTrackersRepository } from './repositories/prisma-time-trackers.repository';
-import { Injectable } from '@nestjs/common';
+import {
+  BadRequestException,
+  ConflictException,
+  Injectable,
+} from '@nestjs/common';
 import { CreateTimeTrackerDto } from './dtos/create-time-tracker.dto';
 import { UpdateTimeTrackerDto } from './dtos/update-time-tracker.dto';
 
@@ -9,6 +13,25 @@ export class TimeTrackersService {
     private readonly timeTrackersRepository: PrismaTimeTrackersRepository,
   ) {}
   async create(createTimeTrackerDto: CreateTimeTrackerDto) {
+    const endDate = createTimeTrackerDto.end_date;
+    const startDate = createTimeTrackerDto.start_date;
+
+    if (startDate > endDate) {
+      throw new BadRequestException(
+        'A data de início é maior que a data de fim',
+      );
+    }
+
+    const dateConflict = await this.timeTrackersRepository.verifyTimeConflict(
+      endDate,
+      startDate,
+    );
+    if (dateConflict.length > 0) {
+      throw new ConflictException(
+        'Já existe uma tarefa nesse intervalo de tempo',
+      );
+    }
+
     return await this.timeTrackersRepository.create(createTimeTrackerDto);
   }