php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #55788 test redirects should abide SHOW_ONLY_GROUPS in run-tests.php
Submitted: 2011-09-26 21:55 UTC Modified: 2011-11-26 17:56 UTC
From: tyrael@php.net Assigned: felipe (profile)
Status: Closed Package: Testing related
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: tyrael@php.net
New email:
PHP Version: OS:

 

 [2011-09-26 21:55 UTC] tyrael@php.net
Description:
------------
currently the test redirects(show_redirect_start() and show_redirect_ends()) will 
always print two lines to the output, ignoring the value of SHOW_ONLY_GROUPS.
I think that it would be more consistent, if those lines could be controlled the 
same way, as we do with everything else.
this change would only noticeable for those who run the test suite with custom 
options (passing the SHOW_ONLY_GROUPS environment variable to make test or the 
argument -g to run-tests.php).
the attached patch is pretty straightforward, you can pass the ---> if you need 
the ---> lines in your testsuite output.


Patches

run-tests.php.diff (last revision 2011-09-28 23:20 UTC by tyrael@php.net)

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-09-28 23:18 UTC] tyrael@php.net
as Hannes asked, I update the patch with the following:
the groupname for redirect is REDIRECT, as it is more clear what it does than -->
I also updated the code, so now it prints 
REDIRECT $tests ($tested [$tested_file]) begin\n
instead of
--> $tests ($tested [$tested_file]) begin\n
so the group name, and the ouput is consistent as for the other groupnames.
I also updated the help to list the possible values for -g (aka SHOW_ONLY_GROUPS)
 [2011-09-28 23:20 UTC] tyrael@php.net
The following patch has been added/updated:

Patch Name: run-tests.php.diff
Revision:   1317252015
URL:        https://bugs.php.net/patch-display.php?bug=55788&patch=run-tests.php.diff&revision=1317252015
 [2011-11-26 17:56 UTC] felipe@php.net
Automatic comment from SVN on behalf of felipe
Revision: http://svn.php.net/viewvc/?view=revision&revision=319964
Log: - Fixed bug #55788 (test redirects should abide SHOW_ONLY_GROUPS in run-tests.php)
  patch by: tyrael at php.net
 [2011-11-26 17:56 UTC] felipe@php.net
This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.


 [2011-11-26 17:56 UTC] felipe@php.net
-Status: Open +Status: Closed -Type: Feature/Change Request +Type: Bug -Assigned To: +Assigned To: felipe
 [2012-04-18 09:47 UTC] laruence@php.net
Automatic comment on behalf of felipe
Revision: http://git.php.net/?p=php-src.git;a=commit;h=adf6003bddbf675b841b719420b5b71466cf4c57
Log: - Fixed bug #55788 (test redirects should abide SHOW_ONLY_GROUPS in run-tests.php)   patch by: tyrael at php.net
 [2012-07-24 23:38 UTC] rasmus@php.net
Automatic comment on behalf of felipe
Revision: http://git.php.net/?p=php-src.git;a=commit;h=adf6003bddbf675b841b719420b5b71466cf4c57
Log: - Fixed bug #55788 (test redirects should abide SHOW_ONLY_GROUPS in run-tests.php)   patch by: tyrael at php.net
 [2013-11-17 09:34 UTC] laruence@php.net
Automatic comment on behalf of felipe
Revision: http://git.php.net/?p=php-src.git;a=commit;h=adf6003bddbf675b841b719420b5b71466cf4c57
Log: - Fixed bug #55788 (test redirects should abide SHOW_ONLY_GROUPS in run-tests.php)   patch by: tyrael at php.net
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Oct 31 22:01:27 2024 UTC