Code review comment for lp:~azzar1/unity/set-startup-id

Revision history for this message
Andrea Azzarone (azzar1) wrote :

> 92 + if self.app_is_running("Calculator"):
> 93 + self.skipTest("Calculator is already running.")
>
> Why not closing it if, instead of skipping the test?
>
> Anyway looks good.

Because a test should leave the system in the same state than it found it.

« Back to merge proposal