feat: TT-35 solve PR comments

Author: eduardisrael

src/app/modules/time-entries/pages/time-entries.component.html

@@ -61,7 +61,7 @@
 </div>
 <div class="modal fade" id="editRecordsByDate" tabindex="-1" role="dialog">
   <div class="modal-dialog modal-dialog-centered modal-lg" role="document">
-    <div class="modal-content" cdkDrag (cdkDragEnded)="dragEnded($event)">
+    <div class="modal-content" cdkDrag (cdkDragEnded)="resetDraggablePosition($event)">
       <div class="modal-header">
         <h5 class="modal-title">{{ entryId ? 'Edit Entry' : 'New Entry' }}</h5>
       </div>

src/app/modules/time-entries/pages/time-entries.component.spec.ts

@@ -443,10 +443,16 @@ describe('TimeEntriesComponent', () => {
     expect(component.entry).toEqual(newEntry);
   });
 
-  it('When I stop dragging the modal, it should call dragEnded', () => {
-    const dragEndEvent = new DragEvent('CdkDragEnd');
-    spyOn(component, 'dragEnded');
-    component.dragEnded(dragEndEvent);
-    expect(component.dragEnded).toHaveBeenCalledWith(dragEndEvent);
+  it('When I stop dragging the modal, it should reset Draggable Position', () => {
+    const dragEndEventStub = {
+      source: {
+        _dragRef: {
+          reset: () => { }
+        }
+      }
+    };
+    spyOn(dragEndEventStub.source._dragRef, 'reset');
+    component.resetDraggablePosition(dragEndEventStub);
+    expect(dragEndEventStub.source._dragRef.reset).toHaveBeenCalled();
   });
 });

src/app/modules/time-entries/pages/time-entries.component.ts

@@ -165,7 +165,7 @@ export class TimeEntriesComponent implements OnInit, OnDestroy {
     this.message = `Are you sure you want to delete ${item.activity_name}?`;
     this.showModal = true;
   }
-  dragEnded(event: any): void {
+  resetDraggablePosition(event: any): void {
     event.source._dragRef.reset();
   }
 }