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

Bug #14816 Unit test failures
Submitted: 2008-10-18 02:39 UTC
From: doconnor Assigned: doconnor
Status: Closed Package: File_Find (version CVS)
PHP Version: 5.2.6 OS:
Roadmaps: (Not assigned)    

 [2008-10-18 02:39 UTC] doconnor (Daniel O'Connor)
Description: ------------ The unit tests are failing in CVS. Test script: --------------- clockwerx@clockwerx-desktop:~/pear/File_Find$ pear run-tests tests/ Expected result: ---------------- Tests pass Actual result: -------------- All tests currently fail. TOTAL TIME: 30:04 0 PASSED TESTS 0 SKIPPED TESTS 7 FAILED TESTS: /home/clockwerx/pear/File_Find/tests/01glob.phpt /home/clockwerx/pear/File_Find/tests/02maptree.phpt /home/clockwerx/pear/File_Find/tests/03maptreemultiple.phpt /home/clockwerx/pear/File_Find/tests/04search.phpt /home/clockwerx/pear/File_Find/tests/05search_inside.phpt /home/clockwerx/pear/File_Find/tests/06match_shell.phpt /home/clockwerx/pear/File_Find/tests/bug2773.phpt


 [2008-10-18 03:12 UTC] doconnor (Daniel O'Connor)
The attached patch makes use of sort() in certain places to ensure reproducible results every time. It updates some expected test results as well.
 [2008-10-26 16:55 UTC] doconnor (Daniel O'Connor)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on by the end of next Sunday (CET). If this was a problem with the 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.
 [2008-10-26 17:29 UTC] techtonik (anatoly techtonik)
Thanks for fixing this. My CVS ports are blocked here and it would take a long for me to get to the problem.
 [2008-10-27 12:36 UTC] techtonik (anatoly techtonik)
I need to reopen this bug report, because submitted patch changes the behaviour of the package instead of changing test logic. File_Find doesn't guarantee that returned results will be sorted and should not do this for future. Sorting can be added as optional feature in a later version, but it is unlikely.
 [2009-05-24 07:54 UTC] doconnor (Daniel O'Connor)
-Status: Assigned +Status: Closed