Review Request 127778: Add test case for KRandom{,Sequence}
Michael Pyne
mpyne at kde.org
Thu Apr 28 01:10:43 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127778/
-----------------------------------------------------------
(Updated April 27, 2016, 6:10 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks.
Changes
-------
Submitted with commit 8f5cdadca7fad0265204d5f9ff16bc8ca1c02f72 by Michael Pyne to branch master.
Bugs: 362161
https://bugs.kde.org/show_bug.cgi?id=362161
Repository: kcoreaddons
Description
-------
A very simple collection of tests, but it's more tests than we had before. However this would not have caught the bug that prompted this since it would require running different processes consecutively to reproduce. But it's at least a base from which to proceed (hopefully with more patches).
Diffs
-----
autotests/CMakeLists.txt a7a6752
autotests/krandomtest.cpp PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/127778/diff/
Testing
-------
The new tests pass (and are supposed to pass), involving combinations of logically different tests in some cases (in other words I don't think I accidentally coded the test to always pass).
Thanks,
Michael Pyne
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160428/ddf51ba2/attachment.html>
More information about the Kde-frameworks-devel
mailing list