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