Review Request 112353: An attempt to fix sometimes failing KArchive test

Martin Klapetek martin.klapetek at gmail.com
Thu Aug 29 10:13:19 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112353/
-----------------------------------------------------------

Review request for KDE Frameworks.


Description
-------

One of the KArchive tests is to compare if the time of creating the archive file is the same as the time of a file inside of the archive. This can occasionally fail thanks to some msecs delay between the operations followed by rounding to seconds, so the times that are being compared can end up as "18:31:00" and "18:31:01", which then fails the test.

What I did is to take the time part out of the comparison and do a separate comparison of the times with allowed 1 sec difference.


Diffs
-----

  tier1/karchive/autotests/karchivetest.cpp 0b2ecdd 

Diff: http://git.reviewboard.kde.org/r/112353/diff/


Testing
-------

100% pass of KArchive tests in 10 consecutive runs.


Thanks,

Martin Klapetek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130829/71526ff5/attachment.html>


More information about the Kde-frameworks-devel mailing list