Merge pull request #4770 from lohxt1/error_requests_cli_tests_issue

lohxt1 · web-flow · commit 8e23a7054fcd · 2025-05-27T18:49:02.000+05:30
fix: consider request execution errors as `CLI Tests` workflow failure
diff --git a/packages/bruno-cli/src/commands/run.js b/packages/bruno-cli/src/commands/run.js
@@ -617,7 +617,7 @@ const handler = async function (argv) {
       }
     }
 
-    if (summary.failedAssertions + summary.failedTests + summary.failedRequests > 0) {
+    if ((summary.failedAssertions + summary.failedTests + summary.failedRequests > 0) || (summary?.errorRequests > 0)) {
       process.exit(constants.EXIT_STATUS.ERROR_FAILED_COLLECTION);
     }
   } catch (err) {

Original file line number	Diff line number	Diff line change
`@@ -617,7 +617,7 @@ const handler = async function (argv) {`
`617`	`617`	`}`
`618`	`618`	`}`
`619`	`619`
`620`		`- if (summary.failedAssertions + summary.failedTests + summary.failedRequests > 0) {`
	`620`	`+ if ((summary.failedAssertions + summary.failedTests + summary.failedRequests > 0) \|\| (summary?.errorRequests > 0)) {`
`621`	`621`	`process.exit(constants.EXIT_STATUS.ERROR_FAILED_COLLECTION);`
`622`	`622`	`}`
`623`	`623`	`} catch (err) {`