allow tagging loggers (#6)

* allow tagging loggers

* update config

.nvmrc

@@ -0,0 +1 @@
+12.21.0

.travis.yml

@@ -1,7 +1,7 @@
 dist: trusty
 language: node_js
 node_js:
-  - "12.14.1"
+  - "12.21.0"
 cache:
   directories:
     - node_modules

README.md

@@ -60,7 +60,17 @@ The logger should be initialized with a custom key to identify the logger.
 ```code
 import Piko from '@adv-ui/piko-logger'
 
-const logger = Piko.logger(key)
+const logger = Piko.logger('MyLibrary')
+
+....
+
+// optionally use a tagged instance of logger to fine grain where the logged action occurs
+
+const taggedLogger = logger.tag('MyFunction')
+
+// using the logger methods, logs will show a reference to 'MyLibrary' 
+// using the taggedLogger methods, logs will show a reference to 'MyLibrary # MyFunction' 
+
 ```
 
 ### Log

package.json

@@ -29,23 +29,23 @@
   "author": "One Punch Team",
   "license": "MIT",
   "devDependencies": {
-    "@babel/cli": "7.8.0",
-    "@babel/core": "7.8.0",
-    "@babel/plugin-transform-modules-commonjs": "7.8.0",
-    "@babel/register": "7.8.0",
-    "@babel/runtime": "7.8.0",
+    "@babel/cli": "7.13.16",
+    "@babel/core": "7.13.16",
+    "@babel/plugin-transform-modules-commonjs": "7.13.8",
+    "@babel/register": "7.13.16",
+    "@babel/runtime": "7.13.17",
     "@s-ui/lint": "3",
-    "@s-ui/test": "2",
+    "@s-ui/test": "4",
     "babel-plugin-transform-define": "2.0.0",
     "babel-preset-sui": "3",
-    "chai": "4.2.0",
-    "codecov": "3.6.5",
-    "html-webpack-plugin": "4.0.4",
-    "mocha": "5.2.0",
-    "nyc": "15.0.0",
+    "chai": "4.3.4",
+    "codecov": "3.8.2",
+    "html-webpack-plugin": "5.3.1",
+    "mocha": "8.4.0",
+    "nyc": "15.1.0",
     "versiona": "4",
-    "jsdom": "16.2.2",
-    "sinon": "9.0.2"
+    "jsdom": "16.5.3",
+    "sinon": "10.0.0"
   },
   "eslintConfig": {
     "extends": [

src/integration-test/PikoTest.js

@@ -347,5 +347,15 @@ describe('Piko', () => {
       expect(logger1._level._level).to.equal(DEFAULT_LEVEL)
       expect(logger2._level._level).to.equal(DEFAULT_LEVEL)
     })
+    it('should create tagged loggers', () => {
+      const baseKey = 'base'
+      window.localStorage.setItem(`piko.level.${baseKey}`, 'debug')
+      const base = Piko.logger(baseKey)
+      const tagged = base.tag('tagged')
+      const aDebugMessage = 'debug testing'
+      tagged.debug(() => aDebugMessage)
+      expect(debugSpy.args[0][0]).to.equal('DEBUG | base # tagged | ')
+      expect(debugSpy.args[0][1]).to.equal(aDebugMessage)
+    })
   })
 })

src/main/domain/Logger.js

@@ -1,10 +1,14 @@
 export class Logger {
-  constructor({level, key = 'DEFAULT'}) {
+  constructor({level, key = ''}) {
     this._key = key
     this._level = level
     this._console = console
   }
 
+  tag(tag = 'default') {
+    return new Logger({level: this._level, key: `${this._key} # ${tag}`})
+  }
+
   trace(log) {
     if (this._level.shouldLogTrace()) {
       this._log({