fix(serve): better handling of --serve-only mode

ArnaudBuchholz · ArnaudBuchholz · commit 9bf0d30dfc91 · 2024-03-08T08:50:39.000-05:00
diff --git a/index.js b/index.js
@@ -58,10 +58,11 @@ async function main () {
       job.port = port
       send({ msg: 'ready', port: job.port })
       output.serving(url)
+      output.reportOnJobProgress()
       if (job.serveOnly) {
+        job.status = 'Serving'
         return
       }
-      output.reportOnJobProgress()
       await notifyAndExecuteTests(job)
       if (job.watch) {
         delete job.start
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ui5-test-runner",
-  "version": "4.3.0",
+  "version": "4.3.1",
   "description": "Standalone test runner for UI5",
   "main": "index.js",
   "bin": {
diff --git a/src/reserve.js b/src/reserve.js
@@ -20,6 +20,6 @@ module.exports = async job => check({
       strict: true,
       'ignore-if-not-found': true
     },
-    ...unhandled(job)
+    ...job.serveOnly ? [{ status: 404 }] : unhandled(job)
   ]
 })

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "ui5-test-runner",`
`3`		`- "version": "4.3.0",`
	`3`	`+ "version": "4.3.1",`
`4`	`4`	`"description": "Standalone test runner for UI5",`
`5`	`5`	`"main": "index.js",`
`6`	`6`	`"bin": {`
Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,6 @@ module.exports = async job => check({`
`20`	`20`	`strict: true,`
`21`	`21`	`'ignore-if-not-found': true`
`22`	`22`	`},`
`23`		`- ...unhandled(job)`
	`23`	`+ ...job.serveOnly ? [{ status: 404 }] : unhandled(job)`
`24`	`24`	`]`
`25`	`25`	`})`