D14184: Reduce unnecessary long wait times on failing signal spies
Roman Gilg
noreply at phabricator.kde.org
Tue Jul 17 13:10:44 BST 2018
romangg created this revision.
romangg added a reviewer: Plasma.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
romangg requested review of this revision.
REVISION SUMMARY
Several signal spies, which were supposed to fail, were waiting for the
default 5 seconds, which is most often unnecessary long.
This patch sets a time limit of 500ms in suitable cases. Reduces for me
the total test time by 40%.
TEST PLAN
Before (with fixed seat and selection tests):
98% tests passed, 1 tests failed out of 45
Total Test time (real) = 173.31 sec
The following tests FAILED:
19 - kwayland-testWindowmanagement (Failed)
After:
98% tests passed, 1 tests failed out of 45
Total Test time (real) = 102.12 sec
The following tests FAILED:
19 - kwayland-testWindowmanagement (Failed)
REPOSITORY
R127 KWayland
BRANCH
autotestsWaits
REVISION DETAIL
https://phabricator.kde.org/D14184
AFFECTED FILES
autotests/client/test_drag_drop.cpp
autotests/client/test_pointer_constraints.cpp
autotests/client/test_selection.cpp
autotests/client/test_text_input.cpp
autotests/client/test_wayland_seat.cpp
autotests/client/test_wayland_surface.cpp
To: romangg, #plasma
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180717/84f9e248/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list