google / mobly

E2E test framework for tests with complex environment requirements.
https://github.com/google/mobly
Apache License 2.0
628 stars 175 forks source link

Fix the behavior of `SnippetClient#is_alive`. #732

Closed xpconanfan closed 3 years ago

xpconanfan commented 3 years ago

SnippetClient#is_alive returns True even when it is paused, which propagates to the SnippetManagermentService. This is in direct contradiction to the definition of Service#is_alive.

This change fixes this problem, and cleans up the hacky code that attempted to cover this behavior.

Fixes #665


This change is Reviewable

xpconanfan commented 3 years ago

Travis CI is still broken. Forcing merge.