fix: Order of static keys to ensure default namespace key is availabl…

…e when creating the default logger
Sphereon-Opensource · Jun 13, 2024 · dc56df2 · dc56df2
1 parent eae66f2
commit dc56df2
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/packages/ssi-types/__tests__/logging.test.ts b/packages/ssi-types/__tests__/logging.test.ts
@@ -1,6 +1,13 @@
-import { Loggers, LogMethod } from '../src'
+import {Loggers, LogMethod} from '../src'
 
 describe('Simple logging', () => {
+
+  it('Should be able to log without providing options for a new namespace', () => {
+    Loggers.DEFAULT.get(''+ Date.now())
+        .log('Random test')
+  })
+
+
   it('Should perform a debug package log', () => {
     Loggers.DEFAULT.options('debug_pkg', { methods: [LogMethod.DEBUG_PKG] })
       .get('debug_pkg')

diff --git a/packages/ssi-types/src/logging/index.ts b/packages/ssi-types/src/logging/index.ts
@@ -46,8 +46,8 @@ export function logOptions(opts?: SimpleLogOptions): Required<SimpleLogOptions>
 }
 
 export class Loggers {
-  public static readonly DEFAULT: Loggers = new Loggers({ defaultLogLevel: LogLevel.INFO, methods: [LogMethod.DEBUG_PKG, LogMethod.EVENT] })
   private static readonly DEFAULT_KEY = '__DEFAULT__'
+  public static readonly DEFAULT: Loggers = new Loggers({ defaultLogLevel: LogLevel.INFO, methods: [LogMethod.DEBUG_PKG, LogMethod.EVENT] })
   private readonly namespaceOptions: Map<string, Required<SimpleLogOptions>> = new Map()
   private readonly loggers: WeakMap<Required<SimpleLogOptions>, ISimpleLogger<any>> = new WeakMap()