Review Request 127778: Add test case for KRandom{,Sequence}

Michael Pyne mpyne at kde.org
Thu Apr 28 00:49:16 UTC 2016


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

Review request for KDE Frameworks.


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/fc8a809a/attachment.html>


More information about the Kde-frameworks-devel mailing list