OZG-7473 extract method

Based on CR comment.

alfa-client/libs/admin/shared/src/lib/admin-save-button/admin-save-button.component.spec.ts

 import { ADMIN_FORMSERVICE } from '@admin-client/shared';
 import { NavigationService } from '@alfa-client/navigation-shared';
-import { AbstractFormService, createStateResource, isValidStateResource, StateResource } from '@alfa-client/tech-shared';
+import { AbstractFormService, createStateResource, isLoaded, StateResource } from '@alfa-client/tech-shared';
 import { dispatchEventFromFixture, getMockComponent, mock, Mock, MockEvent } from '@alfa-client/test-utils';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { faker } from '@faker-js/faker/.';
@@ -17,10 +17,10 @@ import { AdminSaveButtonComponent } from './admin-save-button.component';
 jest.mock('@alfa-client/tech-shared', () => {
   return {
     ...jest.requireActual('@alfa-client/tech-shared'),
-    isValidStateResource: jest.fn(),
+    isLoaded: jest.fn(),
   };
 });
-const isValidStateResourceMock: jest.Mock = isValidStateResource as jest.Mock;
+const isLoadedMock: jest.Mock = isLoaded as jest.Mock;
 
 describe('AdminSaveButtonComponent', () => {
   let component: AdminSaveButtonComponent;
@@ -61,7 +61,32 @@ describe('AdminSaveButtonComponent', () => {
     expect(component).toBeTruthy();
   });
 
+  describe('component', () => {
+    describe('on successful form submission', () => {
+      it('should navigate to success link path', () => {
+        const successLinkPath: string = faker.internet.url();
+        component.successLinkPath = successLinkPath;
+        isLoadedMock.mockReturnValue(true);
+
+        component._navigateOnSuccessfulSubmission(createStateResource(createDummyResource()));
+
+        expect(navigationService.navigate).toHaveBeenCalledWith(successLinkPath);
+      });
+
+      it('should NOT navigate', () => {
+        isLoadedMock.mockReturnValue(false);
+
+        component._navigateOnSuccessfulSubmission(createStateResource(createDummyResource()));
+
+        expect(navigationService.navigate).not.toHaveBeenCalled();
+      });
+    });
+
     describe('on submit', () => {
+      beforeEach(() => {
+        component._navigateOnSuccessfulSubmission = jest.fn();
+      });
+
       it('should call formService', () => {
         dispatchEventFromFixture(fixture, saveButton, MockEvent.CLICK);
 
@@ -75,22 +100,11 @@ describe('AdminSaveButtonComponent', () => {
       });
 
       it('should navigate on successful submit', () => {
-      isValidStateResourceMock.mockReturnValue(true);
-      const linkPath: string = faker.internet.url();
-      component.successLinkPath = linkPath;
-
         dispatchEventFromFixture(fixture, saveButton, MockEvent.CLICK);
         component.stateResource$.subscribe();
 
-      expect(navigationService.navigate).toHaveBeenCalledWith(linkPath);
+        expect(component._navigateOnSuccessfulSubmission).toHaveBeenCalledWith(stateResource);
       });
-
-    it('should NOT navigate on invalid state resource', () => {
-      isValidStateResourceMock.mockReturnValue(false);
-      dispatchEventFromFixture(fixture, saveButton, MockEvent.CLICK);
-      component.stateResource$.subscribe();
-
-      expect(navigationService.navigate).not.toHaveBeenCalled();
     });
   });
 

alfa-client/libs/admin/shared/src/lib/admin-save-button/admin-save-button.component.ts

 import { ADMIN_FORMSERVICE } from '@admin-client/shared';
 import { NavigationService } from '@alfa-client/navigation-shared';
-import { AbstractFormService, createEmptyStateResource, isValidStateResource, StateResource } from '@alfa-client/tech-shared';
+import { AbstractFormService, createEmptyStateResource, isLoaded, StateResource } from '@alfa-client/tech-shared';
 import { CommonModule } from '@angular/common';
 import { Component, inject, Input } from '@angular/core';
 import { Resource } from '@ngxp/rest';
@@ -24,10 +24,14 @@ export class AdminSaveButtonComponent {
   public submit(): void {
     this.stateResource$ = this.formService.submit().pipe(
       tap((stateResource: StateResource<Resource>) => {
-        if (isValidStateResource(stateResource)) {
-          this.navigationService.navigate(this.successLinkPath);
-        }
+        this._navigateOnSuccessfulSubmission(stateResource);
       }),
     );
   }
+
+  _navigateOnSuccessfulSubmission(stateResource: StateResource<Resource>) {
+    if (isLoaded(stateResource)) {
+      this.navigationService.navigate(this.successLinkPath);
+    }
+  }
 }