php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Request #62055 Make run-tests.php support --CGI-- sections
Submitted: 2012-05-17 19:22 UTC Modified: 2017-10-30 20:06 UTC
Votes:2
Avg. Score:4.0 ± 1.0
Reproduced:1 of 1 (100.0%)
Same Version:1 (100.0%)
Same OS:1 (100.0%)
From: zoe@php.net Assigned: cmb (profile)
Status: Closed Package: Testing related
PHP Version: master-Git-2012-05-17 (Git) OS: Any
Private report: No CVE-ID: None
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: zoe@php.net
New email:
PHP Version: OS:

 

 [2012-05-17 19:22 UTC] zoe@php.net 
Description:
------------
ext/zlib/tests/bug60761.phpt

Contains a section --CGI-- which is not a PHPT section. The run-tests.php code 
simply ignores this section.

I'm tempted to just remove it, as it makes no difference to the test. However the 
comments in the test imply that the author intended to run this using php-cgi. It 
is actually using that cli. 

It's a valid test (if you take the --CGI-- out), is it useful in relation to bug 
60761?

Patches

run-tests-cgi (last revision 2015-07-18 13:48 UTC by cmb@php.net)

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2012-05-18 08:37 UTC] zoe@php.net
-Assigned To: +Assigned To: zoe@php.net
 [2012-05-18 08:43 UTC] zoe@php.net 
My bad. --CGI-- is a valid test section but only for tests run using server-
tests.php. run-tests.php just ignores it silently and uses the php-cli.

Would --GZIP_POST-- be helpful? This will use php-cgi...
 [2012-05-18 22:44 UTC] felipe@php.net
-Assigned To: zoe@php.net +Assigned To: zoe
 [2012-10-02 13:42 UTC] zoe@php.net 
This test is not using the CGI. It is just being run by the CLI. 
The options are 
1. to remove the --CGI-- section (which is simply ignored) and the comments and leave it as a valid CLI test.
2. To remove the test
 [2015-07-18 13:48 UTC] cmb@php.net 
The following patch has been added/updated:

Patch Name: run-tests-cgi
Revision:   1437227309
URL:        https://bugs.php.net/patch-display.php?bug=62055&patch=run-tests-cgi&revision=1437227309
 [2015-07-18 13:49 UTC] cmb@php.net 
According to the test file layout documentation[1] the CGI section
is supposed to mark tests that must be run as CGI, but it is not
implemented for run-tests.php yet. Wouldn't it make sense to
implement it? Applying the attached patch "run-tests-cgi" seems to
be sufficient.

[1] <https://qa.php.net/phpt_details.php#cgi_section>
 [2017-10-24 02:56 UTC] kalle@php.net
-Summary: Minor fix to ext/zlib/tests/bug60761.phpt +Summary: Make run-tests.php support --CGI-- sections -Assigned To: zoe +Assigned To: cmb
 [2017-10-24 02:56 UTC] kalle@php.net 
@christoph, I think you should apply the patch, server-tests.php is dead anyway and sapi/cli and sapi/cgi is fairly close enough for run-tests.php to just being able to switch anyway
 [2017-10-30 19:39 UTC] cmb@php.net
-Type: Bug +Type: Feature/Change Request
 [2017-10-30 20:06 UTC] cmb@php.net
-Status: Assigned +Status: Closed
 [2017-10-30 20:06 UTC] cmb@php.net 
Closed with
<http://git.php.net/?p=php-src.git;a=commit;h=dc4427d0caf2d066cd01f91fd0e899217fbceb30>.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Nov 21 11:01:29 2024 UTC