[autotests] Optimistic attempt to fix RemoteAccessTest reilability
ClosedPublic

Authored by davidedmundson on Apr 16 2020, 9:12 PM.

Details

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 :)

Diff Detail

Repository
R127 KWayland
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Apr 16 2020, 9:12 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 16 2020, 9:12 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
davidedmundson requested review of this revision.Apr 16 2020, 9:12 PM
zzag accepted this revision.Apr 17 2020, 7:22 AM
zzag added a subscriber: zzag.

This seems super complicated.

This revision is now accepted and ready to land.Apr 17 2020, 7:22 AM
This revision was automatically updated to reflect the committed changes.

This seems super complicated.

It is :D