Added compact print option for default reporter

packages/jest-cli/src/TestScheduler.js

@@ -290,7 +290,10 @@ export default class TestScheduler {
  this.addReporter(
  this._globalConfig.verbose
  ? new VerboseReporter(this._globalConfig)
- : new DefaultReporter(this._globalConfig),
+ : new DefaultReporter(
+ this._globalConfig,
+ getReporterOption('default', this._globalConfig.reporters),
+ ),
  );
 
  if (collectCoverage) {
@@ -384,3 +387,13 @@ const getEstimatedTime = (timings, workers) => {
  ? max
  : Math.max(timings.reduce((sum, time) => sum + time) / workers, max);
 };
+
+const getReporterOption = (
+ reporterName: string,
+ reporters: Array<string | ReporterConfig>,
+): Object => {
+ const config = reporters.find(
+ item => Array.isArray(item) && item[0] === reporterName,
+ );
+ return (config && config[1]) || {};
+};

packages/jest-cli/src/reporters/default_reporter.js

@@ -9,34 +9,43 @@
 
 /* global stream$Writable, tty$WriteStream */
 
-import type {AggregatedResult, TestResult} from 'types/TestResult';
+import type {
+ AggregatedResult,
+ AssertionResult,
+ TestResult,
+} from 'types/TestResult';
 import type {GlobalConfig, Path, ProjectConfig} from 'types/Config';
 import type {Test} from 'types/TestRunner';
 import type {ReporterOnStartOptions} from 'types/Reporters';
 
 import {clearLine, getConsoleOutput, isInteractive} from 'jest-util';
+import {formatFullTitle, TITLE_INDENT, TITLE_BULLET} from 'jest-message-util';
 import chalk from 'chalk';
 import BaseReporter from './base_reporter';
 import Status from './Status';
 import getResultHeader from './get_result_header';
 import getSnapshotStatus from './get_snapshot_status';
+import {formatTestPath, getLocation} from './utils';
 
 type write = (chunk: string, enc?: any, cb?: () => void) => boolean;
 type FlushBufferedOutput = () => void;
-
-const TITLE_BULLET = chalk.bold('\u25cf ');
+type DefaultReporterOptions = {
+ compact: boolean,
+};
 
 export default class DefaultReporter extends BaseReporter {
  _clear: string; // ANSI clear sequence for the last printed status
  _err: write;
  _globalConfig: GlobalConfig;
+ _options: DefaultReporterOptions;
  _out: write;
  _status: Status;
  _bufferedOutput: Set<FlushBufferedOutput>;
 
- constructor(globalConfig: GlobalConfig) {
+ constructor(globalConfig: GlobalConfig, options: ?DefaultReporterOptions) {
  super();
  this._globalConfig = globalConfig;
+ this._options = options || {};
  this._clear = '';
  this._out = process.stdout.write.bind(process.stdout);
  this._err = process.stderr.write.bind(process.stderr);
@@ -158,11 +167,19 @@ export default class DefaultReporter extends BaseReporter {
  test.context.config,
  testResult,
  );
- this.printTestFileFailureMessage(
- testResult.testFilePath,
- test.context.config,
- testResult,
- );
+ if (this._options.compact) {
+ this.printCompactFailureMessage(
+ testResult.testFilePath,
+ test.context.config,
+ testResult,
+ );
+ } else {
+ this.printTestFileFailureMessage(
+ testResult.testFilePath,
+ test.context.config,
+ testResult,
+ );
+ }
  }
  this.forceFlushBufferedOutput();
  }
@@ -184,7 +201,7 @@ export default class DefaultReporter extends BaseReporter {
  const consoleBuffer = result.console;
  if (consoleBuffer && consoleBuffer.length) {
  this.log(
- ' ' +
+ TITLE_INDENT +
  TITLE_BULLET +
  'Console\n\n' +
  getConsoleOutput(
@@ -208,4 +225,25 @@ export default class DefaultReporter extends BaseReporter {
  const snapshotStatuses = getSnapshotStatus(result.snapshot, didUpdate);
  snapshotStatuses.forEach(this.log);
  }
+
+ printCompactFailureMessage(
+ testPath: Path,
+ config: ProjectConfig,
+ result: TestResult,
+ ) {
+ result.testResults.forEach((assertionResult: AssertionResult) => {
+ if (assertionResult.failureMessages.length) {
+ const location = getLocation(assertionResult);
+ const path = formatTestPath(
+ config ? config : this._globalConfig,
+ result.testFilePath,
+ );
+ const fullTitle = formatFullTitle(assertionResult);
+ const fullPath = `${path}:${location.line}:${location.column}`;
+ this.log(
+ TITLE_INDENT + chalk.red(TITLE_BULLET + fullTitle) + ' ' + fullPath,
+ );
+ }
+ });
+ }
 }

packages/jest-cli/src/reporters/utils.js

@@ -8,7 +8,7 @@
  */
 
 import type {Path, ProjectConfig, GlobalConfig} from 'types/Config';
-import type {AggregatedResult} from 'types/TestResult';
+import type {AggregatedResult, AssertionResult} from 'types/TestResult';
 
 import path from 'path';
 import chalk from 'chalk';
@@ -273,3 +273,14 @@ export const wrapAnsiString = (string: string, terminalWidth: number) => {
  )
  .join('\n');
 };
+
+export const getLocation = (assertionResult: AssertionResult) => {
+ if (assertionResult.location) {
+ return assertionResult.location;
+ }
+ const matches = assertionResult.failureMessages[0].match(/(\d+):(\d+)/) || [];
+ return {
+ column: matches[2],
+ line: matches[1],
+ };
+};

packages/jest-message-util/src/index.ts

@@ -319,13 +319,8 @@ export const formatResultsErrors = (
  message = indentAllLines(message, MESSAGE_INDENT);
 
  const title =
- chalk.bold.red(
- TITLE_INDENT +
- TITLE_BULLET +
- result.ancestorTitles.join(ANCESTRY_SEPARATOR) +
- (result.ancestorTitles.length ? ANCESTRY_SEPARATOR : '') +
- result.title,
- ) + '\n';
+ chalk.bold.red(TITLE_INDENT + TITLE_BULLET + formatFullTitle(result)) +
+ '\n';
 
  return title + '\n' + message + '\n' + stack;
  })
@@ -355,3 +350,10 @@ export const separateMessageFromStack = (content: string) => {
  const stack = messageMatch[2];
  return {message, stack};
 };
+
+export const formatFullTitle = (result: AssertionResult) =>
+ result.ancestorTitles.join(ANCESTRY_SEPARATOR) +
+ (result.ancestorTitles.length ? ANCESTRY_SEPARATOR : '') +
+ result.title;
+
+export {TITLE_INDENT, TITLE_BULLET};