> 387 + TriggerLifecycleEvent(mir_lifecycle_state_resumed); > 388 + TriggerLifecycleEvent(mir_lifecycle_state_will_suspend); > 389 + > 390 + InSequence seq; > ... > > I'm surprised that sending the events before setting the expectations isn't > racy.
Actually it is.
Sticking in...
std::this_thread::sleep_for(std::chrono::seconds(1));
...shouldn't hurt but it does.
« Back to merge proposal
> 387 + TriggerLifecycl eEvent( mir_lifecycle_ state_resumed) ; eEvent( mir_lifecycle_ state_will_ suspend) ;
> 388 + TriggerLifecycl
> 389 +
> 390 + InSequence seq;
> ...
>
> I'm surprised that sending the events before setting the expectations isn't
> racy.
Actually it is.
Sticking in...
std: :this_thread: :sleep_ for(std: :chrono: :seconds( 1));
...shouldn't hurt but it does.