fix: issue where the access url to expectationManager did not include port

This happened if this.serverAccessBaseUrls was set and port=0.

Author: nytamin

shared/packages/expectationManager/src/internalManager/lib/expectationManagerServer.ts

@@ -100,15 +100,27 @@ export class ExpectationManagerServer {
 		} else if (this.serverAccessBaseUrls) {
 			this._serverAccessUrls = this.serverAccessBaseUrls
 		} else {
-			let url = 'ws://127.0.0.1'
-			if (this.serverOptions.type === 'websocket' && this.serverOptions.port === 0) {
-				// When the configured port i 0, the next free port is picked
-				url += `:${this.manager.expectationManagerServer.websocketServer?.port}`
-			}
 			this._serverAccessUrls = {
-				'*': url,
+				'*': 'ws://127.0.0.1',
+			}
+		}
+		this.logger.info(`##### type: ${this.workForceConnectionOptions.type}`)
+		if (this.serverOptions.type === 'websocket' && this.serverOptions.port === 0) {
+			this.logger.info(`##### this.serverOptions.port: ${this.serverOptions.port}`)
+			// When the configured port i 0, the next free port is picked
+			for (const key of Object.keys(this._serverAccessUrls)) {
+				// If no port was specified in the url, add it:
+				if (!this._serverAccessUrls[key].match(/\/\/[^/]+:\d/)) {
+					this.logger.info(`updateing ${key}`)
+					this._serverAccessUrls[key] += `:${this.manager.expectationManagerServer.websocketServer?.port}`
+				}
 			}
 		}
+
+		this.logger.info(`##### this.serverAccessBaseUrls: ${JSON.stringify(this.serverAccessBaseUrls)}`)
+		this.logger.info(`##### websocketServer?.port: ${this.manager.expectationManagerServer.websocketServer?.port}`)
+		this.logger.info(`##### this._serverAccessUrls: ${JSON.stringify(this._serverAccessUrls)}`)
+
 		if (!this._serverAccessUrls) throw new Error(`ExpectationManager.serverAccessUrl not set!`)
 	}
 	terminate(): void {