Mirroristas/CommunitySolidServer
2
0
Fork 0
mirror of https://github.com/CommunitySolidServer/CommunitySolidServer.git synced 2024-10-03 14:55:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor: Allow IfNeededConverter to short-circuit.

Browse Source
This commit is contained in:
Ruben Verborgh 2021-01-19 21:55:50 +01:00
parent dd9d873122
commit a21532ebf8
3 changed files with 35 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
config/presets/representation-conversion.json
View File

@ -79,24 +79,24 @@
{
"@id": "urn:solid-server:default:RepresentationConverter",
"@type": "IfNeededConverter",
"IfNeededConverter:_converter": {
"@type": "WaterfallHandler",
"WaterfallHandler:_handlers": [
{
"@id": "urn:solid-server:default:ContentTypeReplacer"
},
{
"@id": "urn:solid-server:default:RdfToQuadConverter"
},
{
"@id": "urn:solid-server:default:QuadToRdfConverter"
},
{
"@id": "urn:solid-server:default:RdfRepresentationConverter"
}
]
}
"@type": "WaterfallHandler",
"WaterfallHandler:_handlers": [
{
"@type": "IfNeededConverter"
},
{
"@id": "urn:solid-server:default:ContentTypeReplacer"
},
{
"@id": "urn:solid-server:default:RdfToQuadConverter"
},
{
"@id": "urn:solid-server:default:QuadToRdfConverter"
},
{
"@id": "urn:solid-server:default:RdfRepresentationConverter"
}
]
}
]
}

5
src/storage/conversion/IfNeededConverter.ts
View File

@ -1,10 +1,13 @@
import type { Representation } from '../../ldp/representation/Representation';
import { getLoggerFor } from '../../logging/LogUtil';
import { InternalServerError } from '../../util/errors/InternalServerError';
import { UnsupportedAsyncHandler } from '../../util/UnsupportedAsyncHandler';
import { matchingMediaTypes } from './ConversionUtil';
import { RepresentationConverter } from './RepresentationConverter';
import type { RepresentationConverterArgs } from './RepresentationConverter';
const EMPTY_CONVERTER = new UnsupportedAsyncHandler('The content type does not match the preferences');
/**
* A {@link RepresentationConverter} that only converts representations
* that are not compatible with the preferences.
@ -13,7 +16,7 @@ export class IfNeededConverter extends RepresentationConverter {
private readonly converter: RepresentationConverter;
protected readonly logger = getLoggerFor(this);
public constructor(converter: RepresentationConverter) {
public constructor(converter: RepresentationConverter = EMPTY_CONVERTER) {
super();
this.converter = converter;
}

13
test/unit/storage/conversion/IfNeededConverter.test.ts
View File

@ -100,4 +100,17 @@ describe('An IfNeededConverter', (): void => {
expect(innerConverter.handle).toHaveBeenCalledTimes(1);
expect(innerConverter.handle).toHaveBeenCalledWith(args);
});
it('does not support conversion when there is no inner converter.', async(): Promise<void> => {
const emptyConverter = new IfNeededConverter();
const preferences = { type: { 'text/turtle': 0 }};
const args = { identifier, representation, preferences };
await expect(emptyConverter.canHandle(args)).rejects
.toThrow('The content type does not match the preferences');
await expect(emptyConverter.handle(args)).rejects
.toThrow('The content type does not match the preferences');
await expect(emptyConverter.handleSafe(args)).rejects
.toThrow('The content type does not match the preferences');
});
});