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

Bug #15123 Broken CSV unit tests
Submitted: 2008-11-22 03:23 UTC
From: doconnor Assigned: dufuz
Status: Closed Package: File_CSV
PHP Version: 5.2.6 OS:
Roadmaps: (Not assigned)    
Subscription  


 [2008-11-22 03:23 UTC] doconnor (Daniel O'Connor)
Description: ------------ Take a look at http://www.lauken.com/doconnor/results.txt File/tests/CSV/033.phpt File/tests/CSV/034.phpt are currently broken Test script: --------------- pear run-tests tests/CSV/ Expected result: ---------------- Tests pass Actual result: -------------- 18) /home/clockwerx/pear/File/tests/CSV/033.phpt --- Expected +++ Actual @@ -17,7 +17,7 @@ [3] => a"b [4] => 4 ) - + [1] => Array ( [0] => 1 @@ -26,14 +26,14 @@ [3] => 3 [4] => ) - + [2] => Array ( [0] => 1 [1] => 2 - [2] => ,"," - [3] => 4 - [4] => , + [2] => + [3] => + [4] => ",4, ) ) \ No newline at end of file 19) /home/clockwerx/pear/File/tests/CSV/034.phpt --- Expected +++ Actual @@ -16,21 +16,21 @@ [2] => 3 [3] => 4 ) - + [1] => Array ( [0] => 1 [1] => 2 - [2] => , - [3] => 4 + [2] => + [3] => ) - + [2] => Array ( [0] => 1 [1] => 2 [2] => 3 - [3] => , + [3] => ) ) \ No newline at end of file

Comments

 [2008-11-26 14:33 UTC] dufuz (Helgi Þormar Þorbjörnsson)
Please open bugs on the proper package. This is a known "issue", it's a test for functionality that has not yet been implemented, kinda pointless bug report.
 [2009-08-13 02:20 UTC] cweiske (Christian Weiske)
Can't you add a --SKIPIF-- true to them? that would at least make the test comply.
 [2010-03-26 04:08 UTC] kguest (Ken Guest)
If you are happy with tests failing because the tested functionality hasn't even been implemented yet, please either remove those tests as they are redundant or add a skipif section as required.
 [2010-04-26 02:45 UTC] dufuz (Helgi Þormar Þorbjörnsson)
-Status: Open +Status: Closed -Assigned To: +Assigned To: dufuz
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.