[PHP-BUG] Bug #80018 [NEW]: Tests expect var_dump(FLOAT) to print an integer

"jamieliu at google dot com"<[email protected]>
Newsgroups gmane.comp.php.quality-assurance
Message-ID <[email protected]>
From:             jamieliu at google dot com
Operating system: All
PHP version:      master-Git-2020-08-25 (Git)
Package:          Testing related
Bug Type:         Bug
Bug description:Tests expect var_dump(FLOAT) to print an integer

Description:
------------
Many tests parse the result of var_dump() on a float and expect the
result to be an integer:

$ find . -name '*.phpt' | xargs grep 'float(%d)'
./Zend/tests/cast_to_double.phpt:float(%d)
./Zend/tests/settype_double.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk.phpt:float(%d)
./ext/standard/tests/file/disk.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_error.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_error.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_error.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_free_space_basic.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/file/disk_total_space_variation.phpt:float(%d)
./ext/standard/tests/general_functions/type.phpt:float(%d)
./ext/standard/tests/general_functions/type.phpt:float(%d)
./ext/standard/tests/general_functions/type.phpt:float(%d)
./ext/standard/tests/general_functions/gettype_settype_basic.phpt:float(%d)
./ext/standard/tests/general_functions/gettype_settype_basic.phpt:float(%d)
./ext/standard/tests/general_functions/gettype_settype_basic.phpt:float(%d)
./ext/standard/tests/general_functions/gettype_settype_basic.phpt:float(%d)

This is problematic because var_dump() will print sufficiently large
numbers in exponential notation. An example in Linux:

$ mkdir tmp
$ sudo mount -t tmpfs -o size=$((1 << 62)) tmpfs tmp
$ ~/src/php-src/sapi/cli/php --version
PHP 8.0.0-dev (cli) (built: Aug 25 2020 12:55:38) ( NTS )
Copyright (c) The PHP Group
Zend Engine v4.0.0-dev, Copyright (c) Zend Technologies
$ ~/src/php-src/sapi/cli/php -r 'var_dump(disk_total_space("tmp"));'
float(4.611686018427388E+18)

I think every instance of "float(%d)" in such tests could simply be
converted to use "float(%E)" instead.

Test script:
---------------
See description.


-- 
Edit bug report at https://bugs.php.net/bug.php?id=80018&edit=1
-- 
Fix committed:                    https://bugs.php.net/fix.php?id=80018&r=fixed
Fixed in release:                 https://bugs.php.net/fix.php?id=80018&r=alreadyfixed
Need backtrace:                   https://bugs.php.net/fix.php?id=80018&r=needtrace
Need Reproduce Script:            https://bugs.php.net/fix.php?id=80018&r=needscript
Try newer version:                https://bugs.php.net/fix.php?id=80018&r=oldversion
Not developer issue:              https://bugs.php.net/fix.php?id=80018&r=support
Expected behavior:                https://bugs.php.net/fix.php?id=80018&r=notwrong
Not enough info:                  https://bugs.php.net/fix.php?id=80018&r=notenoughinfo
Submitted twice:                  https://bugs.php.net/fix.php?id=80018&r=submittedtwice
register_globals:                 https://bugs.php.net/fix.php?id=80018&r=globals
PHP version support discontinued: https://bugs.php.net/fix.php?id=80018&r=phptooold
Daylight Savings:                 https://bugs.php.net/fix.php?id=80018&r=dst
IIS Stability:                    https://bugs.php.net/fix.php?id=80018&r=isapi
Install GNU Sed:                  https://bugs.php.net/fix.php?id=80018&r=gnused
Floating point limitations:       https://bugs.php.net/fix.php?id=80018&r=float
No Zend Extensions:               https://bugs.php.net/fix.php?id=80018&r=nozend
MySQL Configuration Error:        https://bugs.php.net/fix.php?id=80018&r=mysqlcfg
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.