WIP

src/multiLogger.ts

@@ -1,8 +1,7 @@
 import {Log, Logger} from './logger';
-import {extractErrorMessage} from './utilities';
 
 /**
- * A logger that stores all logs in memory.
+ * A logger send logs to multiple loggers.
  */
 export class MultiLogger<T extends Log = Log> implements Logger<T> {
     private readonly loggers: Array<Logger<T>> = [];
@@ -12,18 +11,8 @@ export class MultiLogger<T extends Log = Log> implements Logger<T> {
     }
 
     public log(log: T): void {
-        const errors: Error[] = [];
-
         for (const logger of this.loggers) {
-            try {
-                logger.log(log);
-            } catch (error) {
-                errors.push(error);
-            }
-        }
-
-        if (errors.length > 0) {
-            throw new Error(`[${errors.map(error => extractErrorMessage(error))}]`);
+            logger.log(log);
         }
     }
 }

test/filteredLogger.test.ts

@@ -38,10 +38,10 @@ describe('A filtered logger', () => {
 
     it('should filter the specified levels', () => {
         const logger = new InMemoryLogger();
-        const suppressedLogger = new FilteredLogger(logger, [LogLevel.ERROR, LogLevel.DEBUG]);
+        const filteredLogger = new FilteredLogger(logger, [LogLevel.ERROR, LogLevel.DEBUG]);
 
         for (const log of logs) {
-            suppressedLogger.log(log);
+            filteredLogger.log(log);
         }
 
         expect(logger.getLogs()).toStrictEqual([debugLog, errorLog]);

test/multiLogger.test.ts

@@ -1,7 +1,7 @@
 import {InMemoryLogger, Log, LogLevel} from '../src';
 import {MultiLogger} from '../src/multiLogger';
 
-describe('An in-memory logger', () => {
+describe('A multi logger', () => {
     const logs: Log[] = [
         {
             level: LogLevel.DEBUG,
@@ -45,21 +45,4 @@ describe('An in-memory logger', () => {
         expect(firstLogger.getLogs()).toEqual(logs);
         expect(secondLogger.getLogs()).toEqual(logs);
     });
-
-    it('should throw an error with underlying fail messages', () => {
-        const firstLogger = new InMemoryLogger();
-        const secondLogger = new InMemoryLogger();
-        const multiLogger = new MultiLogger([firstLogger, secondLogger]);
-
-        const message = 'Some error';
-
-        jest.spyOn(firstLogger, 'log').mockImplementation(() => { throw new Error(message); });
-
-        for (const log of logs) {
-            expect(() => multiLogger.log(log)).toThrow(`${[message]}`);
-        }
-
-        expect(firstLogger.getLogs()).toEqual([]);
-        expect(secondLogger.getLogs()).toEqual(logs);
-    });
 });