Log the stack trace for failures, where available

w3c · Mar 12, 2015 · 3427fed · 3427fed
1 parent a86ab86
commit 3427fed
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 5 deletions.
diff --git a/wptrunner/executors/base.py b/wptrunner/executors/base.py
@@ -60,7 +60,7 @@ def __call__(self, test, result):
         harness_result = test.result_cls(self.harness_codes[result["status"]], result["message"])
         return (harness_result,
                 [test.subtest_result_cls(subtest["name"], self.test_codes[subtest["status"]],
-                                         subtest["message"]) for subtest in result["tests"]])
+                                         subtest["message"], subtest.get("stack", None)) for subtest in result["tests"]])
 testharness_result_converter = TestharnessResultConverter()
 
 

diff --git a/wptrunner/executors/testharness_marionette.js b/wptrunner/executors/testharness_marionette.js
@@ -8,12 +8,13 @@ window.wrappedJSObject.explicit_timeout = %(explicit_timeout)d;
 window.wrappedJSObject.done = function(tests, status) {
   clearTimeout(timer);
   var test_results = tests.map(function(x) {
-    return {name:x.name, status:x.status, message:x.message}
+    return {name:x.name, status:x.status, message:x.message, stack:x.stack}
   });
   marionetteScriptFinished({test:"%(url)s",
                             tests:test_results,
                             status: status.status,
-                            message: status.message});
+                            message: status.message,
+                            stack: status.stack});
 }
 
 window.wrappedJSObject.win = window.open("%(abs_url)s", "%(window_id)s");

diff --git a/wptrunner/testrunner.py b/wptrunner/testrunner.py
@@ -498,7 +498,8 @@ def test_ended(self, test, results):
                                     result.name,
                                     result.status,
                                     message=result.message,
-                                    expected=expected)
+                                    expected=expected,
+                                    stack=result.stack)
 
         # TODO: consider changing result if there is a crash dump file
 

diff --git a/wptrunner/wpttest.py b/wptrunner/wpttest.py
@@ -21,13 +21,14 @@ def __init__(self, status, message, expected=None, extra=None):
 
 
 class SubtestResult(object):
-    def __init__(self, name, status, message, expected=None):
+    def __init__(self, name, status, message, stack=None, expected=None):
         self.name = name
         if status not in self.statuses:
             raise ValueError("Unrecognised status %s" % status)
         self.status = status
         self.message = message
         self.expected = expected
+        self.stack = stack
 
 
 class TestharnessResult(Result):