Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test for bug GH-16870 (Overly restrictive gmp_pow overflow checks) #16898

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

GaryAllan
Copy link

Add test file for Bug #16870

$ /usr/bin/php -v
PHP 8.3.14 (cli) (built: Nov 21 2024 14:44:18) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.3.14, Copyright (c) Zend Technologies
    with Xdebug v3.3.2, Copyright (c) 2002-2024, by Derick Rethans

$ ./run-tests.php -p /usr/bin/php ./ext/gmp/ | grep 16870
PASS Bug GH-16870 (Overly restrictive gmp_pow overflow checks) [ext/gmp/tests/gh16870.phpt] 
$ ./sapi/cli/php -v
PHP 8.5.0-dev (cli) (built: Nov 22 2024 12:04:57) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.5.0-dev, Copyright (c) Zend Technologies

$ ./run-tests.php -p ./sapi/cli/php ./ext/gmp/ | grep 16870
FAIL Bug GH-16870 (Overly restrictive gmp_pow overflow checks) [ext/gmp/tests/gh16870.phpt] 
Bug GH-16870 (Overly restrictive gmp_pow overflow checks) [ext/gmp/tests/gh16870.phpt]

Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

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

Please add an XFAIL section so that CI is green :)

ext/gmp/tests/gh16870.phpt Outdated Show resolved Hide resolved
Add XFAIL header for CI

Co-authored-by: Gina Peter Banyard <[email protected]>
@cmb69
Copy link
Member

cmb69 commented Nov 22, 2024

IMO, it's better to replace the test case of #16884 with this one, instead of running XFAIL tests in CI while the bug fix is pending.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants