fix: safeguard browser.getTimeouts logging (#483)

closes #473 (and merged by the mighty power of grayskull)
ui5-community · Jun 14, 2023 · 7ddd2ba · 7ddd2ba
1 parent df86e83
commit 7ddd2ba
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/examples/ui5-ts-app/package.json b/examples/ui5-ts-app/package.json
@@ -11,8 +11,7 @@
     "start:ui5": "ui5 serve --port 8080 --accept-remote-connections",
     "//test": "run-s test:*",
     "test": "wdio run wdio-ui5.conf.ts",
-    "test:browserstack:local": "wdio run test/e2e/cloud-services/browserstack.conf.local.ts",
-    "test:saucelabs:local": "wdio run test/e2e/cloud-services/saucelabs.conf.local.ts",
+    "test:browserstack": "wdio run test/e2e/cloud-services/browserstack.conf.ts",
     "test:authentication": "run-s authentication:*",
     "//test-h:authentication": "run-s \"authentication:* -- --headless\"",
     "test:authentication:bstack": "BROWSERSTACK=true run-s authentication:*",

diff --git a/src/service.ts b/src/service.ts
@@ -31,7 +31,9 @@ export default class Service implements Services.ServiceInstance {
         await browser.setTimeout({ script: timeout })
 
         Logger.debug(`browser script timeout set to ${timeout}`)
-        Logger.debug(`browser timeouts are ${JSON.stringify(await browser.getTimeouts(), null, 2)}`)
+        if (typeof browser.getTimeouts === "function") {
+            Logger.debug(`browser timeouts are ${JSON.stringify(await browser.getTimeouts(), null, 2)}`)
+        }
 
         if (browser instanceof MultiRemoteDriver) {
             for (const name of (browser as MultiRemoteDriver).instances) {