fix: TT-282 added conditions in method onSubmit and added array of activities in component project-list-hover

Author: kevinjlope

src/app/modules/time-clock/components/entry-fields/entry-fields.component.html

@@ -7,6 +7,7 @@
       formControlName="activity_id"
       [class.is-invalid]="activity_id.invalid && activity_id.touched"
       required>
+      <option value="" selected="selected"></option>
       <option *ngFor="let activity of activities" value="{{ activity.id }}">{{ activity.name }}</option>
     </select>
   </div>

src/app/modules/time-clock/components/entry-fields/entry-fields.component.spec.ts

@@ -323,10 +323,12 @@ describe('EntryFieldsComponent', () => {
   });
 
   it('dispatches an action when onSubmit is called', () => {
+    const isMemberOf = spyOn(component, 'entryFormIsValidate').and.returnValue(true);
     spyOn(store, 'dispatch');
 
     component.onSubmit();
 
+    expect(isMemberOf).toHaveBeenCalled();
     expect(store.dispatch).toHaveBeenCalled();
   });
 

src/app/modules/time-clock/components/entry-fields/entry-fields.component.ts

@@ -1,8 +1,7 @@
-import { AzureAdB2CService } from './../../../login/services/azure.ad.b2c.service';
 import { FeatureToggleGeneralService } from './../../../shared/feature-toggles/feature-toggle-general/feature-toggle-general.service';
 import { ActivityManagementActionTypes } from './../../../activities-management/store/activity-management.actions';
 import { EntryActionTypes, LoadActiveEntry, UpdateCurrentOrLastEntry, UpdateEntry, UpdateEntryRunning } from './../../store/entry.actions';
-import { filter, map } from 'rxjs/operators';
+import { filter} from 'rxjs/operators';
 import { Component, OnDestroy, OnInit } from '@angular/core';
 import { FormBuilder, FormGroup } from '@angular/forms';
 import { Store, ActionsSubject, select } from '@ngrx/store';
@@ -17,8 +16,7 @@ import { ToastrService } from 'ngx-toastr';
 import { formatDate } from '@angular/common';
 import { getTimeEntriesDataSource } from '../../store/entry.selectors';
 import { DATE_FORMAT } from 'src/environments/environment';
-import { Subscription, Observable } from 'rxjs';
-import { FeatureManagerService } from './../../../shared/feature-toggles/feature-toggle-manager.service';
+import { Subscription,  } from 'rxjs';
 import { FeatureToggle } from './../../../../../environments/enum';
 import { CookieService } from 'ngx-cookie-service';
 
@@ -42,7 +40,6 @@ export class EntryFieldsComponent implements OnInit, OnDestroy {
   actionSetDateSubscription: Subscription;
   isCookieFeatureToggleActive: boolean;
   isFeatureToggleActive: boolean;
-
   constructor(
     private formBuilder: FormBuilder,
     private store: Store<Merged>,
@@ -81,7 +78,6 @@ export class EntryFieldsComponent implements OnInit, OnDestroy {
         this.isFeatureToggleActive = flag;
       });
     }
-
     this.loadActiveEntrySubscription = this.actionsSubject$
       .pipe(
         filter(
@@ -143,7 +139,9 @@ export class EntryFieldsComponent implements OnInit, OnDestroy {
   }
 
   onSubmit() {
-    this.store.dispatch(new entryActions.UpdateEntryRunning({ ...this.newData, ...this.entryForm.value }));
+    if (this.entryFormIsValidate()){
+      this.store.dispatch(new entryActions.UpdateEntryRunning({ ...this.newData, ...this.entryForm.value }));
+    }
   }
 
   onUpdateStartHour() {

src/app/modules/time-clock/components/project-list-hover/project-list-hover.component.spec.ts

@@ -39,8 +39,8 @@ describe('ProjectListHoverComponent', () => {
       isLoading: false,
       message: '',
     },
-  };
 
+  };
   beforeEach(
     waitForAsync(() => {
       TestBed.configureTestingModule({
@@ -69,6 +69,12 @@ describe('ProjectListHoverComponent', () => {
 
   it('dispatchs a CreateEntry action on clockIn', () => {
     component.activeEntry = null;
+    const activitiesMock  = [{
+      id: 'xyz',
+      name: 'test',
+      description : 'test1'
+    }];
+    component.activities = activitiesMock;
     spyOn(store, 'dispatch');
 
     component.clockIn(1, 'customer', 'project');

src/app/modules/time-clock/components/project-list-hover/project-list-hover.component.ts

@@ -14,6 +14,10 @@ import {
 } from './../../../customer-management/components/projects/components/store/project.selectors';
 import { EntryActionTypes } from './../../store/entry.actions';
 import { getActiveTimeEntry } from './../../store/entry.selectors';
+import { Activity, } from '../../../shared/models';
+import { LoadActivities } from './../../../activities-management/store/activity-management.actions';
+import { allActivities } from 'src/app/modules/activities-management/store/activity-management.selectors';
+
 @Component({
   selector: 'app-project-list-hover',
   templateUrl: './project-list-hover.component.html',
@@ -22,13 +26,15 @@ import { getActiveTimeEntry } from './../../store/entry.selectors';
 export class ProjectListHoverComponent implements OnInit, OnDestroy {
   keyword = 'search_field';
   listProjects: Project[] = [];
+  activities: Activity[] = [];
   activeEntry;
   projectsForm: FormGroup;
   showClockIn: boolean;
   updateEntrySubscription: Subscription;
   isLoading$: Observable<boolean>;
   projectsSubscription: Subscription;
   activeEntrySubscription: Subscription;
+  loadActivitiesSubscription: Subscription;
 
   constructor(
     private formBuilder: FormBuilder,
@@ -52,6 +58,11 @@ export class ProjectListHoverComponent implements OnInit, OnDestroy {
       });
       this.loadActiveTimeEntry();
     });
+    this.store.dispatch(new LoadActivities());
+    const activities$ = this.store.pipe(select(allActivities));
+    activities$.subscribe((response) => {
+      this.activities = response;
+    });
     this.updateEntrySubscription = this.actionsSubject$
       .pipe(filter((action: any) => action.type === EntryActionTypes.UPDATE_ENTRY_SUCCESS))
       .subscribe((action) => {
@@ -88,6 +99,7 @@ export class ProjectListHoverComponent implements OnInit, OnDestroy {
       start_date: new Date().toISOString(),
       timezone_offset: new Date().getTimezoneOffset(),
       technologies: [],
+      activity_id: this.activities[0].id,
     };
     this.store.dispatch(new entryActions.ClockIn(entry));
     this.projectsForm.setValue({ project_id: `${customerName} - ${name}` });