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

Issue 15001004: Android: fixes option to exclude specific annotated Android instrumentation tests. (Closed)

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

Description

Android: fixes option to exclude specific annotated Android instrumentation tests. Broken at https://chromiumcodereview.appspot.com/14704006/ Wrong number of parameters in run_uiautomator_tests.py BUG=169875, 239410 NOTRY=True TBR=sivachandra@chromium.org,craigdh@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199201

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M build/android/pylib/uiautomator/dispatch.py View 1 chunk +1 line, -1 line 1 comment Download

Messages

Total messages: 3 (0 generated)
bulach
https://codereview.chromium.org/15001004/diff/1/build/android/pylib/uiautomator/dispatch.py File build/android/pylib/uiautomator/dispatch.py (right): https://codereview.chromium.org/15001004/diff/1/build/android/pylib/uiautomator/dispatch.py#newcode35 build/android/pylib/uiautomator/dispatch.py:35: tests = test_pkg._GetAllMatchingTests( this shouldn't be calling a "_" ...
7 years, 7 months ago (2013-05-09 11:22:44 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bulach@chromium.org/15001004/1
7 years, 7 months ago (2013-05-09 11:24:07 UTC) #2
commit-bot: I haz the power
7 years, 7 months ago (2013-05-09 12:50:47 UTC) #3
Message was sent while issue was closed.
Change committed as 199201

Powered by Google App Engine
This is Rietveld 408576698