Fixing other logLevel instances

Author: Ian Walter

cli.js

@@ -8,13 +8,14 @@ async function run () {
   const config = cli({
     name: 'bff',
     options: {
-      logLevel: {
+      log: {
         alias: 'l',
-        default: 'info'
+        description: "Specifies bff-webdriver's print (logging) configuration",
+        default: { level: 'info' }
       }
     }
   })
-  const print = new Print({ level: config.logLevel })
+  const print = new Print(config.log)
   const { _: [command] } = config
 
   try {

index.js

@@ -12,7 +12,7 @@ const shouldUseBsl = ({ browserstackLocal, capabilities: cap }) =>
 module.exports = {
   webdriverVersion,
   async before (context) {
-    print = new Print({ level: context.log.level })
+    print = new Print(context.log)
     try {
       // Set the WebDriver version if not already configured.
       context.webdriver.version = context.webdriver.version || webdriverVersion
@@ -54,7 +54,7 @@ module.exports = {
     }
   },
   registration (file, context) {
-    print = new Print({ level: context.logLevel })
+    print = new Print(context.log)
     try {
       // Extract the WebDriver capabilities from the test configuration.
       const capabilities = Array.isArray(context.webdriver.capabilities)
@@ -89,7 +89,7 @@ module.exports = {
     }
   },
   async beforeEach (file, context) {
-    print = new Print({ level: context.logLevel })
+    print = new Print(context.log)
 
     try {
       print.debug('Adding WebDriver integrations')
@@ -120,7 +120,7 @@ module.exports = {
       context.testContext.browser = await remote({
         path: '/wd/hub',
         ...context.webdriver,
-        logLevel: context.webdriver.logLevel || context.logLevel,
+        logLevel: context.webdriver.logLevel || context.log.level,
         capabilities: context.testContext.capability
       })
 

integrations/appium.js

@@ -5,7 +5,7 @@ const { Print } = require('@ianwalter/print')
 module.exports = class AppiumIntegration {
   constructor (context) {
     // Set up a print instance on the integration instance so it can be reused.
-    this.print = new Print({ level: context.logLevel })
+    this.print = new Print(context.log)
     this.print.debug('Appium integration enabled')
 
     // Define the global capability options.

integrations/browserstack.js

@@ -5,7 +5,7 @@ const { Print } = require('@ianwalter/print')
 module.exports = class BrowserStackIntegration {
   constructor (context) {
     // Set up a print instance on the integration instance so it can be reused.
-    this.print = new Print({ level: context.logLevel })
+    this.print = new Print(context.log)
     this.print.debug('BrowserStack integration enabled')
 
     // Make BrowserStack the WebDriver backend if not already configured.

integrations/zalenium.js

@@ -3,7 +3,7 @@ const { Print } = require('@ianwalter/print')
 module.exports = class ZaleniumIntegration {
   constructor (context) {
     // Set up a print instance on the integration instance so it can be reused.
-    this.print = new Print({ level: context.logLevel })
+    this.print = new Print(context.log)
     this.print.debug('Zalenium integration enabled')
   }
 

package.json

@@ -58,7 +58,9 @@
     "@ianwalter/url": "^2.0.0"
   },
   "bff": {
-    "logLevel": "debug",
+    "log": {
+      "level": "debug"
+    },
     "plugins": [
       "config",
       "index"