fix: TT-477 customer change status (#787)

* fix: TT-477 customer change status

* refactor: TT-477 change of name of variable

* refactor: TT-477 based on scastillo-jp comments

Includes fix of a typo

Author: Davcg5

src/app/modules/customer-management/components/customer-info/components/customer-list/customer-list.component.html

@@ -20,9 +20,9 @@
         <td class="col-4 text-break">{{ customer.id }}</td>
         <td class="col-4 text-break">{{ customer.name }}</td>
         <td class="col-2 text-center">
-          <button 
-            data-toggle="modal" 
-            (click)="editCustomer(customer.id)" 
+          <button
+            data-toggle="modal"
+            (click)="editCustomer(customer.id)"
             type="button" data-target="#editModal"
             class="btn btn-sm btn-primary">
             <i class="fa fa-pen fa-xs"></i>
@@ -48,7 +48,7 @@
   aria-hidden="true"
   [title]="'Disable Customer'"
   [body]="message"
-  (closeModalEvent)="deleteCustomer()"
+  (closeModalEvent)="changeStatus()"
 >
 </app-dialog>
 

src/app/modules/customer-management/components/customer-info/components/customer-list/customer-list.component.spec.ts

@@ -277,6 +277,13 @@ describe('CustomerTableListComponent', () => {
     expect(component.showModal).toBeFalse();
   });
 
+
+  it('changeStatus should set inactive when active', () => {
+    component.changeStatus();
+    expect(component.statusToEdit === 'inactive');
+  });
+
+
   afterEach(() => {
     component.dtTrigger.unsubscribe();
     component.changeCustomerSubscription.unsubscribe();

src/app/modules/customer-management/components/customer-info/components/customer-list/customer-list.component.ts

@@ -40,6 +40,7 @@ export class CustomerListComponent implements OnInit, OnDestroy, AfterViewInit {
   showModal = false;
   idToDelete: string;
   idToEdit: string;
+  statusToEdit: string;
   currentCustomerIdToEdit: string;
   message: string;
   isLoading$: Observable<boolean>;
@@ -170,20 +171,30 @@ export class CustomerListComponent implements OnInit, OnDestroy, AfterViewInit {
 
   openModal(item: Customer) {
     this.idToDelete = item.id;
+    this.statusToEdit = item.status;
     this.message = `Are you sure you want to disable ${item.name}?`;
     this.showModal = true;
   }
 
   switchStatus(item: CustomerUI): void {
+
     if (item.key !== 'inactive') {
       this.openModal(item);
     } else {
       this.showModal = false;
-      this.store.dispatch(new UnarchiveCustomer(item.id));
+      this.store.dispatch(new UnarchiveCustomer(item.id, this.changeOppositeStatus(item.key)));
     }
   }
 
   setActive(status: any): string {
     return status === 'inactive' ? 'inactive' : 'active';
   }
+  changeOppositeStatus(status: string): string {
+    return status === 'inactive' ? 'active' : 'inactive';
+  }
+
+  changeStatus(): void {
+    this.store.dispatch(new UnarchiveCustomer(this.idToDelete, this.changeOppositeStatus(this.statusToEdit)));
+  }
+
 }

src/app/modules/customer-management/store/customer-management.actions.spec.ts

@@ -79,7 +79,7 @@ describe('CustomerManagmentActions', () => {
   });
 
   it('UnarchiveCustomer type is CustomerManagementActionTypes.UNARCHIVE_CUSTOMER', () => {
-    const unArchiveCustomer = new actions.UnarchiveCustomer('id_test');
+    const unArchiveCustomer = new actions.UnarchiveCustomer('id_test', 'active');
     expect(unArchiveCustomer.type).toEqual(actions.CustomerManagementActionTypes.UNARCHIVE_CUSTOMER);
   });
 

src/app/modules/customer-management/store/customer-management.actions.ts

@@ -103,7 +103,7 @@ export class ResetCustomerToEdit implements Action {
 export class UnarchiveCustomer implements Action {
   public readonly type = CustomerManagementActionTypes.UNARCHIVE_CUSTOMER;
 
-  constructor(public payload: string) {}
+  constructor(public payload: string, public status: string) {}
 }
 
 export class UnarchiveCustomerSuccess implements Action {

src/app/modules/customer-management/store/customer-management.effects.ts

@@ -93,8 +93,9 @@ export class CustomerEffects {
   unarchiveCustomer$: Observable<Action> = this.actions$.pipe(
     ofType(actions.CustomerManagementActionTypes.UNARCHIVE_CUSTOMER),
     map((action: actions.UnarchiveCustomer) => ({
+
       id: action.payload,
-      status: 'active',
+      status: action.status,
     })),
     mergeMap((customer: Status) =>
       this.customerService.updateCustomer(customer).pipe(

src/app/modules/customer-management/store/customer-management.reducers.spec.ts

@@ -135,7 +135,7 @@ describe('customerManagementReducer', () => {
   });
 
   it('on UnarchiveCustomer, isLoading is true', () => {
-    const action = new actions.UnarchiveCustomer('1');
+    const action = new actions.UnarchiveCustomer('1', 'inactive');
     const state = customerManagementReducer(initialState, action);
 
     expect(state.isLoading).toEqual(true);