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

Request #17080 Avoid setting include path in unit tests
Submitted: 2010-02-09 07:08 UTC
From: doconnor Assigned: till
Status: Closed Package: File_IMC (version CVS)
PHP Version: 5.2.5 OS:
Roadmaps: 0.4.1    
Subscription  


 [2010-02-09 07:08 UTC] doconnor (Daniel O'Connor)
Description: ------------ php tests\AllTests.php doesn't work so well because of the incorrect manipulation of the include path (path seperator is wrong for windows, ok for linux) Consider just relying on running the tests from 1x level up; and removing the include_path modification.

Comments

 [2010-02-09 07:09 UTC] doconnor (Daniel O'Connor)
 [2010-02-09 16:31 UTC] till (Till Klampaeckel)
Why not fix the path seperator then?
 [2010-02-10 06:44 UTC] doconnor (Daniel O'Connor)
If you aggregate all of the AllTests together, and one randomly changes the include_path on you and the rest don't ... :(
 [2010-06-27 22:13 UTC] till (Till Klampaeckel)
-Status: Open +Status: Closed -Assigned To: +Assigned To: till -Roadmap Versions: +Roadmap Versions: 0.4.1
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. Check r300777, please. I migrated the set_include_path to AllTests and I'm only doing it when run from SVN. I hope no objections there.