fixes after merge with master

This commit is contained in:
lotte
2019-11-25 12:11:40 +01:00
parent 168e74d1fa
commit 23a378bd26
16 changed files with 19 additions and 22 deletions

View File

@@ -6,6 +6,7 @@ import { PaginatedList } from '../../../core/data/paginated-list';
import { FindAllOptions } from '../../../core/data/request.models';
import { ViewMode } from '../../../core/shared/view-mode.model';
import { RelationshipService } from '../../../core/data/relationship.service';
import { Subscription } from 'rxjs';
@Component({
selector: 'ds-related-items',

View File

@@ -15,6 +15,7 @@ import { ItemDataService } from '../../../../../core/data/item-data.service';
import { SelectableListService } from '../../../../../shared/object-list/selectable-list/selectable-list.service';
import { Store } from '@ngrx/store';
import { createSuccessfulRemoteDataObject$ } from '../../../../../shared/testing/utils';
import { PaginatedList } from '../../../../../core/data/paginated-list';
let personListElementComponent: PersonSearchResultListSubmissionElementComponent;
let fixture: ComponentFixture<PersonSearchResultListSubmissionElementComponent>;
@@ -23,7 +24,7 @@ const mockItemWithMetadata: ItemSearchResult = Object.assign(
new ItemSearchResult(),
{
indexableObject: Object.assign(new Item(), {
bitstreams: createSuccessfulRemoteDataObject$([]),
bundles: createSuccessfulRemoteDataObject$(new PaginatedList(undefined, [])),
metadata: {
'dc.title': [
{
@@ -44,7 +45,7 @@ const mockItemWithoutMetadata: ItemSearchResult = Object.assign(
new ItemSearchResult(),
{
indexableObject: Object.assign(new Item(), {
bitstreams: createSuccessfulRemoteDataObject$([]),
bundles: createSuccessfulRemoteDataObject$(new PaginatedList(undefined, [])),
metadata: {
'dc.title': [
{

View File

@@ -1,5 +1,5 @@
import { of as observableOf } from 'rxjs';
import { SearchService } from '../../+search-page/search-service/search.service';
import { SearchService } from '../../core/shared/search/search.service';
export function getMockSearchService(): SearchService {
return jasmine.createSpyObj('searchService', {

View File

@@ -17,11 +17,10 @@ import { ClaimedTaskActionsComponent } from './claimed-task-actions.component';
import { ClaimedTask } from '../../../core/tasks/models/claimed-task-object.model';
import { WorkflowItem } from '../../../core/submission/models/workflowitem.model';
import { createSuccessfulRemoteDataObject } from '../../testing/utils';
import { CoreModule } from '../../../core/core.module';
import { getMockSearchService } from '../../mocks/mock-search-service';
import { getMockRequestService } from '../../mocks/mock-request.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { SearchService } from '../../../core/shared/search/search.service';
let component: ClaimedTaskActionsComponent;
let fixture: ComponentFixture<ClaimedTaskActionsComponent>;

View File

@@ -13,8 +13,8 @@ import { WorkflowItem } from '../../../core/submission/models/workflowitem.model
import { RemoteData } from '../../../core/data/remote-data';
import { MyDSpaceActionsComponent } from '../mydspace-actions';
import { NotificationsService } from '../../notifications/notifications.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { SearchService } from '../../../core/shared/search/search.service';
/**
* This component represents actions related to ClaimedTask object.

View File

@@ -12,10 +12,10 @@ import { ItemActionsComponent } from './item-actions.component';
import { ItemDataService } from '../../../core/data/item-data.service';
import { NotificationsService } from '../../notifications/notifications.service';
import { NotificationsServiceStub } from '../../testing/notifications-service-stub';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { getMockSearchService } from '../../mocks/mock-search-service';
import { getMockRequestService } from '../../mocks/mock-request.service';
import { SearchService } from '../../../core/shared/search/search.service';
let component: ItemActionsComponent;
let fixture: ComponentFixture<ItemActionsComponent>;

View File

@@ -7,8 +7,8 @@ import { MyDSpaceActionsComponent } from '../mydspace-actions';
import { ItemDataService } from '../../../core/data/item-data.service';
import { Item } from '../../../core/shared/item.model';
import { NotificationsService } from '../../notifications/notifications.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { SearchService } from '../../../core/shared/search/search.service';
/**
* This component represents mydspace actions related to Item object.

View File

@@ -11,9 +11,9 @@ import { ResourceType } from '../../core/shared/resource-type';
import { NotificationOptions } from '../notifications/models/notification-options.model';
import { NotificationsService } from '../notifications/notifications.service';
import { TranslateService } from '@ngx-translate/core';
import { SearchService } from '../../+search-page/search-service/search.service';
import { RequestService } from '../../core/data/request.service';
import { Subscription } from 'rxjs';
import { SearchService } from '../../core/shared/search/search.service';
/**
* Abstract class for all different representations of mydspace actions
@@ -76,7 +76,7 @@ export abstract class MyDSpaceActionsComponent<T extends DSpaceObject, TService
// See https://github.com/DSpace/dspace-angular/pull/468
this.searchService.getEndpoint().pipe(
take(1),
tap((cachedHref) => this.requestService.removeByHrefSubstring(cachedHref))
tap((cachedHref: string) => this.requestService.removeByHrefSubstring(cachedHref))
).subscribe(() => this.router.navigateByUrl(url));
}

View File

@@ -18,9 +18,9 @@ import { PoolTask } from '../../../core/tasks/models/pool-task-object.model';
import { WorkflowItem } from '../../../core/submission/models/workflowitem.model';
import { createSuccessfulRemoteDataObject } from '../../testing/utils';
import { getMockRequestService } from '../../mocks/mock-request.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { getMockSearchService } from '../../mocks/mock-search-service';
import { SearchService } from '../../../core/shared/search/search.service';
let component: PoolTaskActionsComponent;
let fixture: ComponentFixture<PoolTaskActionsComponent>;

View File

@@ -13,8 +13,8 @@ import { PoolTaskDataService } from '../../../core/tasks/pool-task-data.service'
import { isNotUndefined } from '../../empty.util';
import { MyDSpaceActionsComponent } from '../mydspace-actions';
import { NotificationsService } from '../../notifications/notifications.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { SearchService } from '../../../core/shared/search/search.service';
/**
* This component represents mydspace actions related to PoolTask object.

View File

@@ -7,7 +7,6 @@ import { TranslateLoader, TranslateModule } from '@ngx-translate/core';
import { MockTranslateLoader } from '../../mocks/mock-translate-loader';
import { RouterStub } from '../../testing/router-stub';
import { RemoteData } from '../../../core/data/remote-data';
import { Item } from '../../../core/shared/item.model';
import { WorkflowItem } from '../../../core/submission/models/workflowitem.model';
import { WorkflowitemActionsComponent } from './workflowitem-actions.component';
@@ -16,9 +15,9 @@ import { NotificationsService } from '../../notifications/notifications.service'
import { NotificationsServiceStub } from '../../testing/notifications-service-stub';
import { createSuccessfulRemoteDataObject } from '../../testing/utils';
import { getMockRequestService } from '../../mocks/mock-request.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { getMockSearchService } from '../../mocks/mock-search-service';
import { SearchService } from '../../../core/shared/search/search.service';
let component: WorkflowitemActionsComponent;
let fixture: ComponentFixture<WorkflowitemActionsComponent>;

View File

@@ -7,8 +7,8 @@ import { MyDSpaceActionsComponent } from '../mydspace-actions';
import { WorkflowItem } from '../../../core/submission/models/workflowitem.model';
import { WorkflowItemDataService } from '../../../core/submission/workflowitem-data.service';
import { NotificationsService } from '../../notifications/notifications.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { SearchService } from '../../../core/shared/search/search.service';
/**
* This component represents actions related to WorkflowItem object.

View File

@@ -11,7 +11,6 @@ import { MockTranslateLoader } from '../../mocks/mock-translate-loader';
import { NotificationsService } from '../../notifications/notifications.service';
import { NotificationsServiceStub } from '../../testing/notifications-service-stub';
import { RouterStub } from '../../testing/router-stub';
import { RemoteData } from '../../../core/data/remote-data';
import { Item } from '../../../core/shared/item.model';
import { WorkspaceItem } from '../../../core/submission/models/workspaceitem.model';
import { WorkspaceitemActionsComponent } from './workspaceitem-actions.component';
@@ -19,8 +18,8 @@ import { WorkspaceitemDataService } from '../../../core/submission/workspaceitem
import { createSuccessfulRemoteDataObject } from '../../testing/utils';
import { RequestService } from '../../../core/data/request.service';
import { getMockRequestService } from '../../mocks/mock-request.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { getMockSearchService } from '../../mocks/mock-search-service';
import { SearchService } from '../../../core/shared/search/search.service';
let component: WorkspaceitemActionsComponent;
let fixture: ComponentFixture<WorkspaceitemActionsComponent>;

View File

@@ -9,8 +9,8 @@ import { WorkspaceItem } from '../../../core/submission/models/workspaceitem.mod
import { MyDSpaceActionsComponent } from '../mydspace-actions';
import { WorkspaceitemDataService } from '../../../core/submission/workspaceitem-data.service';
import { NotificationsService } from '../../notifications/notifications.service';
import { SearchService } from '../../../+search-page/search-service/search.service';
import { RequestService } from '../../../core/data/request.service';
import { SearchService } from '../../../core/shared/search/search.service';
/**
* This component represents actions related to WorkspaceItem object.

View File

@@ -37,18 +37,16 @@ import {
SaveSubmissionSectionFormAction,
SetActiveSectionAction
} from './objects/submission-objects.actions';
import { RemoteData } from '../core/data/remote-data';
import { RemoteDataError } from '../core/data/remote-data-error';
import { throwError as observableThrowError } from 'rxjs/internal/observable/throwError';
import {
createFailedRemoteDataObject,
createSuccessfulRemoteDataObject,
createSuccessfulRemoteDataObject$
} from '../shared/testing/utils';
import { getMockSearchService } from '../shared/mocks/mock-search-service';
import { getMockRequestService } from '../shared/mocks/mock-request.service';
import { SearchService } from '../+search-page/search-service/search.service';
import { RequestService } from '../core/data/request.service';
import { SearchService } from '../core/shared/search/search.service';
describe('SubmissionService test suite', () => {
const config = MOCK_SUBMISSION_CONFIG;