Merge pull request #339 from ioet/335-include-seconds-on-time-diffs

fix: #336 include seconds on time-entries list

Author: enriquezrene

src/app/modules/shared/pipes/substract-date/substract-date.pipe.spec.ts

@@ -7,12 +7,12 @@ describe('SubstractDatePipe', () => {
   });
 
   it('returns the date diff', () => {
-    const fromDate = new Date('2011-04-11T10:20:30Z');
+    const fromDate = new Date('2011-04-11T10:20:40Z');
     const substractDate = new Date('2011-04-11T08:00:30Z');
 
     const diff = new SubstractDatePipe().transform(fromDate, substractDate);
 
-    expect(diff).toBe('02:20');
+    expect(diff).toBe('02:20:10');
   });
 
   it('returns --:-- if fromDate is null', () => {

src/app/modules/shared/pipes/substract-date/substract-date.pipe.ts

@@ -14,7 +14,7 @@ export class SubstractDatePipe implements PipeTransform {
     const startDate = moment(substractDate, 'YYYY-MM-DD HH:mm:ss');
     const endDate = moment(fromDate, 'YYYY-MM-DD HH:mm:ss');
     const duration: any = moment.duration(endDate.diff(startDate));
-    return `${this.formatTime(duration._data.hours)}:${this.formatTime(duration._data.minutes)}`;
+    return `${this.formatTime(duration._data.hours)}:${this.formatTime(duration._data.minutes)}:${this.formatTime(duration._data.seconds)}`;
   }
 
   formatTime(time: number): string {