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

Issue 217043005: Mojo: Make the mojob.sh unit test runner skip previously-passed tests. (Closed)

Created:
6 years, 9 months ago by viettrungluu
Modified:
6 years, 9 months ago
CC:
chromium-reviews, Aaron Boodman, darin (slow to review), viettrungluu+watch_chromium.org, ben+mojo_chromium.org, abarth-chromium
Visibility:
Public.

Description

Mojo: Make the mojob.sh unit test runner skip previously-passed tests. It works for me on my Linux box. It only supports binary tests, not Python. :/ R=darin@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260321

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+215 lines, -9 lines) Patch
A mojo/tools/data/unittests View 1 chunk +16 lines, -0 lines 0 comments Download
M mojo/tools/mojob.sh View 1 chunk +2 lines, -9 lines 0 comments Download
A mojo/tools/pylib/transitive_hash.py View 1 chunk +89 lines, -0 lines 0 comments Download
A mojo/tools/test_runner.py View 1 chunk +108 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
viettrungluu
6 years, 9 months ago (2014-03-28 19:48:51 UTC) #1
darin (slow to review)
LGTM wrong python style guide? :)
6 years, 9 months ago (2014-03-28 22:40:48 UTC) #2
viettrungluu
On 2014/03/28 22:40:48, darin wrote: > LGTM > > wrong python style guide? :) Sssssh. ...
6 years, 9 months ago (2014-03-28 23:23:05 UTC) #3
viettrungluu
6 years, 9 months ago (2014-03-29 00:37:49 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r260321 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698