71806: Curation tasks UI

src/app/curation-form/curation-form.component.spec.ts

@@ -0,0 +1,148 @@
+import { async, ComponentFixture, TestBed } from '@angular/core/testing';
+import { TranslateModule } from '@ngx-translate/core';
+import { CUSTOM_ELEMENTS_SCHEMA } from '@angular/core';
+import { CurationFormComponent } from './curation-form.component';
+import { ScriptDataService } from '../core/data/processes/script-data.service';
+import { ProcessDataService } from '../core/data/processes/process-data.service';
+import { AuthService } from '../core/auth/auth.service';
+import { of as observableOf } from 'rxjs';
+import { RequestEntry } from '../core/data/request.reducer';
+import { DSOSuccessResponse, RestResponse } from '../core/cache/response.models';
+import { Process } from '../process-page/processes/process.model';
+import { createSuccessfulRemoteDataObject$ } from '../shared/remote-data.utils';
+import { EPerson } from '../core/eperson/models/eperson.model';
+import { NotificationsServiceStub } from '../shared/testing/notifications-service.stub';
+import { RouterStub } from '../shared/testing/router.stub';
+import { NotificationsService } from '../shared/notifications/notifications.service';
+import { Router } from '@angular/router';
+import { FormsModule, ReactiveFormsModule } from '@angular/forms';
+import { By } from '@angular/platform-browser';
+
+describe('CurationFormComponent', () => {
+  let comp: CurationFormComponent;
+  let fixture: ComponentFixture<CurationFormComponent>;
+
+  let scriptDataService: ScriptDataService;
+  let processDataService: ProcessDataService;
+  let authService: AuthService;
+  let notificationsService;
+  let router;
+
+  const requestEntry = Object.assign(new RequestEntry(),
+    {response: new DSOSuccessResponse(['process-link'], 200, 'success')});
+  const failedRequestEntry = Object.assign(new RequestEntry(),
+    {response: new RestResponse(false, 400, 'Bad Request')});
+
+  const process = Object.assign(new Process(), {processId: 'process-id'});
+
+  beforeEach(async(() => {
+
+    scriptDataService = jasmine.createSpyObj('scriptDataService', {
+      invoke: observableOf(requestEntry)
+    });
+
+    processDataService = jasmine.createSpyObj('processDataService', {
+      findByHref: createSuccessfulRemoteDataObject$(process)
+    });
+
+    authService = jasmine.createSpyObj('authService', {
+      getAuthenticatedUserFromStore: observableOf(Object.assign(new EPerson(), {email: 'test@mail'}))
+    });
+
+    notificationsService = new NotificationsServiceStub();
+    router = new RouterStub();
+
+    TestBed.configureTestingModule({
+      imports: [TranslateModule.forRoot(), FormsModule, ReactiveFormsModule],
+      declarations: [CurationFormComponent],
+      providers: [
+        {provide: ScriptDataService, useValue: scriptDataService},
+        {provide: ProcessDataService, useValue: processDataService},
+        {provide: AuthService, useValue: authService},
+        {provide: NotificationsService, useValue: notificationsService},
+        {provide: Router, useValue: router},
+      ],
+      schemas: [CUSTOM_ELEMENTS_SCHEMA]
+    }).compileComponents();
+  }));
+
+  beforeEach(() => {
+    fixture = TestBed.createComponent(CurationFormComponent);
+    comp = fixture.componentInstance;
+
+    fixture.detectChanges();
+  });
+  describe('init', () => {
+    it('should initialise the comp and contain the different tasks', () => {
+      expect(comp).toBeDefined();
+
+      const elements = fixture.debugElement.queryAll(By.css('option'));
+      expect(elements.length).toEqual(3);
+      expect(elements[0].nativeElement.innerHTML).toContain('curation-task.task.profileformats.label');
+      expect(elements[1].nativeElement.innerHTML).toContain('curation-task.task.requiredmetadata.label');
+      expect(elements[2].nativeElement.innerHTML).toContain('curation-task.task.checklinks.label');
+    });
+  });
+  describe('hasHandleValue', () => {
+    it('should return true when a dsoHandle value was provided', () => {
+      comp.dsoHandle = 'some-handle';
+      fixture.detectChanges();
+
+      expect(comp.hasHandleValue()).toBeTrue();
+    });
+    it('should return false when no dsoHandle value was provided', () => {
+      expect(comp.hasHandleValue()).toBeFalse();
+    });
+  });
+  describe('submit', () => {
+    it('should submit the selected process and handle to the scriptservice and navigate to the corresponding process page', () => {
+      comp.dsoHandle = 'test-handle';
+      comp.submit();
+
+      expect(scriptDataService.invoke).toHaveBeenCalledWith('curate', [
+        {name: '-t', value: 'profileformats'},
+        {name: '-i', value: 'test-handle'},
+        {name: '-e', value: 'test@mail'},
+      ], []);
+      expect(notificationsService.success).toHaveBeenCalled();
+      expect(processDataService.findByHref).toHaveBeenCalledWith('process-link');
+      expect(router.navigate).toHaveBeenCalledWith(['/processes', 'process-id']);
+    });
+    it('should the selected process and handle to the scriptservice and stay on the page on error', () => {
+      (scriptDataService.invoke as jasmine.Spy).and.returnValue(observableOf(failedRequestEntry));
+
+      comp.dsoHandle = 'test-handle';
+      comp.submit();
+
+      expect(scriptDataService.invoke).toHaveBeenCalledWith('curate', [
+        {name: '-t', value: 'profileformats'},
+        {name: '-i', value: 'test-handle'},
+        {name: '-e', value: 'test@mail'},
+      ], []);
+      expect(notificationsService.error).toHaveBeenCalled();
+      expect(processDataService.findByHref).not.toHaveBeenCalled();
+      expect(router.navigate).not.toHaveBeenCalled();
+    });
+  });
+  it('should use the handle provided by the form when no dsoHandle is provided', () => {
+    comp.form.get('handle').patchValue('form-handle');
+
+    comp.submit();
+
+    expect(scriptDataService.invoke).toHaveBeenCalledWith('curate', [
+      {name: '-t', value: 'profileformats'},
+      {name: '-i', value: 'form-handle'},
+      {name: '-e', value: 'test@mail'},
+    ], []);
+  });
+  it('should use "all" when the handle provided by the form is empty and when no dsoHandle is provided', () => {
+
+    comp.submit();
+
+    expect(scriptDataService.invoke).toHaveBeenCalledWith('curate', [
+      {name: '-t', value: 'profileformats'},
+      {name: '-i', value: 'all'},
+      {name: '-e', value: 'test@mail'},
+    ], []);
+  });
+});