On Fri, Apr 10, 2009 at 2:36 PM, Nicolas Le Cam niko.lecam@gmail.com wrote:
2009/4/10, James Hawkins truiken@gmail.com:
On Fri, Apr 10, 2009 at 6:15 AM, Nicolas Le Cam niko.lecam@gmail.com wrote:
Try2: This time with the patch.
Tested on Win2k/WinXP. Marked two tests as todo_wine.
I was serious before. You can't change the old tests. Please add new tests for the exposed bugs. Same goes for patch 3/4.
-- James Hawkins
I will for test 4 but test 3 wasn't correct before. It was assuming drive was current one and that wasn't correct as revealed by my patch. This was only right if run on systemdrive and not from root drive dir.
You're missing the point. Even in 3/4 you are adding a todo_wine where there was none before. By definition, you are changing what is being tested. The test was not incorrect, it just didn't cover all corner cases. Like I said before, I have no problem with you adding a new test to tickle this bug you've found, but in your attempts to 'fix' the test for this corner case, you cannot add a todo_wine here. This requires you to acquire a deeper understanding of what is being tested instead of blindly throwing new code at the test.