diff options
author | Zhizhou Yang <zhizhouy@google.com> | 2019-11-12 11:20:14 -0800 |
---|---|---|
committer | Zhizhou Yang <zhizhouy@google.com> | 2019-11-13 00:55:03 +0000 |
commit | bdeec3069191c9d6a84c918098b65543bbc297bc (patch) | |
tree | 2e4002fe05e61081285d4aec6c68e2643b20ee29 /crosperf/experiment_runner_unittest.py | |
parent | b4fb691319adfaf6558590d155bc326d13708251 (diff) | |
download | toolchain-utils-bdeec3069191c9d6a84c918098b65543bbc297bc.tar.gz |
crosperf: fix unittest error without prodaccess
After locking mechanism changed, crosperf checks if locks_dir is set or
not and by default uses X20 location. This requires prodaccess thus
unittests will fail without prodaccess if locks_dir is not explicitly
set. This patch set all experiment files in unittests to use /tmp for
locks_dir.
Also fixed some lint error for deprecated functions.
TEST=Passed all unittest.
BUG=None
Change-Id: Iff4623aad9632842010e5efb85a1e75a38bcf73e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1912767
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Zhizhou Yang <zhizhouy@google.com>
Tested-by: Zhizhou Yang <zhizhouy@google.com>
Diffstat (limited to 'crosperf/experiment_runner_unittest.py')
-rwxr-xr-x | crosperf/experiment_runner_unittest.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/crosperf/experiment_runner_unittest.py b/crosperf/experiment_runner_unittest.py index 2ec11ccd..8747de45 100755 --- a/crosperf/experiment_runner_unittest.py +++ b/crosperf/experiment_runner_unittest.py @@ -36,6 +36,7 @@ from cros_utils.file_utils import FileUtils EXPERIMENT_FILE_1 = """ board: parrot remote: chromeos-parrot1.cros chromreos-parrot2.cros + locks_dir: /tmp benchmark: kraken { suite: telemetry_Crosperf |