Skip to content

Commit 293f50d

Browse files
mlucoolMarc Udoff
authored andcommitted
Show diff in xunit
1 parent f9c650c commit 293f50d

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

lib/reporters/xunit.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,8 @@ XUnit.prototype.write = function(line) {
151151
* @param {Test} test
152152
*/
153153
XUnit.prototype.test = function(test) {
154+
Base.useColors = false;
155+
154156
var attrs = {
155157
classname: test.parent.fullTitle(),
156158
name: test.title,
@@ -159,6 +161,10 @@ XUnit.prototype.test = function(test) {
159161

160162
if (test.state === 'failed') {
161163
var err = test.err;
164+
var diff =
165+
Base.hideDiff || !err.actual || !err.expected
166+
? ''
167+
: '\n' + Base.generateDiff(err.actual, err.expected);
162168
this.write(
163169
tag(
164170
'testcase',
@@ -168,7 +174,7 @@ XUnit.prototype.test = function(test) {
168174
'failure',
169175
{},
170176
false,
171-
escape(err.message) + '\n' + escape(err.stack)
177+
escape(err.message) + escape(diff) + '\n' + escape(err.stack)
172178
)
173179
)
174180
);

test/reporters/xunit.spec.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@ describe('XUnit reporter', function() {
1818
var expectedClassName = 'fullTitle';
1919
var expectedTitle = 'some title';
2020
var expectedMessage = 'some message';
21+
var expectedDiff =
22+
'\n + expected - actual\n\n -foo\n +bar\n ';
2123
var expectedStack = 'some-stack';
2224
var expectedWrite = null;
2325

@@ -214,6 +216,8 @@ describe('XUnit reporter', function() {
214216
},
215217
duration: 1000,
216218
err: {
219+
actual: 'foo',
220+
expected: 'bar',
217221
message: expectedMessage,
218222
stack: expectedStack
219223
}
@@ -235,6 +239,8 @@ describe('XUnit reporter', function() {
235239
'" time="1"><failure>' +
236240
expectedMessage +
237241
'\n' +
242+
expectedDiff +
243+
'\n' +
238244
expectedStack +
239245
'</failure></testcase>';
240246

0 commit comments

Comments
 (0)