Merge pull request karma-runner#86 from r-park/fix-issue-79

fix(adapter): remove error message from stack prior to processing stack

src/adapter.js

@@ -53,7 +53,12 @@ function formatFailedStep(step) {
 
   var relevantMessage = [];
   var relevantStack = [];
-  var dirtyRelevantStack = getRelevantStackFrom(step.stack);
+
+  // Remove the message prior to processing the stack to prevent issues like
+  // https://github.com/karma-runner/karma-jasmine/issues/79
+  var stack = step.stack.replace('Error: ' + step.message, '');
+
+  var dirtyRelevantStack = getRelevantStackFrom(stack);
 
   // PhantomJS returns multiline error message for errors coming from specs
   // (for example when calling a non-existing function). This error is present

test/adapter.spec.js

@@ -201,6 +201,18 @@ describe('jasmine adapter', function(){
       expect( formatFailedStep(step) ).toBe( 'MESSAGE' );
     });
 
+
+    it('should properly format message containing new-line characters', function(){
+      // FF does not have the message in the stack trace
+
+      var step = {
+        passed  : false,
+        message : 'Jasmine fail\nmessage',
+        stack   : 'Error: Jasmine fail\nmessage\[email protected]:123'
+      };
+
+      expect( formatFailedStep(step) ).toMatch( 'Jasmine fail\nmessage\[email protected]:123' );
+    });
   });