Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Issue 12256021: [Android] Split out the instrumentation test runner, sharder, and dispatcher. (Closed)

Created:
7 years, 10 months ago by craigdh
Modified:
7 years, 10 months ago
Reviewers:
frankf
CC:
chromium-reviews, klundberg+watch_chromium.org, frankf+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy+watch_chromium.org, nilesh
Visibility:
Public.

Description

[Android] Split out the instrumentation test runner, sharder, and dispatcher. Mostly just moves codes around. BUG=168889 TEST=run_instrumentation_tests.py NOTRY=True Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182892

Patch Set 1 #

Patch Set 2 : run_instrumentation_tests.py: TestResult -> test_result.TestResult #

Patch Set 3 : fix some host_driven imports #

Unified diffs Side-by-side diffs Delta from patch set Stats (+205 lines, -811 lines) Patch
M build/android/pylib/host_driven/java_unittest_utils.py View 1 2 2 chunks +3 lines, -3 lines 0 comments Download
M build/android/pylib/host_driven/python_test_base.py View 1 2 2 chunks +4 lines, -5 lines 0 comments Download
M build/android/pylib/host_driven/python_test_sharder.py View 2 chunks +1 line, -2 lines 0 comments Download
M build/android/pylib/host_driven/run_python_tests.py View 1 2 3 chunks +4 lines, -5 lines 0 comments Download
A build/android/pylib/instrumentation/dispatch.py View 1 chunk +95 lines, -0 lines 0 comments Download
D build/android/pylib/instrumentation/run_java_tests.py View 1 chunk +0 lines, -595 lines 0 comments Download
A + build/android/pylib/instrumentation/test_runner.py View 13 chunks +52 lines, -195 lines 0 comments Download
A build/android/pylib/instrumentation/test_sharder.py View 1 chunk +39 lines, -0 lines 0 comments Download
M build/android/run_instrumentation_tests.py View 1 2 chunks +7 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
craigdh
7 years, 10 months ago (2013-02-13 19:34:24 UTC) #1
frankf
lgtm, make sure to run host-driven tests downstream.
7 years, 10 months ago (2013-02-13 22:39:46 UTC) #2
frankf
nilesh: FYI
7 years, 10 months ago (2013-02-13 22:40:59 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/craigdh@chromium.org/12256021/2001
7 years, 10 months ago (2013-02-15 18:14:04 UTC) #4
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
7 years, 10 months ago (2013-02-15 18:47:06 UTC) #5
commit-bot: I haz the power
7 years, 10 months ago (2013-02-16 01:49:03 UTC) #6

Powered by Google App Engine
This is Rietveld 408576698