TraceServer: Make the 'UP' server check reusable

Prepare the ground for a follow-up fix about adding another such check.

Contributes to fixing #8.

Signed-off-by: Marco Miller <marco.miller@ericsson.com>

Author: marco-miller

src/trace-server.ts

@@ -122,16 +122,12 @@ export class TraceServer {
             },
             async progress => {
                 progress.report({ message: 'Starting up...' });
-                const client = new TspClient(serverUrl);
                 let timeout = false;
                 const timeoutId = setTimeout(() => (timeout = true), millis);
 
                 // eslint-disable-next-line no-constant-condition
                 while (true) {
-                    const health = await client.checkHealth();
-                    const status = health.getModel()?.status;
-
-                    if (health.isOk() && status === 'UP') {
+                    if (await this.isUp(serverUrl)) {
                         this.showStatus(true);
                         this.server?.once(exit, () => {
                             this.stopOrReset(context);
@@ -149,6 +145,13 @@ export class TraceServer {
         );
     }
 
+    private async isUp(serverUrl: string) {
+        const client = new TspClient(serverUrl);
+        const health = await client.checkHealth();
+        const status = health.getModel()?.status;
+        return health.isOk() && status === 'UP';
+    }
+
     private showError(message: string) {
         console.error(message);
         vscode.window.showErrorMessage(message);