Skip to content

Enhancement: Copy run-tests.php from php/php-src #850

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

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

localheinz
Copy link
Contributor

@localheinz localheinz commented Dec 5, 2023

This pull request

  • copies run-tests.php from php/php-src

Follows #847.

πŸ’β€β™‚οΈ Running

php tests/run-tests.php -j3 -q

on current master with PHP 8.2 yields

PHP Deprecated:  dirname(): Passing null to parameter #1 ($path) of type string is deprecated in /Users/am/Sites/php/web-php/tests/run-tests.php on line 267

Deprecated: dirname(): Passing null to parameter #1 ($path) of type string is deprecated in /Users/am/Sites/php/web-php/tests/run-tests.php on line 267
PHP Deprecated:  dirname(): Passing null to parameter #1 ($path) of type string is deprecated in /Users/am/Sites/php/web-php/tests/run-tests.php on line 271

Deprecated: dirname(): Passing null to parameter #1 ($path) of type string is deprecated in /Users/am/Sites/php/web-php/tests/run-tests.php on line 271

=====================================================================
PHP         : /opt/homebrew/Cellar/[email protected]/8.2.13/bin/php
PHP_SAPI    : cli
PHP_VERSION : 8.2.13
ZEND_VERSION: 4.2.13
PHP_OS      : Darwin - Darwin amMacPro.local 22.6.0 Darwin Kernel Version 22.6.0: Thu Nov  2 07:43:57 PDT 2023; root:xnu-8796.141.3.701.17~6/RELEASE_ARM64_T6000 arm64
INI actual  : /opt/homebrew/etc/php/8.2/php.ini
More .INIs  : /opt/homebrew/etc/php/8.2/conf.d/ext-opcache.ini
CWD         : /Users/am/Sites/php/web-php
Extra dirs  :
VALGRIND    : Not used
=====================================================================
TIME START 2023-12-05 12:04:45
=====================================================================
=====================================================================
========= WELCOME TO THE FUTURE: run-tests PARALLEL EDITION =========
=====================================================================
Spawning 3 workers... Done in 0.03s
=====================================================================

PASS sort no notes [tests/UserNotes/Sorter/sort-no-notes.phpt]
PASS clean_AntiSPAM() removes spam terms [tests/clean-anti-spam.phpt]
PASS gen_challenge() generates a spam challenge [tests/gen-challenge.phpt]
PASS sort sample notes from strpos notes [tests/UserNotes/Sorter/sort-notes-full.phpt]
PASS sort a single note with no votes [tests/UserNotes/Sorter/sort-single-note-with-no-votes.phpt]
PASS sort some notes [tests/UserNotes/Sorter/sort-some-notes.phpt]
PASS is_emailable_address() returns whether email is emailable [tests/is-emailable-address.phpt]
PASS test_answer() returns true when answer to spam challenge is valid [tests/test-answer.phpt]
=====================================================================
TIME END 2023-12-05 12:04:45

=====================================================================
TEST RESULT SUMMARY
---------------------------------------------------------------------
Exts skipped    :    0
Exts tested     :   64
---------------------------------------------------------------------

Number of tests :    8                 8
Tests skipped   :    0 (  0.0%) --------
Tests warned    :    0 (  0.0%) (  0.0%)
Tests failed    :    0 (  0.0%) (  0.0%)
Tests passed    :    8 (100.0%) (100.0%)
---------------------------------------------------------------------
Time taken      :    0 seconds
=====================================================================

Also see https://github.com/php/web-php/actions/runs/7100255461/job/19325955316#step:5:9.

@kamil-tekiela
Copy link
Member

Why not just change $php = null; to $php = '';?

@localheinz
Copy link
Contributor Author

Adjusted, @kamil-tekiela!

@kamil-tekiela
Copy link
Member

I see. I am thinking now if we should keep this as an exact copy of the run_tests.php file from php-src. Or if we make changes here, we should also make changes there.

@localheinz
Copy link
Contributor Author

@kamil-tekiela

See php/php-src#12882!

@localheinz localheinz changed the title Fix: Do not pass null to function that expects string Enhancement: Copy run-tests.php from php/php-src Dec 6, 2023
Copy link
Member

@kamil-tekiela kamil-tekiela left a comment

Choose a reason for hiding this comment

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

Thanks. I think this makes more sense.

@kamil-tekiela kamil-tekiela merged commit 1bf886a into php:master Dec 6, 2023
@localheinz localheinz deleted the fix/null branch December 6, 2023 13:16
@localheinz
Copy link
Contributor Author

Thank you, @kamil-tekiela!

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

Successfully merging this pull request may close these issues.

2 participants