diff options
| author | Caroline Tice <cmtice@google.com> | 2015-12-09 12:42:13 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2015-12-10 00:02:46 +0000 |
| commit | 6e8726de19436e28c4d22c3f3dc3bbae87dd4c12 (patch) | |
| tree | 07b843f5713076e46ecc9d697dc55340c25966b3 /crosperf/experiment_factory.py | |
| parent | e55d35ee104a41e5f7ae96f4e326f33f6beaf412 (diff) | |
| download | toolchain-utils-6e8726de19436e28c4d22c3f3dc3bbae87dd4c12.tar.gz | |
Fix various Mock classes & calls so dry runs work.
The --dry_run option has been failing because parameter changes in
real classes have not always been reflected in Mock classes. This CL
fixes that.
BUG=chromium:545211
TEST=tested both real and --dry_run runs of Crosperf.
Change-Id: I0d0aa144e9fb3c40e1a95314691eacffb007cbe4
Reviewed-on: https://chrome-internal-review.googlesource.com/241525
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Diffstat (limited to 'crosperf/experiment_factory.py')
| -rw-r--r-- | crosperf/experiment_factory.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crosperf/experiment_factory.py b/crosperf/experiment_factory.py index 8e207db4..2ba291ba 100644 --- a/crosperf/experiment_factory.py +++ b/crosperf/experiment_factory.py @@ -234,7 +234,7 @@ class ExperimentFactory(object): if test_flag.GetTestMode(): # pylint: disable=too-many-function-args label = MockLabel(label_name, image, chromeos_root, board, my_remote, - image_args, cache_dir, cache_only, + image_args, cache_dir, cache_only, log_level, compiler, chrome_src) else: label = Label(label_name, image, chromeos_root, board, my_remote, |