FAQ
Edit report at https://pear.php.net/bugs/bug.php?id=19169&edit=1

ID: 19169
Updated by: danielc@analysisandsolutions.com
Reported By: danielc at analysisandsolutions dot com
Summary: remove the broken and supurfluous AllTests.php
Status: Open
Type: Bug
Package: Services_Twitter
Package Version: SVN
PHP Version: 5_4 SVN-2011-12-29
Roadmap Versions:
New Comment:

Getting PEAR's continuous integration testing system running is an
urgent priority. Please fix this as soon as possible.


Previous Comments:
------------------------------------------------------------------------

[2011-12-29 22:45:24] danielc

Added #patch bug:19169;patch:alltests;revision:1325198724;.

------------------------------------------------------------------------

[2011-12-29 22:45:07] danielc

Description:
------------
The require statements in AllTests.php use PHPUnit's old file layout. I
was going to ask you to fix this, but then realized this test suite
actually uses PHPT style tests. So the AllTests.php file is superfluous
and leads to the Continuous Integration servers to run the tests twice
(once for AllTests.php and once in PHPT mode). Attached is a patch to
remove the AllTests.php file.

I have the karma to commit this change, if you wish.

------------------------------------------------------------------------

Search Discussions

  • Danielc at Dec 31, 2011 at 9:32 pm
    Edit report at https://pear.php.net/bugs/bug.php?id=19169&edit=1

    ID: 19169
    Updated by: danielc@analysisandsolutions.com
    Reported By: danielc at analysisandsolutions dot com
    Summary: remove the broken and supurfluous AllTests.php
    -Status: Open
    +Status: Closed
    Type: Bug
    Package: Services_Twitter
    Package Version: SVN
    PHP Version: 5_4 SVN-2011-12-29
    -Assigned To:
    +Assigned To: danielc
    Roadmap Versions:
    New Comment:

    -Status: Open
    +Status: Closed
    -Assigned To:
    +Assigned To: danielc



    Previous Comments:
    ------------------------------------------------------------------------

    [2011-12-29 22:48:39] danielc

    Getting PEAR's continuous integration testing system running is an
    urgent priority. Please fix this as soon as possible.

    ------------------------------------------------------------------------

    [2011-12-29 22:45:24] danielc

    Added #patch bug:19169;patch:alltests;revision:1325198724;.

    ------------------------------------------------------------------------

    [2011-12-29 22:45:07] danielc

    Description:
    ------------
    The require statements in AllTests.php use PHPUnit's old file layout. I
    was going to ask you to fix this, but then realized this test suite
    actually uses PHPT style tests. So the AllTests.php file is superfluous
    and leads to the Continuous Integration servers to run the tests twice
    (once for AllTests.php and once in PHPT mode). Attached is a patch to
    remove the AllTests.php file.

    I have the karma to commit this change, if you wish.

    ------------------------------------------------------------------------

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedDec 29, '11 at 9:48p
activeDec 31, '11 at 9:32p
posts2
users1
websitepear.php.net

1 user in discussion

Danielc: 2 posts

People

Translate

site design / logo © 2022 Grokbase