Restart app before running monkey test.

build/android/run_monkey_test.py

 #!/usr/bin/env python
 # 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.
 
 """Runs the Monkey tests on one or more devices."""
 import logging
 import optparse
+import os
 import random
 import sys
 import time
 
+# Don't need these deps.

[bulach, 2012/10/30 10:52:50]

hmm, which bit needs this?
would it be possible to just set this where it's needed, then reset to the old
value afterwards?
it seems unsafe to modify environment variables like this.

something like:
  try:
    old_sdk_root = os.environ.get('ANDROID_SDK_ROOT')
    old_build_top = os.environ.get('ANDROID_BUILD_TOP')
    os.environ['ANDROID_SDK_ROOT'] = ''
    os.environ['ANDROID_BUILD_TOP'] = ''
    DoWhatever()
  finally:
    os.environ['ANDROID_SDK_ROOT'] = old_sdk_root
    os.environ['ANDROID_BUILD_TOP'] = old_build_top
  

[frankf, 2012/10/30 19:05:22]

Good point. I fixed the root cause by moving class constants in apk_info so
there's no look up during import.

On 2012/10/30 10:52:50, bulach wrote:

+os.environ['ANDROID_SDK_ROOT'] = ''
+os.environ['ANDROID_BUILD_TOP'] = ''
+
 from pylib import android_commands
 from pylib import python_test_base
 from pylib import python_test_sharder
 from pylib import test_options_parser
 from pylib import test_result
 
 
 class MonkeyTest(python_test_base.PythonTestBase):
   def testMonkey(self):
     start_ms = int(time.time()) * 1000
 
     # Launch and wait for Chrome to launch.
     self.adb.StartActivity(self.options.package_name,
                            self.options.activity_name,
+                           force_stop=True,
                            wait_for_completion=True,
                            action='android.intent.action.MAIN')
 
     # Chrome crashes are not always caught by Monkey test runner.
     # Verify Chrome has the same PID before and after the test.
     before_pids = self.adb.ExtractPid(self.options.package_name)
 
     # Run the test.
     output = ''
     duration_ms = 0
@@ skipping 46 matching lines @@
            '--throttle %d' % throttle,
            '-s %d' % seed,
            '-v ' * verbosity,
            '--monitor-native-crashes',
            '--kill-process-after-error',
            extra_args,
            '%d' % event_count]
     return self.adb.RunShellCommand(' '.join(cmd), timeout_time=timeout_ms)
 
 
-
 def DispatchPythonTests(options):
   """Dispatches the Monkey tests, sharding it if there multiple devices."""
   logger = logging.getLogger()
   logger.setLevel(logging.DEBUG)
 
   available_tests = [MonkeyTest('testMonkey')]
   attached_devices = android_commands.GetAttachedDevices()
   if not attached_devices:
     raise Exception('You have no devices attached or visible!')
 
   # Actually run the tests.
   logging.debug('Running monkey tests.')
   available_tests *= len(attached_devices)
-  options.ensure_value('shard_retries',1)
+  options.ensure_value('shard_retries', 1)
   sharder = python_test_sharder.PythonTestSharder(
       attached_devices, available_tests, options)
   result = sharder.RunShardedTests()
   result.LogFull('Monkey', 'Monkey', options.build_type)
   result.PrintAnnotation()
 
+
 def main():
   desc = 'Run the Monkey tests on 1 or more devices.'
   parser = optparse.OptionParser(description=desc)
   test_options_parser.AddBuildTypeOption(parser)
   parser.add_option('--package-name', help='Allowed package.')
   parser.add_option('--activity-name',
                     default='com.google.android.apps.chrome.Main',
                     help='Name of the activity to start [default: %default].')
   parser.add_option('--category',
                     help='A list of allowed categories [default: ""].')
@@ skipping 21 matching lines @@
     parser.error('Missing package name')
 
   if options.category:
     options.category = options.category.split(',')
 
   DispatchPythonTests(options)
 
 
 if __name__ == '__main__':
   main()