Attempt at normalizing error stacks across browsers.

Failed expectations now have a `stack` property, remove `trace.stack`

lib/jasmine-core/jasmine-html.js

@@ -79,9 +79,8 @@ jasmine.HtmlReporter = function(options) {
 
       for (var i = 0; i < result.failedExpectations.length; i++) {
         var expectation = result.failedExpectations[i];
-        var stack = (expectation.trace && expectation.trace.stack) || "";
         messages.appendChild(createDom("div", {className: "result-message"}, expectation.message));
-        messages.appendChild(createDom("div", {className: "stack-trace"}, stack));
+        messages.appendChild(createDom("div", {className: "stack-trace"}, expectation.stack));
       }
 
       failures.push(failure);

lib/jasmine-core/jasmine.js

@@ -439,29 +439,66 @@ jasmine.util.argsToArray = function(args) {
   var arrayOfArgs = [];
   for (var i = 0; i < args.length; i++) arrayOfArgs.push(args[i]);
   return arrayOfArgs;
-};jasmine.exceptionFormatter = function(e) {
+};jasmine.ExceptionFormatter = function() {
-  var message = e.name
+  this.message = function(error) {
+    var message = error.name
       + ': '
-    + e.message
+      + error.message;
-    + ' in '
+
-    + (e.fileName || e.sourceURL || '')
+    if (error.fileName || error.sourceURL) {
-    + ' (line '
+      message += " in " + (error.fileName || error.sourceURL);
-    + (e.line || e.lineNumber || '')
+    }
-    + ')';
+
+    if (error.line || error.lineNumber) {
+      message += " (line " + (error.line || error.lineNumber) + ")"
+    }
 
     return message;
   };
-//TODO: expectation result may make more sense as a presentation of an expectation.
+
-jasmine.buildExpectationResult = function(params) {
+  this.stack = function(error) {
+    return error ? error.stack : null;
+  }
+};//TODO: expectation result may make more sense as a presentation of an expectation.
+jasmine.buildExpectationResult = function(options) {
+  var messageFormatter = options.messageFormatter || function() {},
+    stackFormatter = options.stackFormatter || function() {};
+
   return {
-    type: 'expect',
+    matcherName: options.matcherName,
-    matcherName: params.matcherName,
+    expected: options.expected,
-    expected: params.expected,
+    actual: options.actual,
-    actual:  params.actual,
+    message: message(),
-    message: params.passed ? 'Passed.' : params.message,
+    stack: stack(),
-    trace: params.passed ? '' : (params.trace || new Error(this.message)),
+    passed: options.passed
-    passed: params.passed
   };
+
+  function message() {
+    if (options.passed) {
+      return "Passed."
+    } else if (options.message) {
+      return options.message
+    } else if (options.error) {
+      return messageFormatter(options.error);
+    }
+    return ""
+  }
+
+  function stack() {
+    if (options.passed) {
+      return "";
+    }
+
+    var error = options.error;
+    if (!error) {
+      try {
+        throw new Error(message());
+      } catch (e) {
+        error = e;
+      }
+    }
+    return stackFormatter(error);
+  }
 };
 (function() {
   jasmine.Env = function(options) {
@@ -536,16 +573,19 @@ jasmine.buildExpectationResult = function(params) {
       }
     };
 
-    var exceptionFormatter = jasmine.exceptionFormatter;
-
     var specConstructor = jasmine.Spec;
 
     var getSpecName = function(spec, currentSuite) {
       return currentSuite.getFullName() + ' ' + spec.description + '.';
     };
 
-    var buildExpectationResult = jasmine.buildExpectationResult;
+    // TODO: we may just be able to pass in the fn instead of wrapping here
-    var expectationResultFactory = function(attrs) {
+    var buildExpectationResult = jasmine.buildExpectationResult,
+      exceptionFormatter = new jasmine.ExceptionFormatter(),
+      expectationResultFactory = function(attrs) {
+        attrs.messageFormatter = exceptionFormatter.message;
+        attrs.stackFormatter = exceptionFormatter.stack;
+
         return buildExpectationResult(attrs);
       };
 
@@ -1043,21 +1083,19 @@ jasmine.Matchers.matcherFn_ = function(matcherName, matcherFunction) {
         message += ".";
       }
     }
-    var expectationResult = jasmine.buildExpectationResult({
+
+    this.spec.addExpectationResult(result, {
       matcherName: matcherName,
       passed: result,
       expected: matcherArgs.length > 1 ? matcherArgs : matcherArgs[0],
       actual: this.actual,
       message: message
     });
-    this.spec.addExpectationResult(result, expectationResult);
     return jasmine.undefined;
   };
 };
 
 
-
-
 /**
  * toBe: compares the actual to the expected using ===
  * @param expected
@@ -1583,9 +1621,10 @@ jasmine.Spec = function(attrs) {
 
 jasmine.Spec.prototype.addExpectationResult = function(passed, data) {
   this.encounteredExpectations = true;
-  if (!passed) {
+  if (passed) {
-    this.result.failedExpectations.push(data);
+    return;
   }
+  this.result.failedExpectations.push(this.expectationResultFactory(data));
 };
 
 jasmine.Spec.prototype.expect = function(actual) {
@@ -1608,14 +1647,13 @@ jasmine.Spec.prototype.execute = function(onComplete) {
   this.queueRunner({
     fns: allFns,
     onException: function(e) {
-      self.addExpectationResult(false, self.expectationResultFactory({
+      self.addExpectationResult(false, {
         matcherName: "",
         passed: false,
         expected: "",
         actual: "",
-        message: self.exceptionFormatter(e),
+        error: e
-        trace: e
+      });
-      }));
     },
     onComplete: complete
   });
@@ -1652,7 +1690,7 @@ jasmine.Spec.prototype.status = function() {
 
 jasmine.Spec.prototype.getFullName = function() {
   return this.getSpecName(this);
-}
+};
 jasmine.Suite = function(attrs) {
   this.env = attrs.env;
   this.id = attrs.id;

spec/core/ExceptionFormatterSpec.js

@@ -1,13 +1,14 @@
 describe("ExceptionFormatter", function() {
-
+  describe("#message", function() {
     it('formats Firefox exception messages', function() {
       var sampleFirefoxException = {
           fileName: 'foo.js',
-        line: '1978',
+          lineNumber: '1978',
           message: 'you got your foo in my bar',
           name: 'A Classic Mistake'
         },
-      message = jasmine.exceptionMessageFor(sampleFirefoxException);
+        exceptionFormatter = new jasmine.ExceptionFormatter(),
+        message = exceptionFormatter.message(sampleFirefoxException);
 
       expect(message).toEqual('A Classic Mistake: you got your foo in my bar in foo.js (line 1978)');
     });
@@ -15,12 +16,37 @@ describe("ExceptionFormatter", function() {
     it('formats Webkit exception messages', function() {
       var sampleWebkitException = {
           sourceURL: 'foo.js',
-        lineNumber: '1978',
+          line: '1978',
           message: 'you got your foo in my bar',
           name: 'A Classic Mistake'
         },
-      message = jasmine.exceptionMessageFor(sampleWebkitException);
+        exceptionFormatter = new jasmine.ExceptionFormatter(),
+        message = exceptionFormatter.message(sampleWebkitException);
 
       expect(message).toEqual('A Classic Mistake: you got your foo in my bar in foo.js (line 1978)');
     });
+
+    it('formats V8 exception messages', function() {
+      var sampleV8 = {
+          message: 'you got your foo in my bar',
+          name: 'A Classic Mistake'
+        },
+        exceptionFormatter = new jasmine.ExceptionFormatter(),
+        message = exceptionFormatter.message(sampleV8);
+
+      expect(message).toEqual('A Classic Mistake: you got your foo in my bar');
+
+    });
+  });
+
+  describe("#stack", function() {
+    it("formats stack traces from Webkit, Firefox or node.js", function() {
+      var error = new Error("an error");
+      expect(new jasmine.ExceptionFormatter().stack(error)).toMatch(/ExceptionFormatterSpec\.js.*\d+/)
+    });
+
+    it("returns null if no Error provided", function() {
+      expect(new jasmine.ExceptionFormatter().stack()).toBeNull();
+    });
+  });
 });

spec/core/ExpectationResultSpec.js

@@ -1,33 +1,47 @@
 describe("buildExpectationResult", function() {
-
   it("defaults to passed", function() {
     var result = jasmine.buildExpectationResult({passed: 'some-value'});
     expect(result.passed).toBe('some-value');
   });
 
-  it("has a type of expect", function() {
-    var result = jasmine.buildExpectationResult({});
-    expect(result.type).toBe('expect');
-  });
-
   it("message defaults to Passed for passing specs", function() {
     var result = jasmine.buildExpectationResult({passed: true, message: 'some-value'});
     expect(result.message).toBe('Passed.');
   });
 
-  it("message returns the message for failing specs", function() {
+  it("message returns the message for failing expecations", function() {
     var result = jasmine.buildExpectationResult({passed: false, message: 'some-value'});
     expect(result.message).toBe('some-value');
   });
 
-  it("trace passes trace if exists", function() {
+  it("delegates message formatting to the provided formatter if there was an Error", function() {
-    var result = jasmine.buildExpectationResult({trace: 'some-value'});
+    var fakeError = {message: 'foo'},
-    expect(result.trace).toBe('some-value');
+      messageFormatter = jasmine.createSpy("exception message formatter").andReturn(fakeError.message);
+
+    var result = jasmine.buildExpectationResult(
+      {
+        passed: false,
+        error: fakeError,
+        messageFormatter: messageFormatter
       });
 
-  it("trace returns a new error if trace is falsy", function() {
+    expect(messageFormatter).toHaveBeenCalledWith(fakeError);
-    var result = jasmine.buildExpectationResult({trace: false});
+    expect(result.message).toEqual('foo');
-    expect(result.trace).toEqual(jasmine.any(Error));
+  });
+
+  it("delegates stack formatting to the provided formatter if there was an Error", function() {
+    var fakeError = {stack: 'foo'},
+      stackFormatter = jasmine.createSpy("stack formatter").andReturn(fakeError.stack);
+
+    var result = jasmine.buildExpectationResult(
+      {
+        passed: false,
+        error: fakeError,
+        stackFormatter: stackFormatter
+      });
+
+    expect(stackFormatter).toHaveBeenCalledWith(fakeError);
+    expect(result.stack).toEqual('foo');
   });
 
   it("matcherName returns passed matcherName", function() {
@@ -44,5 +58,4 @@ describe("buildExpectationResult", function() {
     var result = jasmine.buildExpectationResult({actual: 'some-value'});
     expect(result.actual).toBe('some-value');
   });
-
 });

spec/core/MatchersSpec.js

@@ -660,9 +660,6 @@ describe("jasmine.Matchers", function() {
     });
 
     it("should provide an inverted default message", function() {
-      match(37).not.toBeGreaterThan(42);
-      expect(lastResult().message).toEqual("Passed.");
-
       match(42).not.toBeGreaterThan(37);
       expect(lastResult().message).toEqual("Expected 42 not to be greater than 37.");
     });
@@ -677,14 +674,10 @@ describe("jasmine.Matchers", function() {
         }
       });
 
-      match(true).custom();
-      expect(lastResult().message).toEqual("Passed.");
       match(false).custom();
       expect(lastResult().message).toEqual("Expected it was called.");
       match(true).not.custom();
       expect(lastResult().message).toEqual("Expected it wasn't called.");
-      match(false).not.custom();
-      expect(lastResult().message).toEqual("Passed.");
     });
   });
 

spec/html/HtmlReporterSpec.js

@@ -369,10 +369,8 @@ describe("New HtmlReporter", function() {
           failedExpectations: [
             {
               message: "a failure message",
-              trace: {
               stack: "a stack trace"
             }
-            }
           ]
         };
         reporter.specStarted(failingResult);

src/core/Env.js

@@ -71,16 +71,19 @@
       }
     };
 
-    var exceptionFormatter = jasmine.exceptionFormatter;
-
     var specConstructor = jasmine.Spec;
 
     var getSpecName = function(spec, currentSuite) {
       return currentSuite.getFullName() + ' ' + spec.description + '.';
     };
 
-    var buildExpectationResult = jasmine.buildExpectationResult;
+    // TODO: we may just be able to pass in the fn instead of wrapping here
-    var expectationResultFactory = function(attrs) {
+    var buildExpectationResult = jasmine.buildExpectationResult,
+      exceptionFormatter = new jasmine.ExceptionFormatter(),
+      expectationResultFactory = function(attrs) {
+        attrs.messageFormatter = exceptionFormatter.message;
+        attrs.stackFormatter = exceptionFormatter.stack;
+
         return buildExpectationResult(attrs);
       };
 

src/core/ExceptionFormatter.js

@@ -1,12 +1,21 @@
-jasmine.exceptionMessageFor = function(e) {
+jasmine.ExceptionFormatter = function() {
-  var message = e.name
+  this.message = function(error) {
+    var message = error.name
       + ': '
-    + e.message
+      + error.message;
-    + ' in '
+
-    + (e.fileName || e.sourceURL || '')
+    if (error.fileName || error.sourceURL) {
-    + ' (line '
+      message += " in " + (error.fileName || error.sourceURL);
-    + (e.line || e.lineNumber || '')
+    }
-    + ')';
+
+    if (error.line || error.lineNumber) {
+      message += " (line " + (error.line || error.lineNumber) + ")"
+    }
 
     return message;
   };
+
+  this.stack = function(error) {
+    return error ? error.stack : null;
+  }
+};

src/core/ExpectationResult.js

@@ -1,12 +1,41 @@
 //TODO: expectation result may make more sense as a presentation of an expectation.
-jasmine.buildExpectationResult = function(params) {
+jasmine.buildExpectationResult = function(options) {
+  var messageFormatter = options.messageFormatter || function() {},
+    stackFormatter = options.stackFormatter || function() {};
+
   return {
-    type: 'expect',
+    matcherName: options.matcherName,
-    matcherName: params.matcherName,
+    expected: options.expected,
-    expected: params.expected,
+    actual: options.actual,
-    actual:  params.actual,
+    message: message(),
-    message: params.passed ? 'Passed.' : params.message,
+    stack: stack(),
-    trace: params.passed ? '' : (params.trace || new Error(this.message)),
+    passed: options.passed
-    passed: params.passed
   };
+
+  function message() {
+    if (options.passed) {
+      return "Passed."
+    } else if (options.message) {
+      return options.message
+    } else if (options.error) {
+      return messageFormatter(options.error);
+    }
+    return ""
+  }
+
+  function stack() {
+    if (options.passed) {
+      return "";
+    }
+
+    var error = options.error;
+    if (!error) {
+      try {
+        throw new Error(message());
+      } catch (e) {
+        error = e;
+      }
+    }
+    return stackFormatter(error);
+  }
 };

src/core/Matchers.js

@@ -53,21 +53,19 @@ jasmine.Matchers.matcherFn_ = function(matcherName, matcherFunction) {
         message += ".";
       }
     }
-    var expectationResult = jasmine.buildExpectationResult({
+
+    this.spec.addExpectationResult(result, {
       matcherName: matcherName,
       passed: result,
       expected: matcherArgs.length > 1 ? matcherArgs : matcherArgs[0],
       actual: this.actual,
       message: message
     });
-    this.spec.addExpectationResult(result, expectationResult);
     return jasmine.undefined;
   };
 };
 
 
-
-
 /**
  * toBe: compares the actual to the expected using ===
  * @param expected

src/core/Spec.js

@@ -26,9 +26,10 @@ jasmine.Spec = function(attrs) {
 
 jasmine.Spec.prototype.addExpectationResult = function(passed, data) {
   this.encounteredExpectations = true;
-  if (!passed) {
+  if (passed) {
-    this.result.failedExpectations.push(data);
+    return;
   }
+  this.result.failedExpectations.push(this.expectationResultFactory(data));
 };
 
 jasmine.Spec.prototype.expect = function(actual) {
@@ -51,14 +52,13 @@ jasmine.Spec.prototype.execute = function(onComplete) {
   this.queueRunner({
     fns: allFns,
     onException: function(e) {
-      self.addExpectationResult(false, self.expectationResultFactory({
+      self.addExpectationResult(false, {
         matcherName: "",
         passed: false,
         expected: "",
         actual: "",
-        message: self.exceptionFormatter(e),
+        error: e
-        trace: e
+      });
-      }));
     },
     onComplete: complete
   });
@@ -95,4 +95,4 @@ jasmine.Spec.prototype.status = function() {
 
 jasmine.Spec.prototype.getFullName = function() {
   return this.getSpecName(this);
-}
+};

src/html/HtmlReporter.js

@@ -79,9 +79,8 @@ jasmine.HtmlReporter = function(options) {
 
       for (var i = 0; i < result.failedExpectations.length; i++) {
         var expectation = result.failedExpectations[i];
-        var stack = (expectation.trace && expectation.trace.stack) || "";
         messages.appendChild(createDom("div", {className: "result-message"}, expectation.message));
-        messages.appendChild(createDom("div", {className: "stack-trace"}, stack));
+        messages.appendChild(createDom("div", {className: "stack-trace"}, expectation.stack));
       }
 
       failures.push(failure);