Reviewers: Jakob,
Message:
PTAL. Need to merge back for the infrastructure to work on all branches.
Description:
[test] Remove default for zero test cases.
BUG=
Please review this at https://codereview.chromium.org/1153103008/
Base URL: https://chromium.googlesource.com/v8/[email protected]
Affected files (+6, -8 lines):
M tools/run-tests.py
M tools/testrunner/local/progress.py
Index: tools/run-tests.py
diff --git a/tools/run-tests.py b/tools/run-tests.py
index
f7e77ca1543a1ab67c15c4fe4b90d58cf5737a57..a20b55ae86775f2e44ff43bf9a2d90def77e5503
100755
--- a/tools/run-tests.py
+++ b/tools/run-tests.py
@@ -584,10 +584,6 @@ def Execute(arch, mode, args, options, suites,
workspace):
if options.report:
verbose.PrintReport(all_tests)
- if num_tests == 0:
- print "No tests to run."
- return 0
-
# Run the tests, either locally or distributed on the network.
start_time = time.time()
progress_indicator = progress.PROGRESS_INDICATORS[options.progress]()
Index: tools/testrunner/local/progress.py
diff --git a/tools/testrunner/local/progress.py
b/tools/testrunner/local/progress.py
index
ed9d315b4ed36e4a2e90dd817e03f7d0794e621c..f303ab9736290c2946fc8092737c2fd0d4681cb4
100644
--- a/tools/testrunner/local/progress.py
+++ b/tools/testrunner/local/progress.py
@@ -192,10 +192,12 @@ class CompactProgressIndicator(ProgressIndicator):
def PrintProgress(self, name):
self.ClearLine(self.last_status_length)
elapsed = time.time() - self.start_time
+ progress = 0 if not self.runner.total else (
+ ((self.runner.total - self.runner.remaining) * 100) //
+ self.runner.total)
status = self.templates['status_line'] % {
'passed': self.runner.succeeded,
- 'remaining': (((self.runner.total - self.runner.remaining) * 100) //
- self.runner.total),
+ 'progress': progress,
'failed': len(self.runner.failed),
'test': name,
'mins': int(elapsed) / 60,
@@ -212,7 +214,7 @@ class ColorProgressIndicator(CompactProgressIndicator):
def __init__(self):
templates = {
'status_line': ("[%(mins)02i:%(secs)02i|"
- "\033[34m%%%(remaining) 4d\033[0m|"
+ "\033[34m%%%(progress) 4d\033[0m|"
"\033[32m+%(passed) 4d\033[0m|"
"\033[31m-%(failed) 4d\033[0m]: %(test)s"),
'stdout': "\033[1m%s\033[0m",
@@ -228,7 +230,7 @@ class
MonochromeProgressIndicator(CompactProgressIndicator):
def __init__(self):
templates = {
- 'status_line': ("[%(mins)02i:%(secs)02i|%%%(remaining) 4d|"
+ 'status_line': ("[%(mins)02i:%(secs)02i|%%%(progress) 4d|"
"+%(passed) 4d|-%(failed) 4d]: %(test)s"),
'stdout': '%s',
'stderr': '%s',
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
