Bug #67177 [Ana->Csd]: run-tests.php doesn't work well with LEAK

[email protected]
Newsgroups php.bugs,php.qa
Message-ID <[email protected]>
Edit report at https://bugs.php.net/bug.php?id=67177&edit=1

 ID:                 67177
 Updated by:         [email protected]
 Reported by:        [email protected]
 Summary:            run-tests.php doesn't work well with LEAK
-Status:             Analyzed
+Status:             Closed
 Type:               Bug
 Package:            Testing related
 PHP Version:        5.6.0beta2
-Assigned To:        
+Assigned To:        nikic
 Block user comment: N
 Private report:     N

 New Comment:

FAILED, BORKED, and LEAKED now all result in non-zero exit status, and --show-mem also exists.


Previous Comments:
------------------------------------------------------------------------
[2014-05-02 10:29:26] [email protected]

@tyrael: A LEAK&FAIL test is only counted towards the number of leaked tests, but not towards the number of failed tests (otherwise it would be counted twice). That's why exit status will be 0 even though there were failed tests (that also happened to have memory errors).

------------------------------------------------------------------------
[2014-05-02 07:28:19] [email protected]

FAILED tests should fail the execution when REPORT_EXIT_STATUS=1 is set, could you show an exemple where that doesn't work?
when you call run-tests.php with -m, it will mark the leaking tests as LEAKED, which isn't FAILED, so that won't result in a non-zero exit status.
--show-mem sounds reasonable, and --show-all should also show the .mem if there is one.

------------------------------------------------------------------------
[2014-05-02 07:00:39] [email protected]

Description:
------------
 * REPORT_EXIT_STATUS=1 results in exit=0 even though there are FAILED&LEAKED tests.
 * There is no --show-mem or --show-leak option.



------------------------------------------------------------------------



--
Edit this bug report at https://bugs.php.net/bug.php?id=67177&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.