run-tests: fixed exit code not being set on BORKED tests

dktapps · cmb69 · commit 729cd8bacd89 · 2021-02-23T12:50:03.000+01:00
When no test paths are specified this shows up when 'make test' is used on a PECL extension without specifying tests to run (or in php-src too, I guess...) Closes GH-6717.
diff --git a/run-tests.php b/run-tests.php
@@ -1015,7 +1015,7 @@ function test_sort($a, $b)
 junit_save_xml();
 if (getenv('REPORT_EXIT_STATUS') !== '0' &&
 	getenv('REPORT_EXIT_STATUS') !== 'no' &&
-	($sum_results['FAILED'] || $sum_results['LEAKED'])) {
+	($sum_results['FAILED'] || $sum_results['BORKED'] || $sum_results['LEAKED'])) {
 	exit(1);
 }
 exit(0);

Original file line number	Diff line number	Diff line change
`@@ -1015,7 +1015,7 @@ function test_sort($a, $b)`
`1015`	`1015`	`junit_save_xml();`
`1016`	`1016`	`if (getenv('REPORT_EXIT_STATUS') !== '0' &&`
`1017`	`1017`	`getenv('REPORT_EXIT_STATUS') !== 'no' &&`
`1018`		`- ($sum_results['FAILED'] \|\| $sum_results['LEAKED'])) {`
	`1018`	`+ ($sum_results['FAILED'] \|\| $sum_results['BORKED'] \|\| $sum_results['LEAKED'])) {`
`1019`	`1019`	`exit(1);`
`1020`	`1020`	`}`
`1021`	`1021`	`exit(0);`