D17989: [autotests] Avoid useless waits in testToplevelOpenCloseAnimation

davidedmundson accepted this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.


  Good idea

INLINE COMMENTS

> toplevel_open_close_animation_test.cpp:139
> +    // Eventually, the animation will be complete.
> +    QTRY_VERIFY(!effect->isActive());
>  

There's a QTRY_VERIFY_WITH_TIMEOUT which would allow us to keep the check that everything lasted < 1s.

REPOSITORY
  R108 KWin

BRANCH
  avoid-useless-waits

REVISION DETAIL
  https://phabricator.kde.org/D17989

To: zzag, #kwin, davidedmundson
Cc: davidedmundson, kwin, kvanton, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart