fixes for Windows

Author: jochen-testingbot

src/providers/espresso.ts

@@ -8,6 +8,7 @@ import { io, Socket } from 'socket.io-client';
 import TestingBotError from '../models/testingbot_error';
 import utils from '../utils';
 import Upload from '../upload';
+import platform from '../utils/platform';
 
 export interface EspressoRunInfo {
   id: number;
@@ -395,7 +396,7 @@ export default class Espresso {
   }
 
   private clearLine(): void {
-    process.stdout.write('\r\x1b[K');
+    platform.clearLine();
   }
 
   private formatElapsedTime(seconds: number): string {
@@ -534,14 +535,12 @@ export default class Espresso {
       this.handleShutdown();
     };
 
-    process.on('SIGINT', this.signalHandler);
-    process.on('SIGTERM', this.signalHandler);
+    platform.setupSignalHandlers(this.signalHandler);
   }
 
   private removeSignalHandlers(): void {
     if (this.signalHandler) {
-      process.removeListener('SIGINT', this.signalHandler);
-      process.removeListener('SIGTERM', this.signalHandler);
+      platform.removeSignalHandlers(this.signalHandler);
       this.signalHandler = null;
     }
   }

src/providers/maestro.ts

@@ -13,6 +13,7 @@ import TestingBotError from '../models/testingbot_error';
 import utils from '../utils';
 import Upload from '../upload';
 import { detectPlatformFromFile } from '../utils/file-type-detector';
+import platform from '../utils/platform';
 
 export interface MaestroRunAssets {
   logs?: string[];
@@ -720,7 +721,7 @@ export default class Maestro {
   }
 
   private clearLine(): void {
-    process.stdout.write('\r\x1b[K');
+    platform.clearLine();
   }
 
   private formatElapsedTime(seconds: number): string {
@@ -852,7 +853,7 @@ export default class Maestro {
         responseType: 'arraybuffer',
         headers: {
           'User-Agent': utils.getUserAgent(),
-        }
+        },
       });
 
       await fs.promises.writeFile(filePath, response.data);
@@ -1112,14 +1113,12 @@ export default class Maestro {
       this.handleShutdown();
     };
 
-    process.on('SIGINT', this.signalHandler);
-    process.on('SIGTERM', this.signalHandler);
+    platform.setupSignalHandlers(this.signalHandler);
   }
 
   private removeSignalHandlers(): void {
     if (this.signalHandler) {
-      process.removeListener('SIGINT', this.signalHandler);
-      process.removeListener('SIGTERM', this.signalHandler);
+      platform.removeSignalHandlers(this.signalHandler);
       this.signalHandler = null;
     }
   }

src/providers/xcuitest.ts

@@ -8,6 +8,7 @@ import { io, Socket } from 'socket.io-client';
 import TestingBotError from '../models/testingbot_error';
 import utils from '../utils';
 import Upload from '../upload';
+import platform from '../utils/platform';
 
 export interface XCUITestRunInfo {
   id: number;
@@ -395,7 +396,7 @@ export default class XCUITest {
   }
 
   private clearLine(): void {
-    process.stdout.write('\r\x1b[K');
+    platform.clearLine();
   }
 
   private formatElapsedTime(seconds: number): string {
@@ -537,14 +538,12 @@ export default class XCUITest {
       this.handleShutdown();
     };
 
-    process.on('SIGINT', this.signalHandler);
-    process.on('SIGTERM', this.signalHandler);
+    platform.setupSignalHandlers(this.signalHandler);
   }
 
   private removeSignalHandlers(): void {
     if (this.signalHandler) {
-      process.removeListener('SIGINT', this.signalHandler);
-      process.removeListener('SIGTERM', this.signalHandler);
+      platform.removeSignalHandlers(this.signalHandler);
       this.signalHandler = null;
     }
   }

src/utils/platform.ts

@@ -0,0 +1,58 @@
+/**
+ * Cross-platform utilities for terminal operations and signal handling
+ */
+
+const isWindows = process.platform === 'win32';
+
+/**
+ * Clear the current line in the terminal.
+ * Uses ANSI escape codes on Unix/macOS, and space overwrite on Windows.
+ */
+export function clearLine(): void {
+  if (isWindows) {
+    // Windows fallback: overwrite with spaces and return to start
+    // Use readline if available for better Windows support
+    if (process.stdout.clearLine && process.stdout.cursorTo) {
+      process.stdout.clearLine(0);
+      process.stdout.cursorTo(0);
+    } else {
+      // Fallback: write spaces to clear typical line width
+      process.stdout.write('\r' + ' '.repeat(120) + '\r');
+    }
+  } else {
+    // Unix/macOS: ANSI escape sequence
+    process.stdout.write('\r\x1b[K');
+  }
+}
+
+/**
+ * Setup signal handlers for graceful shutdown.
+ * SIGINT (Ctrl+C) works on all platforms.
+ * SIGTERM only works on Unix/macOS.
+ */
+export function setupSignalHandlers(handler: () => void): void {
+  process.on('SIGINT', handler);
+
+  // SIGTERM is not supported on Windows
+  if (!isWindows) {
+    process.on('SIGTERM', handler);
+  }
+}
+
+/**
+ * Remove signal handlers.
+ */
+export function removeSignalHandlers(handler: () => void): void {
+  process.removeListener('SIGINT', handler);
+
+  if (!isWindows) {
+    process.removeListener('SIGTERM', handler);
+  }
+}
+
+export default {
+  isWindows,
+  clearLine,
+  setupSignalHandlers,
+  removeSignalHandlers,
+};