Revision: 21341
Author: [email protected]
Date: Fri May 16 13:18:13 2014 UTC
Log: Move check for existence of generated tests from run-tests.py to
presubmit.py
Inspired by https://codereview.chromium.org/275143002#msg3
[email protected]
Review URL: https://codereview.chromium.org/290023002
http://code.google.com/p/v8/source/detail?r=21341
Modified:
/branches/bleeding_edge/tools/presubmit.py
/branches/bleeding_edge/tools/run-tests.py
=======================================
--- /branches/bleeding_edge/tools/presubmit.py Fri May 9 12:59:24 2014 UTC
+++ /branches/bleeding_edge/tools/presubmit.py Fri May 16 13:18:13 2014 UTC
@@ -306,7 +306,8 @@
if self.IgnoreDir(dir_part):
break
else:
- if self.IsRelevant(file) and not self.IgnoreFile(file):
+ if (self.IsRelevant(file) and os.path.exists(file)
+ and not self.IgnoreFile(file)):
result.append(join(path, file))
if output.wait() == 0:
return result
@@ -416,6 +417,13 @@
return success
+def CheckGeneratedRuntimeTests(workspace):
+ code = subprocess.call(
+ [sys.executable,
join(workspace, "tools", "generate-runtime-tests.py"),
+ "check"])
+ return code == 0
+
+
def GetOptions():
result = optparse.OptionParser()
result.add_option('--no-lint', help="Do not run cpplint", default=False,
@@ -434,6 +442,7 @@
print "Running copyright header, trailing whitespaces and " \
"two empty lines between declarations check..."
success = SourceProcessor().Run(workspace) and success
+ success = CheckGeneratedRuntimeTests(workspace) and success
if success:
return 0
else:
=======================================
--- /branches/bleeding_edge/tools/run-tests.py Wed May 14 13:30:57 2014 UTC
+++ /branches/bleeding_edge/tools/run-tests.py Fri May 16 13:18:13 2014 UTC
@@ -336,13 +336,8 @@
workspace = os.path.abspath(join(os.path.dirname(sys.argv[0]), ".."))
if not options.no_presubmit:
print ">>> running presubmit tests"
- code = subprocess.call(
+ exit_code = subprocess.call(
[sys.executable, join(workspace, "tools", "presubmit.py")])
- exit_code = code
- code = subprocess.call(
- [sys.executable,
join(workspace, "tools", "generate-runtime-tests.py"),
- "check"])
- exit_code = exit_code or code
suite_paths = utils.GetSuitePaths(join(workspace, "test"))
--
--
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.
