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