third_party/android_testrunner/README.chromium - Issue 10824227: Organize adb install cmds and reboot on failure

Side by Side Diff: third_party/android_testrunner/README.chromium

Issue 10824227: Organize adb install cmds and reboot on failure (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 8 years, 4 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch | Annotate | Revision Log

« build/android/pylib/android_commands.py ('K') | « build/android/pylib/test_package_apk.py ('k') | third_party/android_testrunner/adb_interface.py » ('j') | third_party/android_testrunner/adb_interface.py » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 Name: Android Test runner script	1 Name: Android Test runner script

2 URL: http://source.android.com	2 URL: http://source.android.com

3 Version: 3.1.4	3 Version: 3.1.4

4 License: Apache Version 2.0	4 License: Apache Version 2.0

5 Security Critical: no	5 Security Critical: no

6	6

7 Description:	7 Description:

8 This package is the scripts used to run the unit test for Android and from	8 This package is the scripts used to run the unit test for Android and from

9 Android Gingerbread.	9 Android Gingerbread.

10	10

11 Local Modifications:	11 Local Modifications:

12 1. Added \|silent_log\| argument to \|StartInstrumentation\| so that output can be	12 1. Added \|silent_log\| argument to \|StartInstrumentation\| so that output can be

13 suppressed.	13 suppressed.

14 2. Changed error message handling in \|StartInstrumentation\| from \|shortMsg\| to	14 2. Changed error message handling in \|StartInstrumentation\| from \|shortMsg\| to

15 \|longMsg\| to provide more information when debugging.	15 \|longMsg\| to provide more information when debugging.

	16 3. Added optional cmd_logger arguments to \|SendCommand\| and \|SendShellCommand\|
	bulach 2012/08/09 12:36:52 hmm... I'm not sure this would be acceptable on th hmm... I'm not sure this would be acceptable on the upstream of this repository since it already contains a logging mechanim.. and it'd be nice to avoid forking... :) since both of them already log via their logger.SilentLog, which we can turn on via logger.SetVerbose(), how about just add that on the calling code? Isaac (away) 2012/08/10 04:44:35 I didn't do that because I'm not sure about the si Show quoted text On 2012/08/09 12:36:52, bulach wrote: > hmm... I'm not sure this would be acceptable on the upstream of this repository > since it already contains a logging mechanim.. and it'd be nice to avoid > forking... :) > > since both of them already log via their logger.SilentLog, which we can turn on > via logger.SetVerbose(), how about just add that on the calling code? I didn't do that because I'm not sure about the side effects of enabling the silentlogger and don't want to enable and disable for each command because it significantly complicates design.. I will just undo these changes I guess.
	17 so exact commands can be printed to logger of callers choice.

16	18

17 Here is the detail steps	19 Here is the detail steps

18 1. Checkout Android source code	20 1. Checkout Android source code

19	21

20 $ repo init -u git://android.git.kernel.org/platform/manifest.git -b gingerbread	22 $ repo init -u git://android.git.kernel.org/platform/manifest.git -b gingerbread

21 $ repo sync	23 $ repo sync

22 $ cd development	24 $ cd development

23 $ git reset --hard 76f63551d36b1de63c63f357e5f0646ed8c306bb	25 $ git reset --hard 76f63551d36b1de63c63f357e5f0646ed8c306bb

24	26

25 2. Copy the related files from	27 2. Copy the related files from

26 <android_gingerbread_tree>/development/testrunner/	28 <android_gingerbread_tree>/development/testrunner/

27	29

28 More information can be found in	30 More information can be found in

29 http://source.android.com/source/downloading.html	31 http://source.android.com/source/downloading.html

OLD	NEW