I don't think this fix is related to lp:1335741. I also think it can be simplified as above.
lp:1335741 reports an error during the test, not during teardown and can be "fixed" by adding the following line to the start of client_can_query_current_orientation:
I don't think this fix is related to lp:1335741. I also think it can be simplified as above.
lp:1335741 reports an error during the test, not during teardown and can be "fixed" by adding the following line to the start of client_ can_query_ current_ orientation:
EXPECT_ FALSE(wait_ for_event( mir_event_ type_orientatio n, std::chrono: :milliseconds( 1)));
That isn't a *good* fix but it does strongly suggests that the problem there is with setup not teardown.
So, I suggest we should break the association with lp:1335741 and then land this (or something equivalent).