Fix error reporting

- Error list is not reset by reference anymore
- All errors are include in the report, not only the first one

lib/runner/reporters/junit.js

@@ -51,11 +51,11 @@ module.exports = new (function() {
   function adaptErrMessages(module) {
     Object.keys(module.completed).forEach(function(item) {
       var testcase = module.completed[item];
-      var errmessages = testcase.errmessages;
-      if (errmessages.length) {
-        var stackParts = errmessages[0].split('\n');
-        stackParts.shift();
-        testcase.errorStackTrace = Utils.stackTraceFilter(stackParts);
+      var errorMessages = testcase.errmessages;
+      for (var i = 0; i < errorMessages.length; i++) {
+        if (errorMessages[i].stackTrace) {
+          errorMessages[i] = Utils.stackTraceFilter(errorMessages[i].split('\n'));
+        }
       }
     });
   }

lib/runner/reporters/junit.xml.ejs

@@ -8,16 +8,18 @@
     tests="<%= module.tests %>" time="<%= module.time %>" timestamp="<%= module.timestamp %>">
   <% for (var item in module.completed) {
     var testcase = module.completed[item];
-    var assertions = testcase.assertions %>
+    var assertions = testcase.assertions;
+    var errorMessages = testcase.errmessages || [];
+  %>
     <testcase name="<%= item %>" classname="<%= className %>" time="<%= testcase.time %>" assertions="<%= assertions.length %>"><%
       for (var i = 0; i < assertions.length; i++) { %><% if (assertions[i].failure) { %>  <failure message="<%= assertions[i].message %>"><%= assertions[i].stackTrace %></failure><% } %>
 <% if (assertions[i].screenshots && assertions[i].screenshots.length > 0) { %><system-out><% for (var j = 0; j < assertions[i].screenshots.length; j++) { %>[[ATTACHMENT|<%= assertions[i].screenshots[j] %>]]<% } %></system-out><% } %>
     <% }
     if (testcase.failed > 0 && testcase.stackTrace) {
     %><failure message="<%= testcase.message %>"><%= testcase.stackTrace %></failure><% }
-    if (testcase.errors > 0 && testcase.errorStackTrace) {
-    %>
-    <error message="Error executing test code"><%= testcase.errorStackTrace %></error><% } %>
+    for (var i = 0; i < errorMessages.length; i++) { %>
+      <error message="Error executing test code"><%= errorMessages[i] %></error>
+    <% } %>
     </testcase><% if (systemerr != '') {%>
       <system-err>
         <%= systemerr %>

lib/runner/run.js

@@ -102,7 +102,7 @@ Runner.prototype.runTestModule = function(modulePath, fullPaths) {
         timeMs : time,
         time : (time/1000).toPrecision(4),
         stackTrace : results.stackTrace,
-        errmessages: errors
+        errmessages: errors.slice()
       };
 
       if (Array.isArray(errors) && errors.length) {

package.json

@@ -1,7 +1,7 @@
 {
   "name": "nightwatch",
   "description": "A node.js bindings implementation for selenium 2.0/webdriver",
-  "version": "0.9.15.1",
+  "version": "0.9.15.2",
   "author": {
     "name": "Andrei Rusu",
     "email": "[email protected]"