[telemetry_mini] Add AndroidActions class

[telemetry_mini] Add AndroidActions class

The class wraps around common user actions to perform on an Android
device during the execution of a story.

Actions may be repeated, and synthetic waits are inserted between
actions to make them more realistic.

BUG=chromium:752963
Review-Url: https://codereview.chromium.org/2996123002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/6b82af1025e0bd8c103398af4b9f342d61f105a9

[perezju, 2017-08-17 09:23:17]

perezju@chromium.org changed reviewers:
+ nednguyen@google.com

[nednguyen, 2017-08-17 10:28:58]

This loosk like our android_action_runner class :P

lgtm with a question

https://codereview.chromium.org/2996123002/diff/20001/experimental/telemetry_...
File experimental/telemetry_mini/telemetry_mini.py (right):

https://codereview.chromium.org/2996123002/diff/20001/experimental/telemetry_...
experimental/telemetry_mini/telemetry_mini.py:202: def GoBack(self, **kwargs):
why do you add these  **kwargs to just delete them later?

[perezju, 2017-08-17 12:00:51]

On 2017/08/17 10:28:58, nednguyen wrote:
> This loosk like our android_action_runner class :P

Yep. It's actually a nice exercise to be able to recreate and tweak the design
of existing Telemetry concepts with more flexibility, without having to worry
about implications of those changes on our huge existing code base.

Eventually we might be able to transfer some of the lessons learned here back to
"big" Telemetry. :)

> lgtm with a question
> 
>
https://codereview.chromium.org/2996123002/diff/20001/experimental/telemetry_...
> File experimental/telemetry_mini/telemetry_mini.py (right):
> 
>
https://codereview.chromium.org/2996123002/diff/20001/experimental/telemetry_...
> experimental/telemetry_mini/telemetry_mini.py:202: def GoBack(self, **kwargs):
> why do you add these  **kwargs to just delete them later?

I was also not supper happy about those. They are a workaround for some pylint
bug, and the alternative
(peppering all callers with "pylint: disable=foo") seemed far more terrible.
Added some docs on
the _UserAction decorator to explain the issue.

[perezju, 2017-08-17 12:05:03]

The CQ bit was checked by perezju@chromium.org

[perezju, 2017-08-17 12:05:03]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2996123002/#ps40001
(title: "explain **kwargs")

[commit-bot: I haz the power, 2017-08-17 12:05:09]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-08-17 12:32:01]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1502971503010850,
"parent_rev": "c3080ae7dd30591a9e88f4ddf0980c9113eec0d0", "commit_rev":
"6b82af1025e0bd8c103398af4b9f342d61f105a9"}

[commit-bot: I haz the power, 2017-08-17 12:32:05]

Description was changed from

==========
[telemetry_mini] Add AndroidActions class

The class wraps around common user actions to perform on an Android
device during the execution of a story.

Actions may be repeated, and synthetic waits are inserted between
actions to make them more realistic.

BUG=chromium:752963
==========

to

==========
[telemetry_mini] Add AndroidActions class

The class wraps around common user actions to perform on an Android
device during the execution of a story.

Actions may be repeated, and synthetic waits are inserted between
actions to make them more realistic.

BUG=chromium:752963

Review-Url: https://codereview.chromium.org/2996123002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-08-17 12:32:05]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...