TT-39 fix: imporve coverage

Author: Guido Quezada

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

@@ -267,6 +267,16 @@ describe('DetailsFieldsComponent', () => {
     expect(endHourInput).toBeNull();
   });
 
+  it('when the current entry is not running, then the end_date and end_hour of the form should be set', () => {
+    const startDate = new Date('2020-12-30T09:45:00');
+    component.entryToEdit = { ...entryToEdit, start_date: startDate };
+    spyOn(component.entryForm, 'patchValue');
+
+    component.onGoingToWorkOnThisChange({ currentTarget: { checked: false } });
+
+    expect(component.entryForm.patchValue).toHaveBeenCalledWith( { end_date: '2020-12-30', end_hour: '09:45', } );
+  });
+
   it('when creating a new entry, then the new entry should be marked as not run', () => {
     component.entryToEdit = null;
 

src/app/modules/shared/components/month-picker/month-picker.component.ts

@@ -35,11 +35,11 @@ export class MonthPickerComponent implements OnInit {
 
   changeYear(changeAction: string) {
     this.selectedYearMoment[changeAction](1, 'year');
+    this.selectedYearText = moment(this.selectedYearMoment).format('YYYY');
     this.selectedYear = this.selectedYearMoment.year();
 
     const monthIndex = changeAction === 'add' ? 0 : 11;
     this.selectMonth(monthIndex);
-    this.selectedYearText = moment(this.selectedYearMoment).format('YYYY');
   }
 
   selectMonth(monthIndex: number) {