Diffusion KWin 72b9a75cd7a7

[autotests] Avoid useless waits in testToplevelOpenCloseAnimation

Authored by zzag on Jan 5 2019, 3:32 PM.

Description

[autotests] Avoid useless waits in testToplevelOpenCloseAnimation

Summary:
Use QTRY_VERIFY instead of qWait + QVERIFY to avoid useless waits.
Effects like scale or glide don't last longer than 200ms.

Test Plan:
Before:

$ dbus-run-session ctest -R testToplevelOpenCloseAnimation
Test project /home/vlad/Workspace/KDE/build/kde/workspace/kwin
    Start 123: kwin-testToplevelOpenCloseAnimation
1/2 Test #123: kwin-testToplevelOpenCloseAnimation ...............   Passed    6.43 sec
    Start 124: kwin-testToplevelOpenCloseAnimation-waylandonly
2/2 Test #124: kwin-testToplevelOpenCloseAnimation-waylandonly ...   Passed    6.33 sec

100% tests passed, 0 tests failed out of 2

Total Test time (real) =  12.76 sec

After:

$ dbus-run-session ctest -R testToplevelOpenCloseAnimation
Test project /home/vlad/Workspace/KDE/build/kde/workspace/kwin
    Start 123: kwin-testToplevelOpenCloseAnimation
1/2 Test #123: kwin-testToplevelOpenCloseAnimation ...............   Passed    2.05 sec
    Start 124: kwin-testToplevelOpenCloseAnimation-waylandonly
2/2 Test #124: kwin-testToplevelOpenCloseAnimation-waylandonly ...   Passed    2.00 sec

100% tests passed, 0 tests failed out of 2

Total Test time (real) =   4.05 sec

Reviewers: KWin, davidedmundson

Reviewed By: KWin, davidedmundson

Subscribers: davidedmundson, kwin

Tags: KWin

Differential Revision: https://phabricator.kde.org/D17989

Details

Committed
zzagJan 5 2019, 4:32 PM
Reviewer
KWin
Differential Revision
D17989: [autotests] Avoid useless waits in testToplevelOpenCloseAnimation
Parents
R108:6b3e55d6f515: Implement isPopupWindow for internal windows
Branches
Unknown
Tags
Unknown