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

Unified Diff: tools/test_lib.py

Issue 8439032: Add option to command tester for running a test multiple times. Use that (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client/
Patch Set: xxx Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« tools/process_perf_combined.py ('K') | « tools/process_perf_combined.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/test_lib.py
===================================================================
--- tools/test_lib.py (revision 7086)
+++ tools/test_lib.py (working copy)
@@ -102,23 +102,6 @@
def PopenBufSize():
return 1000 * 1000
-def RunCmdWithInput(cmd, input_data):
- try:
- sys.stdout.flush() # Make sure stdout stays in sync on the bots.
- p = subprocess.Popen(cmd,
- bufsize=PopenBufSize(),
- stdin=subprocess.PIPE)
- p.communicate(input_data)
- retcode = p.wait()
- if retcode != 0:
- print ('Failed to cmd %s (retcode=%d)' %
- (cmd, retcode))
- return False
- except OSError:
- print 'RunCmdWithInput exception: ' + str(sys.exc_info()[1])
- return False
- return True
-
def RunTestWithInput(cmd, input_data):
"""Run a test where we only care about the return code."""
assert type(cmd) == list
« tools/process_perf_combined.py ('K') | « tools/process_perf_combined.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698