diff options
| author | Cassidy Burden <cburden@google.com> | 2016-06-14 16:47:02 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-06-15 10:53:41 -0700 |
| commit | 9c665a01b55ee4e672b29ff352989a640eac084f (patch) | |
| tree | 87d3b9e0fbcc4874aa94ec8a5e196ffba1092b00 /binary_search_tool/test/is_good.py | |
| parent | 51ba54f3375c17e2a33c1854b635b144802457a4 (diff) | |
| download | toolchain-utils-9c665a01b55ee4e672b29ff352989a640eac084f.tar.gz | |
Fix binary search tool install unit tests
Change from using module wide variable common.installed to using a file
called "./installed". Previously common.installed wouldn't actually be
set properly because there would be two separate instances of Python
running.
TEST=More robust install unit test made
Change-Id: Iee87a52e0f54fc63837b25918764e804747525db
Reviewed-on: https://chrome-internal-review.googlesource.com/264745
Commit-Ready: Cassidy Burden <cburden@google.com>
Tested-by: Cassidy Burden <cburden@google.com>
Reviewed-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'binary_search_tool/test/is_good.py')
| -rwxr-xr-x | binary_search_tool/test/is_good.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/binary_search_tool/test/is_good.py b/binary_search_tool/test/is_good.py index e2ab4642..f7cc8af7 100755 --- a/binary_search_tool/test/is_good.py +++ b/binary_search_tool/test/is_good.py @@ -3,13 +3,14 @@ from __future__ import print_function +import os import sys import common def Main(): - if not common.installed: + if not os.path.exists('./installed'): return 1 working_set = common.ReadWorkingSet() for w in working_set: |