ci: group test failures separately to make it easier to see all the test failures without having to scroll thousands of lines of output

Jake Champion · JakeChampion · commit 2f661188eb17 · 2024-04-11T20:29:34.000+01:00
diff --git a/integration-tests/js-compute/test.js b/integration-tests/js-compute/test.js
@@ -201,14 +201,17 @@ for (const result of results) {
         failed.push(result.reason)
     }
 }
+core.endGroup()
 
 if (failed.length) {
     process.exitCode = 1;
-    console.log(white, `\nFailed tests:\n`, white);
-}
+    core.startGroup('Failed tests')
+
+    for (const result of failed) {
+        console.log(red, cross, result, white);
+    }
 
-for (const result of failed) {
-    console.log(red, cross, result, white);
+    core.endGroup()
 }
 
 
@@ -218,7 +221,6 @@ if (!local && failed.length) {
         core.notice(`You can debug the service on ${domain}`)
     }
 }
-core.endGroup()
 
 if (!local && !failed.length) {
     const teardownPath = join(fixturePath, 'teardown.js')

Original file line number	Diff line number	Diff line change
`@@ -201,14 +201,17 @@ for (const result of results) {`
`201`	`201`	`failed.push(result.reason)`
`202`	`202`	`}`
`203`	`203`	`}`
	`204`	`+core.endGroup()`
`204`	`205`
`205`	`206`	`if (failed.length) {`
`206`	`207`	`process.exitCode = 1;`
`207`		- console.log(white, `\nFailed tests:\n`, white);
`208`		`-}`
	`208`	`+ core.startGroup('Failed tests')`
	`209`	`+`
	`210`	`+ for (const result of failed) {`
	`211`	`+ console.log(red, cross, result, white);`
	`212`	`+ }`
`209`	`213`
`210`		`-for (const result of failed) {`
`211`		`- console.log(red, cross, result, white);`
	`214`	`+ core.endGroup()`
`212`	`215`	`}`
`213`	`216`
`214`	`217`
`@@ -218,7 +221,6 @@ if (!local && failed.length) {`
`218`	`221`	core.notice(`You can debug the service on ${domain}`)
`219`	`222`	`}`
`220`	`223`	`}`
`221`		`-core.endGroup()`
`222`	`224`
`223`	`225`	`if (!local && !failed.length) {`
`224`	`226`	`const teardownPath = join(fixturePath, 'teardown.js')`