This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1759/ |
On červen 2nd, 2014, 2:58 odp. UTC, Tomáš Bžatek wrote:
After long time trying to understand relations between test classes it finally makes sense to me. This set is simple and I have no objections against it.
It fails on my machine however, I'm trying to figure out what's wrong at the moment (likely a race condition somewhere in the provider).
On červen 2nd, 2014, 4:01 odp. UTC, Tomáš Bžatek wrote:
So yeah, failed tests usually mean that changes have been made in burst and indication manager didn't find any difference as a new user has been created and deleted right afterwards, all this before indication manager notices. Adding 1 sec. sleeps at the right places makes the tests succeed.
Also this is probably perfectly fine:
..expected stream: '50ac,50ad'
..got: '49ac,ad,ac,49ad'I wouldn't count on specific indication order delivery as things are async from the nature. Unless there's a settle timeout in the "test_round_50" case.
- Tomáš
On květen 21st, 2014, 3:22 odp. UTC, Alois Mahdal wrote:
Review request for OpenLMI Developers.
By Alois Mahdal.
Updated Kvě. 21, 2014, 3:22 odp.
Repository:
openlmi-providers
Description
Diffs
|