D28892: [autotests] Optimistic attempt to fix RemoteAccessTest reilability

David Edmundson noreply at phabricator.kde.org
Thu Apr 16 22:12:27 BST 2020


davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
  In this test we are waiting on 4 events. 2 things via 2 threads. It was
  unstable.
  
  This patch avoids hardcoding a bunch of ifs() handling recieving
  different orders, by waiting for both events.
  
  We can't use QTRY_COMPARE as ConnectionThread does magic things with
  QCoreApplication::eventDispatcher which don't work quite the same.
  
  This is a bit of a shot in the dark. It passes 100% of the time locally,
  lets see what CI manages to do :)

TEST PLAN
  Ran test :)

REPOSITORY
  R127 KWayland

BRANCH
  master

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

AFFECTED FILES
  autotests/client/test_remote_access.cpp

To: davidedmundson, #kwin
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200416/c4602ea5/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list