Add monkey command to android_commands.

build/android/pylib/android_commands.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Provides an interface to communicate with the device via the adb command.
 
 Assumes adb binary is currently on system path.
 """
 
 import collections
 import datetime
 import logging
 import os
+import random
 import re
 import shlex
 import subprocess
 import sys
 import tempfile
 import time
 
 import pexpect
 import io_stats_parser
 
@@ skipping 281 matching lines @@
     uninstall_command = 'uninstall %s' % package
 
     logging.info('>>> $' + uninstall_command)
     return self._adb.SendCommand(uninstall_command, timeout_time=60)
 
   def Install(self, package_file_path, reinstall=False):
     """Installs the specified package to the device.
 
     Args:
       package_file_path: Path to .apk file to install.
-      reinstall: Whether to reinstall over existing package
+      reinstall: Reinstall an existing apk.
 
     Returns:
       A status string returned by adb install
     """
     assert os.path.isfile(package_file_path)
 
     if reinstall:
       install_cmd = 'install -r %s'
     else:
       install_cmd = 'install %s'
 
     return self._adb.SendCommand(install_cmd % package_file_path,
                                  timeout_time=2*60, retry_count=0)
 
-  def ManagedInstall(self, apk_path, keep_data, package_name=None,
+  def ManagedInstall(self, apk_path, package_name=None, reinstall=False,
                      reboots_on_failure=2):
     """Installs specified package and reboots device on timeouts.
 
     Args:
       apk_path: Path to .apk file to install.
-      keep_data: Whether to keep data if package already exists
-      package_name: Package name (only needed if keep_data=False)
+      package_name: Package name (only needed if reinstall=False)
+      reinstall: Whether to reinstall instead of uninstalling first.

[Isaac (away), 2012/08/22 02:03:28]

I know I said it was OK earlier but I think reinstall is more confusing than
keep_data.  Would rather keep the current param name and add more info in the
comment here about what it does (i.e. if false, uninstall/install, if true,
install -r.

[frankf, 2012/08/22 21:46:55]

Done.

       reboots_on_failure: number of time to reboot if package manager is frozen.
 
     Returns:
       A status string returned by adb install
     """
     reboots_left = reboots_on_failure
     while True:
       try:
-        if not keep_data:
+        if not reinstall:
+          assert package_name
           self.Uninstall(package_name)
-        install_status = self.Install(apk_path, keep_data)
+        install_status = self.Install(apk_path, reinstall=reinstall)
         if 'Success' in install_status:
           return install_status
       except errors.WaitForResponseTimedOutError:
         print '@@@STEP_WARNINGS@@@'
         logging.info('Timeout on installing %s' % apk_path)
 
       if reboots_left <= 0:
         raise Exception('Install failure')
 
       # Force a hard reboot on last attempt
@@ skipping 621 matching lines @@
           if len(process_results) <= 8:
             continue
           # Column 0 is the executable name
           # Column 1 is the pid
           # Column 8 is the Inode in use
           if process_results[8] == socket_name:
             pids.append((int(process_results[1]), process_results[0]))
         break
     logging.info('PidsUsingDevicePort: %s', pids)
     return pids
+
+  def RunMonkey(self, package_name, category=None, throttle=100, seed=None,
+                event_count=10000, verbose=1, extra_args=''):
+    """Runs monkey test for a given package.
+
+    Args:
+      package_name: Allowed package.
+      category: A list of allowed categories.
+      throttle: Delay between events (ms).
+      seed: Seed value for pseduo-random generator. Same seed value
+        generates the same sequence of events. Seed is randomized by
+        default.
+      event_count: Number of events to generate.
+      verbose: Verbosity level [0-3].
+      extra_args: A string of other args to pass to the command verbatim.
+
+    Returns:
+      Output of the test run.
+    """
+    if category is None:

[Isaac (away), 2012/08/22 02:03:28]

the preferred way of doing this is:

category = category or []

[frankf, 2012/08/22 21:46:55]

Done.

+      category = []
+
+    if not seed:
+      seed = random.randint(1, 100)

[Isaac (away), 2012/08/22 02:03:28]

seed = seed or random.randint(1, 100)

[frankf, 2012/08/22 21:46:55]

Done.

+
+    cmd = ['monkey',
+           '-p %s' % package_name,
+           ' '.join(['-c %s' % c for c in category]),
+           '--throttle %d' % throttle,
+           '-s %d' % seed,
+           ' '.join(['-v'] * verbose),

[Isaac (away), 2012/08/22 02:03:28]

don't like this syntax -- can you figure out an alternate way to pass an
optional '-v' ?

[frankf, 2012/08/22 21:46:55]

The only other way is restrict verbose to a bool (i.e. either nothing or -v -v
-v). This is more flexible. Renamed verbose->verbosity to make it clear it's not
a bool.

On 2012/08/22 02:03:28, Isaac wrote:

[Isaac (away), 2012/08/22 21:53:19]

Oh, I didn't realize you wanted to pass multiple -v.

How about ' -v' * verbosity

+           extra_args,
+           '%s' % event_count]
+    return self.RunShellCommand(' '.join(cmd),
+                                timeout_time=event_count*throttle*1.5)