fix(auth): throw error when auth env vars are missing

examples/wdio-ui5-lts/wdio.conf.js

@@ -5,7 +5,9 @@ const url = `https://ui5.sap.com/${ui5Version}/test-resources/sap/m/demokit/orde
 
 export const config = {
     wdi5: {
-        logLevel: "verbose"
+        logLevel: "verbose",
+        // Increase timeout to handle "waitAsync is already running" errors
+        waitforui5Timeout: 30000
     },
     baseUrl: url,
 

src/lib/authentication/Authenticator.ts

@@ -19,15 +19,23 @@ export default class Authenticator {
         if (browser.isMultiremote) {
             envName = `wdi5_${this.browserInstanceName}_username`
         }
-        return process.env[envName] || ""
+        const sUserName = process.env[envName]
+        if (!sUserName) {
+            throw new Error(`Environment variable ${envName} for username is not set!`)
+        }
+        return sUserName
     }
 
     getPassword(): string {
         let envName = "wdi5_password"
         if (browser.isMultiremote) {
             envName = `wdi5_${this.browserInstanceName}_password`
         }
-        return process.env[envName] || ""
+        const sPassword = process.env[envName]
+        if (!sPassword) {
+            throw new Error(`Environment variable ${envName} for password is not set!`)
+        }
+        return sPassword
     }
 
     async getIsLoggedIn(): Promise<boolean> {

test/Authenticator.test.ts

@@ -0,0 +1,28 @@
+import Authenticator from "../src/lib/authentication/Authenticator.js"
+import { throws } from "node:assert"
+
+describe("base authenticator", function () {
+    before(function () {
+        globalThis.browser = { isMultiremote: false }
+    })
+
+    after(function () {
+        globalThis.browser = null
+    })
+
+    it("should throw if username is not provided", function () {
+        const auth = new Authenticator("fake")
+        throws(() => auth.getUsername(), {
+            name: /Error/,
+            message: /for username is not set/
+        })
+    })
+
+    it("should throw if password is not provided", function () {
+        const auth = new Authenticator("fake")
+        throws(() => auth.getPassword(), {
+            name: /Error/,
+            message: /for password is not set/
+        })
+    })
+})