Skip to content

Commit 945dbea

Browse files
committed
Merge pull request #354 from sindresorhus/fix-error-stacks
Show full stack trace for non-assert errors
2 parents b79fdfd + cbea003 commit 945dbea

File tree

3 files changed

+9
-7
lines changed

3 files changed

+9
-7
lines changed

api.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,14 @@ Api.prototype._handleTest = function (test) {
8383
if (test.error.originalMessage) {
8484
message = test.error.originalMessage + ' ' + message;
8585
}
86+
8687
test.error.message = message;
8788
}
89+
90+
if (test.error.name !== 'AssertionError') {
91+
test.error.message = 'failed with "' + test.error.message + '"';
92+
}
93+
8894
this.errors.push(test);
8995
} else {
9096
test.error = null;

lib/test.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -125,12 +125,7 @@ Test.prototype.run = function () {
125125
self.exit();
126126
})
127127
.catch(function (err) {
128-
self._setAssertError(new assert.AssertionError({
129-
actual: err,
130-
message: 'Promise rejected → ' + err,
131-
operator: 'promise'
132-
}));
133-
128+
self._setAssertError(err);
134129
self.exit();
135130
});
136131
} else if (!this.metadata.callback) {

test/promise.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,8 @@ test('reject', function (t) {
245245
});
246246
}).run().catch(function (err) {
247247
t.ok(err);
248-
t.is(err.name, 'AssertionError');
248+
t.is(err.name, 'Error');
249+
t.is(err.message, 'unicorn');
249250
t.end();
250251
});
251252
});

0 commit comments

Comments
 (0)