WIP: Try different quit approach for CI

Author: pimterry

test/smoke.spec.ts

@@ -5,7 +5,8 @@ test('app launches and loads UI from server', async () => {
     const electronApp = await electron.launch({
         cwd: path.join(import.meta.dirname, '..'),
         args: ['.'],
-        timeout: 60000
+        timeout: 60000,
+        ...(process.env.CI && { env: { ...process.env, ELECTRON_ENABLE_LOGGING: '1' } })
     });
 
     electronApp.process().stdout?.on('data', (data) => {
@@ -23,18 +24,6 @@ test('app launches and loads UI from server', async () => {
     await expect(window.locator('h1:has-text("Intercept HTTP")')).toBeVisible({ timeout: 60000 });
     console.log('saw title');
 
-    await Promise.race([
-        electronApp.close(),
-        new Promise((_, reject) => setTimeout(() => reject(new Error('App failed to close cleanly')), 5000))
-    ]).catch(async (error) => {
-        console.warn('Warning:', error.message, '- force killing process');
-        electronApp.process().kill('SIGKILL');
-        if (!process.env.CI) {
-            console.log('(Note that shutdown issues like this are ignored in CI)');
-            throw error;
-        } else {
-            console.log('Ignoring error (assuming this is a CI issue)')
-        }
-    });
+    await electronApp.close();
     console.log('closed app');
 });