fix: #203 Update-customer-for-active-clock-ins

Author: macrisguncay

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

@@ -6,6 +6,7 @@ import { ProjectListHoverComponent } from './project-list-hover.component';
 import { ProjectState } from '../../../customer-management/components/projects/components/store/project.reducer';
 import { getCustomerProjects } from '../../../customer-management/components/projects/components/store/project.selectors';
 import { FilterProjectPipe } from '../../../shared/pipes';
+import { CreateEntry, UpdateActiveEntry } from '../../store/entry.actions';
 
 describe('ProjectListHoverComponent', () => {
   let component: ProjectListHoverComponent;
@@ -53,11 +54,36 @@ describe('ProjectListHoverComponent', () => {
     expect(component).toBeTruthy();
   });
 
-  it('clock-in dispatchs a new action', () => {
+  it('clock-in dispatchs a CreateEntry action', () => {
+    const entry = {
+      project_id: '2b87372b-3d0d-4dc0-832b-ae5863cd39e5',
+      start_date: new Date().toISOString(),
+    };
+
+    component.activeEntry = null;
+    spyOn(store, 'dispatch');
+
+    component.clockIn('2b87372b-3d0d-4dc0-832b-ae5863cd39e5');
+
+    expect(store.dispatch).toHaveBeenCalledWith(new CreateEntry(entry));
+  });
+
+  it('clock-in dispatchs a UpdateActiveEntry action', () => {
+    const entry = {
+      id: '123',
+      project_id: '2b87372b-3d0d-4dc0-832b-ae5863cd39e5',
+      start_date: new Date().toISOString(),
+    };
+    const updatedEntry = {
+      id: '123',
+      project_id: '123372b-3d0d-4dc0-832b-ae5863cd39e5',
+    };
+
+    component.activeEntry = entry;
     spyOn(store, 'dispatch');
 
-    component.clockIn('id');
+    component.clockIn('123372b-3d0d-4dc0-832b-ae5863cd39e5');
 
-    expect(store.dispatch).toHaveBeenCalled();
+    expect(store.dispatch).toHaveBeenCalledWith(new UpdateActiveEntry(updatedEntry));
   });
 });

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

@@ -21,6 +21,7 @@ export class ProjectListHoverComponent implements OnInit {
   showButton = '';
   keyword = 'name';
   nameActiveProject: string;
+  activeEntry;
 
   constructor(private store: Store<ProjectState>) { }
 
@@ -32,13 +33,13 @@ export class ProjectListHoverComponent implements OnInit {
       this.listProjects = projects;
       this.loadActiveTimeEntry();
     });
-
   }
 
   private loadActiveTimeEntry() {
     this.store.dispatch(new entryActions.LoadActiveEntry());
     const activeEntry$ = this.store.pipe(select(getActiveTimeEntry));
     activeEntry$.subscribe((activeEntry) => {
+      this.activeEntry = activeEntry;
       if (activeEntry) {
         for (const project of this.listProjects) {
           if (project.id === activeEntry.project_id) {
@@ -53,7 +54,12 @@ export class ProjectListHoverComponent implements OnInit {
   }
 
   clockIn(id: string) {
-    const newEntry = { project_id: id, start_date: new Date().toISOString() };
-    this.store.dispatch(new entryActions.CreateEntry(newEntry));
+    if (this.activeEntry) {
+      const entry = {id: this.activeEntry.id, project_id: id};
+      this.store.dispatch(new entryActions.UpdateActiveEntry(entry));
+    } else {
+      const newEntry = { project_id: id, start_date: new Date().toISOString() };
+      this.store.dispatch(new entryActions.CreateEntry(newEntry));
+    }
   }
 }