TTA-142: Alphabetically sort list of users on reports (#925)

Co-authored-by: Jimmy Jaramillo <[email protected]>

Author: jimmyjaramillo

src/app/modules/reports/components/time-entries-table/time-entries-table.component.spec.ts

@@ -212,10 +212,10 @@ describe('Reports Page', () => {
 
     it('the sume of hours of entries selected is equal to {hours:0, minutes:0, seconds:0}', () => {
       let checked = true;
-      let {hours, minutes, seconds}:TotalHours = component.sumHoursEntriesSelected(timeEntryList[0], checked);
+      let {hours, minutes, seconds}: TotalHours = component.sumHoursEntriesSelected(timeEntryList[0], checked);
       checked = false;
-      ({hours, minutes,seconds} = component.sumHoursEntriesSelected(timeEntryList[0], checked));
-      expect({hours, minutes, seconds}).toEqual({hours:0, minutes:0, seconds:0});
+      ({hours, minutes, seconds} = component.sumHoursEntriesSelected(timeEntryList[0], checked));
+      expect({hours, minutes, seconds}).toEqual({hours: 0, minutes: 0, seconds: 0});
     });
 
     it('should export data with the correct format', () => {

src/app/modules/reports/components/time-entries-table/time-entries-table.component.ts

@@ -69,7 +69,7 @@ export class TimeEntriesTableComponent implements OnInit, OnDestroy, AfterViewIn
       },
     ],
     columnDefs: [{ type: 'date', targets: 2}, {orderable: false, targets: [0]}],
-    order: [[1,'asc'],[2,'desc'],[4,'desc']]
+    order: [[1, 'asc'], [2, 'desc'], [4, 'desc']]
   };
   dtTrigger: Subject<any> = new Subject();
   @ViewChild(DataTableDirective, { static: false })
@@ -79,7 +79,7 @@ export class TimeEntriesTableComponent implements OnInit, OnDestroy, AfterViewIn
   rerenderTableSubscription: Subscription;
   resultSum: TotalHours;
   resultSumEntriesSelected: TotalHours;
-  resultSumEntriesSelected$:Observable<TotalHours>;
+  resultSumEntriesSelected$: Observable<TotalHours>;
   totalHoursSubscription: Subscription;
   dateTimeOffset: ParseDateTimeOffset;
 
@@ -96,7 +96,9 @@ export class TimeEntriesTableComponent implements OnInit, OnDestroy, AfterViewIn
     this.actionsSubject$
       .pipe(filter((action: any) => action.type === UserActionTypes.LOAD_USERS_SUCCESS))
       .subscribe((action) => {
-        this.users = action.payload;
+        const sortUsers = [...action.payload];
+        sortUsers.sort((a, b) => a.name.localeCompare(b.name));
+        this.users = sortUsers;
       });
   }