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

change: improve getLoggerFactoryOrThrow error message

Browse Source
This commit is contained in:
Ruben Taelman 2020-09-23 09:30:33 +02:00 committed by Ruben Taelman
parent 4c534e9d5d
commit b62f4eb016
4 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/logging/LazyLoggerFactory.ts
View File

@ -34,7 +34,7 @@ export class LazyLoggerFactory implements LoggerFactory {
public getLoggerFactoryOrThrow(): LoggerFactory {
if (!this.loggerFactory) {
throw new Error('Illegal logging during initialization');
throw new Error('No logger factory has been set yet. Can be caused logger invocation during initialization.');
}
return this.loggerFactory;
}

2
test/unit/logging/LazyLogger.test.ts
View File

@ -12,7 +12,7 @@ describe('LazyLogger', (): void => {
it('throws when no logger factory is set in the lazy logger factory.', async(): Promise<void> => {
expect((): any => logger.log('debug', 'my message', { abc: true }))
.toThrow(new Error('Illegal logging during initialization'));
.toThrow(new Error('No logger factory has been set yet. Can be caused logger invocation during initialization.'));
});
it('creates a new logger using the factory.', async(): Promise<void> => {

4
test/unit/logging/LazyLoggerFactory.test.ts
View File

@ -31,7 +31,7 @@ describe('LazyLoggerFactory', (): void => {
it('throws when retrieving the inner factory if none has been set.', async(): Promise<void> => {
expect((): any => LazyLoggerFactory.getInstance().getLoggerFactoryOrThrow())
.toThrow(new Error('Illegal logging during initialization'));
.toThrow(new Error('No logger factory has been set yet. Can be caused logger invocation during initialization.'));
});
it('Returns the inner factory if one has been set.', async(): Promise<void> => {
@ -60,6 +60,6 @@ describe('LazyLoggerFactory', (): void => {
it('errors on invoking LazyLoggers if a factory has not been set yet.', async(): Promise<void> => {
const logger = LazyLoggerFactory.getInstance().createLogger('MyLabel');
expect((): any => logger.log('debug', 'my message', { abc: true }))
.toThrow(new Error('Illegal logging during initialization'));
.toThrow(new Error('No logger factory has been set yet. Can be caused logger invocation during initialization.'));
});
});

2
test/unit/logging/LogUtil.test.ts
View File

@ -28,6 +28,6 @@ describe('LogUtil', (): void => {
expect(setGlobalLoggerFactory(new VoidLoggerFactory()));
expect(setGlobalLoggerFactory(undefined));
expect((): any => LazyLoggerFactory.getInstance().getLoggerFactoryOrThrow())
.toThrow(new Error('Illegal logging during initialization'));
.toThrow(new Error('No logger factory has been set yet. Can be caused logger invocation during initialization.'));
});
});