diff options
author | George Burgess IV <gbiv@google.com> | 2016-08-28 23:10:29 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2016-08-29 10:25:35 -0700 |
commit | 479a91d1a637ec4e355699c1726806f01ac09cdd (patch) | |
tree | a302bda1d81632afaa11ce813899fd6f891f4dee /crosperf/experiment_factory.py | |
parent | afb8cc77e82c35faedfe541d097fc01fd1d7ca3d (diff) | |
download | toolchain-utils-479a91d1a637ec4e355699c1726806f01ac09cdd.tar.gz |
crosperf: Fix a failing test.
An earlier commit added extra spacing between remotes in
crosperf/default_remotes. This resulted in
experiment_factory_unittests.py reporting that there were 5 remotes in
some cases (2 of which were ''), since we were splitting on one space.
It seems sensible to split on any amount of whitespace, instead.
BUG=None
TEST=./run_tests.sh passes.
Change-Id: I95bd85ba92e5e1f5b30b0273b1b4a6c8853b1995
Reviewed-on: https://chrome-internal-review.googlesource.com/281538
Commit-Ready: George Burgess <gbiv@google.com>
Tested-by: George Burgess <gbiv@google.com>
Reviewed-by: Caroline Tice <cmtice@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 0689d033..e04f93f5 100644 --- a/crosperf/experiment_factory.py +++ b/crosperf/experiment_factory.py @@ -269,7 +269,7 @@ class ExperimentFactory(object): for line in f: key, v = line.split(':') if key.strip() == board: - remotes = v.strip().split(' ') + remotes = v.strip().split() if remotes: return remotes else: |