Updating tests to match what we expect the code to return now.

Author: renemadsen

eform-client/src/app/plugins/modules/time-planning-pn/components/plannings/time-plannings-table/time-plannings-table.component.spec.ts

@@ -171,7 +171,7 @@ describe('TimePlanningsTableComponent', () => {
         }
       };
 
-      expect(component.getCellClass(row, '0')).toBe('grey-background');
+      expect(component.getCellClass(row, '0')).toBe('red-background');
     });
 
     it('should return green-background for cell with work started and ended', () => {
@@ -274,7 +274,7 @@ describe('TimePlanningsTableComponent', () => {
         }
       };
 
-      expect(component.getCellClass(row, '0')).toBe('grey-background');
+      expect(component.getCellClass(row, '0')).toBe('red-background');
     });
 
     it('should return grey-background when message is set', () => {
@@ -316,42 +316,42 @@ describe('TimePlanningsTableComponent', () => {
     });
   });
 
-  describe('isInOlderThanToday', () => {
-    it('should return false for null date', () => {
-      expect(component.isInOlderThanToday(null as any)).toBe(false);
-    });
-
-    it('should return false for undefined date', () => {
-      expect(component.isInOlderThanToday(undefined as any)).toBe(false);
-    });
-
-    it('should return true for date in the past', () => {
-      const pastDate = new Date();
-      pastDate.setDate(pastDate.getDate() - 5);
-      expect(component.isInOlderThanToday(pastDate)).toBe(true);
-    });
-
-    it('should return false for today', () => {
-      const today = new Date();
-      expect(component.isInOlderThanToday(today)).toBe(false);
-    });
-
-    it('should return false for future date', () => {
-      const futureDate = new Date();
-      futureDate.setDate(futureDate.getDate() + 5);
-      expect(component.isInOlderThanToday(futureDate)).toBe(false);
-    });
-
-    it('should handle string dates', () => {
-      const pastDateString = '2020-01-01';
-      expect(component.isInOlderThanToday(pastDateString as any)).toBe(true);
-    });
-
-    it('should return false for invalid date string', () => {
-      const invalidDate = 'invalid-date';
-      expect(component.isInOlderThanToday(invalidDate as any)).toBe(false);
-    });
-  });
+  // describe('isInOlderThanToday', () => {
+  //   it('should return false for null date', () => {
+  //     expect(component.isInOlderThanToday(null as any)).toBe(false);
+  //   });
+  //
+  //   it('should return false for undefined date', () => {
+  //     expect(component.isInOlderThanToday(undefined as any)).toBe(false);
+  //   });
+  //
+  //   it('should return true for date in the past', () => {
+  //     const pastDate = new Date();
+  //     pastDate.setDate(pastDate.getDate() - 5);
+  //     expect(component.isInOlderThanToday(pastDate)).toBe(true);
+  //   });
+  //
+  //   it('should return false for today', () => {
+  //     const today = new Date();
+  //     expect(component.isInOlderThanToday(today)).toBe(false);
+  //   });
+  //
+  //   it('should return false for future date', () => {
+  //     const futureDate = new Date();
+  //     futureDate.setDate(futureDate.getDate() + 5);
+  //     expect(component.isInOlderThanToday(futureDate)).toBe(false);
+  //   });
+  //
+  //   it('should handle string dates', () => {
+  //     const pastDateString = '2020-01-01';
+  //     expect(component.isInOlderThanToday(pastDateString as any)).toBe(true);
+  //   });
+  //
+  //   it('should return false for invalid date string', () => {
+  //     const invalidDate = 'invalid-date';
+  //     expect(component.isInOlderThanToday(invalidDate as any)).toBe(false);
+  //   });
+  // });
 
   describe('getStopTimeDisplay', () => {
     it('should return empty string when startedAt is null', () => {

eform-client/src/app/plugins/modules/time-planning-pn/components/plannings/time-plannings-table/time-plannings-table.component.ts

@@ -205,30 +205,32 @@ export class TimePlanningsTableComponent implements OnInit, OnChanges {
 
   protected readonly JSON = JSON;
 
-  isInOlderThanToday(date: Date): boolean {
-    // take the midnight of the date to compare ot today's midnight
-    if (!date) {
-      return false;
-    }
-    // Convert the date to a string to ensure it's in a valid format
-    if (typeof date === 'string') {
-      date = new Date(date);
-    }
-    // Compare the date with today's date
-    if (isNaN(date.getTime())) {
-      console.error('Invalid date:', date);
-      return false; // or handle the error as needed
-    }
-    // Create a new Date object for today at midnight
-    const todayMidnight = new Date();
-    todayMidnight.setHours(0, 0, 0, 0); // Set to midnight
-    const dateMidnight = new Date(date);
-    dateMidnight.setHours(0, 0, 0, 0); // Set to midnight
-    return dateMidnight <todayMidnight;
-  }
+  // isInOlderThanToday(date: Date): boolean {
+  //   // take the midnight of the date to compare ot today's midnight
+  //   if (!date) {
+  //     return false;
+  //   }
+  //   // Convert the date to a string to ensure it's in a valid format
+  //   if (typeof date === 'string') {
+  //     date = new Date(date);
+  //   }
+  //   // Compare the date with today's date
+  //   if (isNaN(date.getTime())) {
+  //     console.error('Invalid date:', date);
+  //     return false; // or handle the error as needed
+  //   }
+  //   // Create a new Date object for today at midnight
+  //   const todayMidnight = new Date();
+  //   todayMidnight.setHours(0, 0, 0, 0); // Set to midnight
+  //   const dateMidnight = new Date(date);
+  //   dateMidnight.setHours(0, 0, 0, 0); // Set to midnight
+  //   return dateMidnight <todayMidnight;
+  // }
 
   getStopTimeDisplay(startedAt: string | null, stoppedAt: string | null): string {
-    if (!startedAt || !stoppedAt) return '';
+    if (!startedAt || !stoppedAt) {
+      return '';
+    }
     const startDate = new Date(startedAt);
     const stopDate = new Date(stoppedAt);
     if (