fix: TT-20 testing

Author: wobravo

src/app/modules/shared/components/details-fields/details-fields.component.spec.ts

@@ -513,15 +513,23 @@ describe('DetailsFieldsComponent', () => {
     expect(endDateInput.max).toEqual(expectedDate);
   });
 
-  const params = [
-    {formatDate: 'YYY-MM-DD'},
-    {dateFormatHour: 'DD/MM/YYYY HH:mm'},
-  ];
-  params.map((param) => {
-    fit('should return the difference if there is data in the time in and in the time out', () => {
-      const date = moment('2016-01-05');
-      expect(4).toEqual(component.getTimeDifference());
-    });
+  fit('should return the difference if there is data in the time in and in the time out', () => {
+    component.ngOnChanges();
+    const StartHour = '08:00';
+    const EndHour = '19:00';
+    const StartDate = '2021-04-14';
+    const EndDate = '2021-04-15';
+    const startDateInput: HTMLInputElement = fixture.debugElement.nativeElement.querySelector('#start_date');
+    const startHourInput: HTMLInputElement = fixture.debugElement.nativeElement.querySelector('#start_hour');
+    const endDateInput: HTMLInputElement = fixture.debugElement.nativeElement.querySelector('#end_date');
+    const endHourInput: HTMLInputElement = fixture.debugElement.nativeElement.querySelector('#end_hour');
+    startDateInput.value = StartDate;
+    startHourInput.value = StartHour;
+    endDateInput.value = EndDate;
+    endHourInput.value = EndHour;
+    spyOn(component, 'getTimeDifference');
+
+    expect(component.getTimeDifference).toHaveBeenCalled();
   });
 
   /*

src/app/modules/shared/components/details-fields/details-fields.component.ts

@@ -18,7 +18,7 @@ import { EntryActionTypes } from './../../../time-clock/store/entry.actions';
 import { SaveEntryEvent } from './save-entry-event';
 import { ProjectSelectedEvent } from './project-selected-event';
 import { get } from 'lodash';
-import { DATE_FORMAT, DATE_FORMAT_HOUR } from 'src/environments/environment';
+import { DATE_FORMAT } from 'src/environments/environment';
 import { TechnologiesComponent } from '../technologies/technologies.component';
 
 type Merged = TechnologyState & ProjectState & ActivityState & EntryState;
@@ -130,17 +130,15 @@ export class DetailsFieldsComponent implements OnChanges, OnInit {
     this.end_date.setValue($event);
   }
 
-  getTimeDifference(){
-    const startDate = moment(`${this.start_date.value} ${this.start_hour.value}`).format(DATE_FORMAT_HOUR);
-    const endDate = moment(`${this.end_date.value} ${this.end_hour.value}`).format(DATE_FORMAT_HOUR);
-    if (this.end_hour.value !== '00:00') {
-      const diffDate = moment(endDate, DATE_FORMAT_HOUR).diff(moment(startDate, DATE_FORMAT_HOUR));
+  getTimeDifference() {
+    const startDate = moment(`${this.start_date.value} ${this.start_hour.value}`);
+    const endDate = moment(`${this.end_date.value} ${this.end_hour.value}`);
+    if (startDate <= endDate) {
+      const diffDate = endDate.diff(startDate);
       const duration = moment.duration(diffDate);
-      const diferenceTime = Math.floor(duration.asHours()) + moment.utc(diffDate).format(':mm');
-      return diferenceTime;
-    } else {
-      return '0:00';
+      return moment.utc(duration.asMilliseconds()).format('HH:mm');
     }
+    return '00:00';
   }
 
   ngOnChanges(): void {

src/environments/environment.ts

@@ -18,7 +18,6 @@ export const STACK_EXCHANGE_ACCESS_TOKEN = keys.STACK_EXCHANGE_ACCESS_TOKEN;
 export const AZURE_APP_CONFIGURATION_CONNECTION_STRING = keys.AZURE_APP_CONFIGURATION_CONNECTION_STRING;
 export const DATE_FORMAT = 'yyyy-MM-dd';
 export const DATE_FORMAT_YEAR = 'YYYY-MM-DD';
-export const DATE_FORMAT_HOUR = 'DD/MM/YYYY HH:mm';
 export const GROUPS = {
   ADMIN: 'time-tracker-admin',
   TESTER: 'time-tracker-tester',