wfprev 795

Author: yzlucas

client/wfprev-war/src/main/angular/src/app/components/edit-project/endorsement-approval/endorsement-approval.component.spec.ts

@@ -234,4 +234,18 @@ describe('EndorsementApprovalComponent', () => {
     component.fiscal = { ...mockFiscal, planFiscalStatusCode: { planFiscalStatusCode: FiscalStatuses.PREPARED } };
     expect(component.showDraftTooltip).toBeFalse();
   });
+
+  it('should reset isSaving when isSaving change goes from true to false', () => {
+    component.isSaving = true;
+    component.ngOnChanges({
+      isSaving: {
+        currentValue: false,
+        previousValue: true,
+        firstChange: false,
+        isFirstChange: () => false,
+      }
+    });
+
+    expect(component.isSaving).toBeFalse();
+  });
 });

client/wfprev-war/src/main/angular/src/app/components/edit-project/project-details/project-details.component.spec.ts

@@ -704,6 +704,17 @@ describe('ProjectDetailsComponent', () => {
         expect(mockProjectService.getProjectByProjectGuid).not.toHaveBeenCalled();
       });
 
+      it('should reset isSaving after updateProject success', () => {
+        component.onSaveProjectDescription();
+        expect(component.isSaving).toBeFalse();
+      });
+
+      it('should reset isSaving after updateProject error', () => {
+        mockProjectService.updateProject.and.returnValue(throwError(() => new Error('fail')));
+        component.onSaveProjectDescription();
+        expect(component.isSaving).toBeFalse();
+      });
+
     });
 
     describe('onSaveLatLong Method', () => {
@@ -721,6 +732,7 @@ describe('ProjectDetailsComponent', () => {
 
         expect(component.isLatLongDirty).toBeFalse();
         expect(component.projectDetail.latitude).toBeGreaterThan(0);
+        expect(component.isSaving).toBeFalse();
       });
     });
 
@@ -803,6 +815,7 @@ describe('ProjectDetailsComponent', () => {
           fireCentreOrgUnitId: 123,
         })
       );
+      expect(component.isSaving).toBeFalse();
     });
 
     describe('refreshFiscalData Method', () => {
@@ -1151,6 +1164,7 @@ describe('ProjectDetailsComponent', () => {
       'OK',
       jasmine.objectContaining({ panelClass: 'snackbar-error' })
     );
+    expect(component.isSaving).toBeFalse();
   });
 
   describe('project description input/paste', () => {