KUrl test false alarm
Peter Kümmel
syntheticpp at gmx.net
Wed Nov 11 19:53:53 CET 2009
Patrick Spendrin wrote:
> Peter Kümmel schrieb:
>> Looking at some unit tests I found that a test for KUrl fails:
>
>> FAIL! : KUrlTest::testSetFileName() Compared values are not the same
>> Actual (singleFile.path()): /bar.bin
>> Expected (QString("bar.bin")): bar.bin
>> ..\..\..\..\..\..\svn\trunk\KDE\kdelibs\kdecore\tests\kurltest.cpp(634) : failure location
>
>
>> In KUrl I found this comment:
>> //This is necessary because QUrl has the "path" part including the first slash
>> //Without this QUrl doesn't understand that this is a path, and some operations fail
>> //e.g. C:/blah needs to become /C:/blah
>
>> Therefore I assume the FAIL is a false alarm, see attached patch.
> replace path() with toLocalFile() should help.
But shouldn't we also test if on Windows the string returned by path() starts with a slash?
And there are no tests for toLocalFile, so I write some.
Peter
More information about the Kde-windows
mailing list