Skip to content

Commit

Permalink
Set tests exit status by default.
Browse files Browse the repository at this point in the history 
Use REPORT_EXIT_STATUS=0 or REPORT_EXIT_STATUS=no to prevent it.

Based on discussion in https://www.mail-archive.com/internals@lists.php.net/msg94141.html
  • Loading branch information
@smalyshev
smalyshev committed Mar 18, 2018
1 parent dfc07f0 commit f19c94b
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion run-tests.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -841,7 +841,9 @@ function save_or_mail_results()

junit_save_xml();

if (getenv('REPORT_EXIT_STATUS') == 1 && ($sum_results['FAILED'] || $sum_results['BORKED'])) {
if (getenv('REPORT_EXIT_STATUS') !== '0' &&
getenv('REPORT_EXIT_STATUS') !== 'no' &&
($sum_results['FAILED'] || $sum_results['BORKED'])) {
exit(1);
}

Expand Down

1 comment on commit f19c94b

@krakjoe
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

php-src/run-tests.php

Line 982 in f19c94b

if (getenv('REPORT_EXIT_STATUS') == 1 && ($sum_results['FAILED'] || $sum_results['BORKED'])) {
?

Please sign in to comment.