Skip to content

Commit

Permalink
Include fullName in formattedAssertion (jestjs#4273)
Browse files Browse the repository at this point in the history
  • Loading branch information
aifreedom authored and cpojer committed Aug 15, 2017
1 parent 8226ea7 commit 04b4c8c
Show file tree
Hide file tree
Showing 3 changed files with 40 additions and 1 deletion.
37 changes: 37 additions & 0 deletions packages/jest-util/src/__tests__/format_test_results_test.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
/**
* Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
*
* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*
*/

'use strict';

const formatTestResults = require('../formatTestResults');

describe('formatTestResults', () => {
const assertion = {
fullName: 'TestedModule#aMethod when some condition is met returns true',
status: 'passed',
title: 'returns true',
};

const results = {
testResults: [
{
numFailingTests: 0,
perfStats: {end: 2, start: 1},
testResults: [assertion],
},
],
};

it('includes test full name', () => {
const result = formatTestResults(results, null, null);
expect(result.testResults[0].assertionResults[0].fullName).toEqual(
assertion.fullName,
);
});
});
1 change: 1 addition & 0 deletions packages/jest-util/src/format_test_results.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ function formatTestAssertion(
): FormattedAssertionResult {
const result: FormattedAssertionResult = {
failureMessages: null,
fullName: assertion.fullName,
status: assertion.status,
title: assertion.title,
};
Expand Down
3 changes: 2 additions & 1 deletion types/TestResult.js
Original file line number Diff line number Diff line change
Expand Up @@ -94,9 +94,10 @@ export type AssertionResult = {|
|};

export type FormattedAssertionResult = {
failureMessages: Array<string> | null,
fullName: string,
status: Status,
title: string,
failureMessages: Array<string> | null,
};

export type AggregatedResultWithoutCoverage = {
Expand Down

0 comments on commit 04b4c8c

Please sign in to comment.