From 3d3f58b6ffa969374559a43eaa313e6662e0f86e Mon Sep 17 00:00:00 2001
From: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
Date: Fri, 8 Sep 2023 14:10:37 -0700
Subject: [PATCH] Have playwright addon tests only start server, not build it

---
 addons/xterm-addon-canvas/package.json              |  2 +-
 addons/xterm-addon-canvas/test/playwright.config.ts |  2 +-
 addons/xterm-addon-webgl/package.json               |  2 +-
 addons/xterm-addon-webgl/test/WebglRenderer.test.ts |  2 +-
 addons/xterm-addon-webgl/test/playwright.config.ts  |  2 +-
 demo/start-server-only.js                           | 10 ++++++++++
 package.json                                        |  1 +
 7 files changed, 16 insertions(+), 5 deletions(-)
 create mode 100644 demo/start-server-only.js

diff --git a/addons/xterm-addon-canvas/package.json b/addons/xterm-addon-canvas/package.json
index 2c8fcb61..76b97758 100644
--- a/addons/xterm-addon-canvas/package.json
+++ b/addons/xterm-addon-canvas/package.json
@@ -20,7 +20,7 @@
     "prepackage": "npm run build",
     "package": "../../node_modules/.bin/webpack",
     "prepublishOnly": "npm run package",
-    "start": "node ../../demo/start"
+    "start-server-only": "node ../../demo/start-server-only"
   },
   "peerDependencies": {
     "xterm": "^5.0.0"
diff --git a/addons/xterm-addon-canvas/test/playwright.config.ts b/addons/xterm-addon-canvas/test/playwright.config.ts
index 3d1fe4fb..b0e565c5 100644
--- a/addons/xterm-addon-canvas/test/playwright.config.ts
+++ b/addons/xterm-addon-canvas/test/playwright.config.ts
@@ -26,7 +26,7 @@ const config: PlaywrightTestConfig = {
   ],
   reporter: 'list',
   webServer: {
-    command: 'npm start',
+    command: 'npm run start-server-only',
     port: 3000,
     timeout: 120000,
     reuseExistingServer: !process.env.CI
diff --git a/addons/xterm-addon-webgl/package.json b/addons/xterm-addon-webgl/package.json
index 7e9765fc..1f88e150 100644
--- a/addons/xterm-addon-webgl/package.json
+++ b/addons/xterm-addon-webgl/package.json
@@ -20,7 +20,7 @@
     "prepackage": "npm run build",
     "package": "../../node_modules/.bin/webpack",
     "prepublishOnly": "npm run package",
-    "start": "node ../../demo/start"
+    "start-server-only": "node ../../demo/start-server-only"
   },
   "peerDependencies": {
     "xterm": "^5.0.0"
diff --git a/addons/xterm-addon-webgl/test/WebglRenderer.test.ts b/addons/xterm-addon-webgl/test/WebglRenderer.test.ts
index d24ed5a3..99710cb9 100644
--- a/addons/xterm-addon-webgl/test/WebglRenderer.test.ts
+++ b/addons/xterm-addon-webgl/test/WebglRenderer.test.ts
@@ -23,7 +23,7 @@ test.afterAll(async () => await ctx.page.close());
 test.describe('WebGL Renderer Integration Tests', async () => {
   // HACK: webgl2 is often not supported in headless firefox on Linux
   // https://github.com/microsoft/playwright/issues/11566
-  test.skip(({ browserName, userAgent}) => (userAgent?.includes('Linux') ?? false) && browserName === 'firefox');
+  test.skip(({ browserName, userAgent }) => (userAgent?.includes('Linux') ?? false) && browserName === 'firefox');
 
   injectSharedRendererTests(ctxWrapper);
 });
diff --git a/addons/xterm-addon-webgl/test/playwright.config.ts b/addons/xterm-addon-webgl/test/playwright.config.ts
index 3d1fe4fb..b0e565c5 100644
--- a/addons/xterm-addon-webgl/test/playwright.config.ts
+++ b/addons/xterm-addon-webgl/test/playwright.config.ts
@@ -26,7 +26,7 @@ const config: PlaywrightTestConfig = {
   ],
   reporter: 'list',
   webServer: {
-    command: 'npm start',
+    command: 'npm run start-server-only',
     port: 3000,
     timeout: 120000,
     reuseExistingServer: !process.env.CI
diff --git a/demo/start-server-only.js b/demo/start-server-only.js
new file mode 100644
index 00000000..b240966f
--- /dev/null
+++ b/demo/start-server-only.js
@@ -0,0 +1,10 @@
+/**
+ * Copyright (c) 2018 The xterm.js authors. All rights reserved.
+ * @license MIT
+ */
+
+// @ts-check
+
+const startServer = require('./server.js');
+
+startServer();
diff --git a/package.json b/package.json
index 77be8f26..534f44ec 100644
--- a/package.json
+++ b/package.json
@@ -28,6 +28,7 @@
     "package-headless": "webpack --config ./webpack.config.headless.js",
     "postpackage-headless": "node ./bin/package_headless.js",
     "start": "node demo/start",
+    "start-server-only": "node demo/start-server-only",
     "build-demo": "webpack --config ./demo/webpack.config.js",
     "start-debug": "node --inspect-brk demo/start",
     "lint": "eslint -c .eslintrc.json --max-warnings 0 --ext .ts src/ addons/",