| 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 | ||
[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-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.