Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.9.2

Bug #19127 upgrade PHPUnit require statements & other fixes (for PEAR QA Team)
Submitted: 2011-12-11 10:10 UTC
From: danielc Assigned: danielc
Status: Closed Package: PEAR_Info (version SVN)
PHP Version: 5_4 SVN-2011-12-11 OS:
Roadmaps: (Not assigned)    
Subscription  


 [2011-12-11 10:10 UTC] danielc (Daniel Convissor)
Description: ------------ Current versions of PHPUnit have a different layout, so calls to require PHPUnit/Framework.php fail. Newer versions also have an autoloader. The require statements need to be adjusted so both older and newer versions to work. In addition, I adjusted all the tests to be able to run from both an installed state and from an SVN checkout. In particular, render.phpt needed fixing because in SVN it was going fatal because it was trying to open the stylesheet at the unsubstituted "@data_dir@/@package_name@/pearinfo.css", which, uh, sure won't work. :) The existing tests repeat a lot of code setting directory names, etc. They're nearly all the same in each file and they're set in both the FILE and CLEAN sections of each file. So I consolidated them to some files that get included. I did this to create a SKIP section for render.phpt, but once I got that going, I realized how to work around the files not being found. Having both an AllTests.php and a PEAR_Info_TestSuite_Standard.php is unnecessary. All of the functionality has been consolidated to AllTests.php. Finally, I set an svn:ignore for .depdblock .lock and .depdb in tests/pear_dir. Getting PEAR's continuous integration testing system running is an urgent priority. Please fix this as soon as possible. I have karma and the changes on hand so can easily commit them if you like.

Comments

 [2011-12-11 10:11 UTC] danielc (Daniel Convissor)
Added patch qa.
 [2011-12-29 09:27 UTC] danielc (Daniel Convissor)
-Status: Open +Status: Closed -Assigned To: +Assigned To: danielc