Bug #70580 [Com]: Test Zend/tests/bug42143.phpt is broken

Edit report at https://bugs.php.net/bug.php?id=70580&edit=1 ID: 70580 Comment by: carusogabriel@php.net Reported by: rainer dot jung at kippdata dot de Summary: Test Zend/tests/bug42143.phpt is broken Status: Re-Opened Type: Bug Package: Testing related Operating System: Solaris PHP Version: 7.0.0RC3 Assigned To: carusogabriel Block user comment: N Private report: N New Comment: @cmb My apologies, I didn't pay attention to that detail. I'll try to emulate Solaris locally and see if we can fix, or drop those lines as suggested. Btw, thanks for the Wiki Systems link (https://wiki.php.net/systems), I didn't know about it. Previous Comments: ------------------------------------------------------------------------ [2018-08-05 14:20:19] cmb@php.net gcov.php.net runs on CentOS 6[1], so the test is likely to succeed. The report is about the test failing on Solaris, which still might be the case. I agree with Rainer, that the two lines should be dropped. Kalle's argument to keep the lines for better code coverage is somewhat moot, since our log() and log10() implementations are thin wrappers over C's log() and log10(), respectively, so basically we're testing implementation defined behavior here. [1] <https://wiki.php.net/systems/nex2> ------------------------------------------------------------------------ [2018-08-05 05:00:40] carusogabriel@php.net Closing as this tests isn't failing anymore: http://gcov.php.net/viewer.php?version=PHP_HEAD&func=tests ------------------------------------------------------------------------ [2015-10-24 04:48:25] kalle@php.net Now its been a while with proper testing writing, but I think its possible that we change the EXPECT section to EXPECTF and then have something like: %NaN|INF% or similar, I should probably consult run-tests.php for this, but it seems most reasonable we keep the test for most code path coverage. ------------------------------------------------------------------------ [2015-09-25 12:05:16] rainer dot jung at kippdata dot de This applies also to PHP 5.4-5.6 (latest releases). ------------------------------------------------------------------------ [2015-09-25 11:59:46] rainer dot jung at kippdata dot de Description: ------------ The test Zend/tests/bug42143.phpt contains: 13 var_dump(log10(-1) == log(-1)); 14 var_dump(log10(-1) != log10(-1)); On Linux, that test works, because the logarithm of -1 returns NAN as expected. On Solaris the log returns -INF and that part of the test fails. This behavior is allowed by the specifications: See in http://pubs.opengroup.org/onlinepubs/9699919799/functions/log.html "For finite values of x that are less than 0 ..., a domain error shall occur, and either a NaN (if supported), or an implementation-defined value shall be returned." Since there are a lot of other but similar tests in Zend/tests/bug42143.phpt, I suggest to drop these two lines. See attached patch. ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=70580&edit=1