feat: uncomplete habit regardless time

src/adapters/database/inMemory/MemoryHabitCompletionDateRepository.ts

@@ -14,20 +14,39 @@ export class MemoryHabitCompletionDateRepository
   }
 
   async findByHabitIdAndDate(habitId: string, completedDate: string) {
-    return this.habitCompletionDates.find(
-      habit =>
-        habit.habitId === habitId && habit.completedDate === completedDate,
-    );
+    const completeDateWithoutHour = new Date(completedDate)
+      .toISOString()
+      .substring(0, 10);
+
+    return this.habitCompletionDates.find(habit => {
+      const storeCompletedDateWithoutHour = new Date(habit.completedDate)
+        .toISOString()
+        .substring(0, 10);
+
+      return (
+        habit.habitId === habitId &&
+        completeDateWithoutHour === storeCompletedDateWithoutHour
+      );
+    });
   }
 
   async list() {
     return this.habitCompletionDates;
   }
 
   async delete(habitId: string, completedDate: string) {
-    this.habitCompletionDates = this.habitCompletionDates.filter(
-      habit =>
-        !(habit.habitId === habitId && habit.completedDate === completedDate),
-    );
+    const completedDateWithoutHour = new Date(completedDate)
+      .toISOString()
+      .substring(0, 10);
+
+    this.habitCompletionDates = this.habitCompletionDates.filter(habit => {
+      const storedCompletedDateWithouHour = new Date(habit.completedDate)
+        .toISOString()
+        .substring(0, 10);
+      return !(
+        habit.habitId === habitId &&
+        storedCompletedDateWithouHour === completedDateWithoutHour
+      );
+    });
   }
 }

src/adapters/database/postgres/PgHabitCompDateRepository.ts

@@ -36,8 +36,8 @@ export class PgHabitCompDateRepository
   async findByHabitIdAndDate(habitId: string, completedDate: string) {
     const [habit] = (
       await pool.query<PgHabitCompletionDate>(
-        `SELECT * FROM habit_completion_dates 
-        WHERE habit_id = $1 AND completed_date = $2`,
+        `SELECT *, completed_date AT TIME ZONE 'UTC' as completed_date FROM habit_completion_dates
+      WHERE habit_id = $1 AND DATE(completed_date) = $2`,
         [habitId, completedDate],
       )
     ).rows;
@@ -51,7 +51,7 @@ export class PgHabitCompDateRepository
   async delete(habitId: string, completedDate: string) {
     await pool.query(
       `DELETE FROM habit_completion_dates 
-      WHERE habit_id = $1 AND completed_date = $2`,
+      WHERE habit_id = $1 AND DATE(completed_date) = $2`,
       [habitId, completedDate],
     );
   }

src/application/tests/HabitCompletionDateUseCase.test.ts

@@ -69,4 +69,14 @@ describe('Complete a Habit', () => {
       habitCompletionDateUseCase.execute(habitId, completedDate),
     ).rejects.toEqual({ statusCode: 400, message: 'habit not found' });
   });
+
+  it('should be possible to uncomplete a habit regardless of the time.', async () => {
+    const createdHabit = await createHabitUseCase.execute('run');
+    const habitCompletionDate = await habitCompletionDateUseCase.execute(
+      createdHabit.id,
+      completedDate,
+    );
+    expect(habitCompletionDate?.habitId).toBe(createdHabit.id);
+    expect(habitCompletionDate?.completedDate).toBe(completedDate);
+  });
 });

src/domain/entities/HabitCompletionDate.ts

@@ -1,5 +1,5 @@
 export interface HabitCompletionDate {
   id: string;
   habitId: string;
-  completedDate?: string;
+  completedDate: string;
 }

src/infra/web/routes/e2e/habitCompDate.routes.test.ts

@@ -85,4 +85,14 @@ describe('Habit Complete Date Controllers', () => {
     expect(response.status).toBe(400);
     expect(response.body).toBe('habit not found');
   });
+
+  it('POST (should be able to uncomplete a habit regardless of the time)', async () => {
+    const response = await request(app).post('/').send({
+      habitId: '4567',
+      completedDate: '2000-02-03T23:59:59.000Z',
+    });
+    const completedHabits = await memoryHabitCompDateRepository.list();
+    expect(completedHabits).toHaveLength(1);
+    expect(response.status).toBe(200);
+  });
 });