Code review comment for lp:~chris.gagnon/unity8/autopilot-helper-to-background-and-signal-app

Revision history for this message
Leo Arias (elopio) wrote :

I'm more interested in this one than in the rest:
restart_unity_if_not_testable()

Can you put it in a Fixture? Something called like: UnityWithTestability.

About send_app_to_background, is it better to call the slots, or would it be better to tell autopilot to click the home button? I have a branch that opens the launcher and clicks, in case you prefer that.

Instead of currentFocusedAppID, use focusedApplicationId. Even better if you use the method from MainWindow: get_current_focused_app_id

review: Needs Fixing

« Back to merge proposal