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

[Aleix Pol Gonzalez]

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


Ship it!




Ship It!

- Aleix Pol Gonzalez


On April 28, 2016, 2:49 a.m., Michael Pyne wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127778/
> -----------------------------------------------------------
> 
> (Updated April 28, 2016, 2:49 a.m.)
> 
> 
> 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/f3333855/attachment-0001.html>