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

Issue 9091006: Allow using '--option value' as well as '--option=value' in dart test script. (Closed)

Created:
8 years, 11 months ago by Mads Ager (google)
Modified:
8 years, 11 months ago
Reviewers:
Bill Hesse
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Allow using '--option value' as well as '--option=value' in dart test script. R=whesse@google.com BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=2945

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -1 line) Patch
M tools/testing/dart/test_options.dart View 1 chunk +8 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Mads Ager (google)
8 years, 11 months ago (2012-01-04 13:09:38 UTC) #1
Bill Hesse
LGTM. Do you want to add a test to say that --component --mode debug would ...
8 years, 11 months ago (2012-01-04 14:42:35 UTC) #2
Mads Ager (google)
8 years, 11 months ago (2012-01-04 14:48:49 UTC) #3
On 2012/01/04 14:42:35, Bill Hesse wrote:
> LGTM.  Do you want to add a test to say that --component --mode debug would be
> an error? (a string starting with -- as a value?)

I don't have to. All the options have a list of recognized values. Therefore,
--component --mode will already fail and say that --mode is an unknown value for
--component.

Powered by Google App Engine
This is Rietveld 408576698