Reviewers: Christian Plesner Hansen,

Description:
Added crash detection to tests on Linux.

Added the timeout condition to the CommandOutput class.

Please review this at http://codereview.chromium.org/8695

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
   M     test/mozilla/mozilla.status
   M     tools/test.py


Index: test/mozilla/mozilla.status
===================================================================
--- test/mozilla/mozilla.status (revision 637)
+++ test/mozilla/mozilla.status (working copy)
@@ -193,7 +193,7 @@
  # unicode version converts it to itself.
  ecma/String/15.5.4.12-5: FAIL_OK

-# Creates a linked list of arrays until we run out of memory.
+# Creates a linked list of arrays until we run out of memory or timeout.
  js1_5/Regress/regress-312588: FAIL_OK


Index: tools/test.py
===================================================================
--- tools/test.py       (revision 637)
+++ tools/test.py       (working copy)
@@ -314,8 +314,9 @@

  class CommandOutput(object):

-  def __init__(self, exit_code, stdout, stderr):
+  def __init__(self, exit_code, timed_out, stdout, stderr):
      self.exit_code = exit_code
+    self.timed_out = timed_out
      self.stdout = stdout
      self.stderr = stderr

@@ -372,7 +373,11 @@
      if platform.system() == 'Windows':
        return 0x80000000 & self.output.exit_code and not (0x3FFFFF00 &  
self.output.exit_code)
      else:
-      return False
+      # Timed out tests will have exit_code -signal.SIGTERM.
+      if self.output.timed_out:
+        return False
+      return self.output.exit_code < 0 and \
+             self.output.exit_code != -signal.SIGABRT

    def HasFailed(self):
      execution_failed = self.test.DidFail(self.output)
@@ -471,7 +476,7 @@
        PrintError(str(e))
    CheckedUnlink(outname)
    CheckedUnlink(errname)
-  return CommandOutput(exit_code, output, errors)
+  return CommandOutput(exit_code, timed_out, output, errors)


  def ExecuteNoCapture(args, context, timeout=None):
@@ -480,7 +485,7 @@
      timeout,
      args = args,
    )
-  return CommandOutput(exit_code, "", "")
+  return CommandOutput(exit_code, False, "", "")


  def CarCdr(path):



--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to