fix tests and lint issues

This commit is contained in:
Art Lowel
2020-04-28 16:35:51 +02:00
parent e4c4a1ecd4
commit 673926c36b
5 changed files with 9 additions and 7 deletions

View File

@@ -73,6 +73,7 @@ import { of as observableOf } from 'rxjs';
import { createSuccessfulRemoteDataObject } from '../../../testing/utils';
import { FormService } from '../../form.service';
import { SubmissionService } from '../../../../submission/submission.service';
import { FormBuilderService } from '../form-builder.service';
describe('DsDynamicFormControlContainerComponent test suite', () => {
@@ -180,6 +181,7 @@ describe('DsDynamicFormControlContainerComponent test suite', () => {
{ provide: RelationshipService, useValue: {} },
{ provide: SelectableListService, useValue: {} },
{ provide: FormService, useValue: {} },
{ provide: FormBuilderService, useValue: {} },
{ provide: SubmissionService, useValue: {} },
{
provide: SubmissionObjectDataService,

View File

@@ -390,7 +390,6 @@ export class DsDynamicFormControlContainerComponent extends DynamicFormControlCo
modalComp.item = this.item;
modalComp.collection = this.collection;
modalComp.submissionId = this.model.submissionId;
modalComp.selectEvent
}
/**

View File

@@ -61,7 +61,6 @@ export abstract class Reorderable {
}
}
/**
* Represents a single existing relationship value as metadata in submission
*/
@@ -118,7 +117,7 @@ export class ExistingRelationListElementComponent implements OnInit, OnChanges,
* Removes the selected relationship from the list
*/
removeSelection() {
this.selectableListService.deselectSingle(this.listId, Object.assign(new ItemSearchResult(), { indexableObject: this.relatedItem$ }));
this.selectableListService.deselectSingle(this.listId, Object.assign(new ItemSearchResult(), { indexableObject: this.relatedItem$.getValue() }));
this.store.dispatch(new RemoveRelationshipAction(this.submissionItem, this.relatedItem$.getValue(), this.relationshipOptions.relationshipType, this.submissionId));
}

View File

@@ -665,7 +665,7 @@ describe('SectionFormOperationsService test suite', () => {
spyOn(service, 'getFieldPathFromEvent').and.returnValue('path/1');
spyOn(service, 'getFieldPathSegmentedFromChangeEvent').and.returnValue('path');
spyOn(service, 'getFieldValueFromChangeEvent').and.returnValue(new FormFieldMetadataValueObject('test'));
spyOn(service, 'getArrayIndexFromEvent').and.returnValue(1);
spyOn(service, 'getArrayIndexFromEvent').and.returnValue(0);
spyOn(serviceAsAny, 'getValueMap');
spyOn(serviceAsAny, 'dispatchOperationsFromMap');
formBuilderService.isQualdropGroup.and.returnValue(false);
@@ -676,8 +676,10 @@ describe('SectionFormOperationsService test suite', () => {
serviceAsAny.dispatchOperationsFromChangeEvent(pathCombiner, event, previousValue, false);
expect(jsonPatchOpBuilder.add).toHaveBeenCalledWith(
pathCombiner.getPath('path/1'),
new FormFieldMetadataValueObject('test'));
pathCombiner.getPath('path'),
new FormFieldMetadataValueObject('test'),
true
);
});
});

View File

@@ -339,7 +339,7 @@ describe('SubmissionSectionformComponent test suite', () => {
comp.updateForm(sectionData, parsedSectionErrors);
expect(comp.initForm).not.toHaveBeenCalled();
expect(comp.initForm).toHaveBeenCalled();
expect(comp.checksForErrors).toHaveBeenCalled();
expect(comp.sectionData.data).toEqual(sectionData);
});