getConsoleOutput receives global noStackTrace

CHANGELOG.md

@@ -8,6 +8,7 @@
 
 ### Fixes
 
+- `[jest-console]` `getConsoleOutput` to receive global stack trace config and use it to format stack trace ([#10081](https://github.com/facebook/jest/pull/10081))
 - `[jest-jasmine2]` Stop adding `:` after an error that has no message ([#9990](https://github.com/facebook/jest/pull/9990))
 - `[jest-diff]` Control no diff message color with `commonColor` in diff options ([#9997](https://github.com/facebook/jest/pull/9997))
 - `[jest-snapshot]` Fix TypeScript compilation ([#10008](https://github.com/facebook/jest/pull/10008))

packages/jest-console/src/__tests__/getConsoleOutput.test.ts

@@ -0,0 +1,57 @@
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+import {formatStackTrace} from 'jest-message-util';
+import getConsoleOutput from '../getConsoleOutput';
+import BufferedConsole from '../BufferedConsole';
+import {LogType} from '../types';
+
+jest.mock('jest-message-util', () => ({
+  formatStackTrace: jest.fn(),
+}));
+
+describe('getConsoleOutput', () => {
+  formatStackTrace.mockImplementation(() => 'throw new Error("Whoops!");');
+  const cases = [
+    'assert',
+    'count',
+    'debug',
+    'dir',
+    'dirxml',
+    'error',
+    'group',
+    'groupCollapsed',
+    'info',
+    'log',
+    'time',
+    'warn',
+  ];
+
+  cases.forEach(logType => {
+    it(`takes noStackTrace and pass it on for ${logType}`, () => {
+      getConsoleOutput(
+        'someRootPath',
+        true,
+        BufferedConsole.write([], logType as LogType, 'message', 4),
+        {
+          rootDir: 'root',
+          testMatch: [],
+        },
+        true,
+      );
+      expect(formatStackTrace).toHaveBeenCalled();
+      expect(formatStackTrace).toHaveBeenCalledWith(
+        expect.anything(),
+        expect.anything(),
+        expect.objectContaining({
+          noCodeFrame: expect.anything(),
+          noStackTrace: true,
+        }),
+      );
+    });
+  });
+});

packages/jest-console/src/getConsoleOutput.ts

@@ -11,18 +11,21 @@ import {
   StackTraceOptions,
   formatStackTrace,
 } from 'jest-message-util';
+import type {Config} from '@jest/types';
 import type {ConsoleBuffer} from './types';
 
 export default (
-  // TODO: remove in 26
+  // TODO: remove in 27
   root: string,
+  // TODO: this is covered by GlobalConfig, switch over in 27
   verbose: boolean,
   buffer: ConsoleBuffer,
-  // TODO: make mandatory and take Config.ProjectConfig in 26
+  // TODO: make mandatory and take Config.ProjectConfig in 27
   config: StackTraceConfig = {
     rootDir: root,
     testMatch: [],
   },
+  globalConfig: Config.GlobalConfig,
 ): string => {
   const TITLE_INDENT = verbose ? '  ' : '    ';
   const CONSOLE_INDENT = TITLE_INDENT + '  ';
@@ -40,12 +43,12 @@ export default (
     if (type === 'warn') {
       message = chalk.yellow(message);
       typeMessage = chalk.yellow(typeMessage);
-      noStackTrace = false;
+      noStackTrace = false || globalConfig.noStackTrace;
       noCodeFrame = false;
     } else if (type === 'error') {
       message = chalk.red(message);
       typeMessage = chalk.red(typeMessage);
-      noStackTrace = false;
+      noStackTrace = false || globalConfig.noStackTrace;
       noCodeFrame = false;
     }
 

packages/jest-message-util/src/__tests__/__snapshots__/messages.test.ts.snap

@@ -19,34 +19,19 @@ exports[`codeframe 1`] = `
 "
 `;
 
-exports[`formatStackTrace should strip node internals 1`] = `
-"<bold><red>  <bold>● </><bold>Unix test</></>
-
-
-        Expected value to be of type:
-          \\"number\\"
-        Received:
-          \\"\\"
-        type:
-          \\"string\\"
-<dim></>
-<dim>      <dim>at Object.it (</><dim>__tests__/test.js<dim>:8:14)</><dim></>
-"
-`;
-
-exports[`getConsoleOutput does not print code frame when noCodeFrame = true 1`] = `
+exports[`formatStackTrace does not print code frame when noCodeFrame = true 1`] = `
 "
       <dim>at Object.<anonymous> (</>file.js<dim>:1:7)</>
         "
 `;
 
-exports[`getConsoleOutput does not print codeframe when noStackTrace = true 1`] = `
+exports[`formatStackTrace does not print codeframe when noStackTrace = true 1`] = `
 "
       <dim>at Object.<anonymous> (</>file.js<dim>:1:7)</>
         "
 `;
 
-exports[`getConsoleOutput prints code frame and stacktrace 1`] = `
+exports[`formatStackTrace prints code frame and stacktrace 1`] = `
 "
     </><red><bold>></></><gray> 1 | </><cyan>throw</> <cyan>new</> <yellow>Error</>(<green>\\"Whoops!\\"</>)<yellow>;</></>
     </> <gray>   | </>      <red><bold>^</></></>
@@ -55,6 +40,21 @@ exports[`getConsoleOutput prints code frame and stacktrace 1`] = `
         "
 `;
 
+exports[`formatStackTrace should strip node internals 1`] = `
+"<bold><red>  <bold>● </><bold>Unix test</></>
+
+
+        Expected value to be of type:
+          \\"number\\"
+        Received:
+          \\"\\"
+        type:
+          \\"string\\"
+<dim></>
+<dim>      <dim>at Object.it (</><dim>__tests__/test.js<dim>:8:14)</><dim></>
+"
+`;
+
 exports[`no codeframe 1`] = `
 "  <bold>● </>Test suite failed to run
 

packages/jest-message-util/src/__tests__/messages.test.ts

@@ -290,7 +290,7 @@ it('no stack', () => {
   expect(message).toMatchSnapshot();
 });
 
-describe('getConsoleOutput', () => {
+describe('formatStackTrace', () => {
   it('prints code frame and stacktrace', () => {
     readFileSync.mockImplementationOnce(() => 'throw new Error("Whoops!");');
     const message = formatStackTrace(

packages/jest-reporters/src/default_reporter.ts

@@ -183,6 +183,7 @@ export default class DefaultReporter extends BaseReporter {
             !!this._globalConfig.verbose,
             result.console,
             config,
+            this._globalConfig,
           ),
       );
     }

packages/jest-runner/src/runTest.ts

@@ -121,6 +121,7 @@ async function runTestInternal(
       // 4 = the console call is buried 4 stack frames deep
       BufferedConsole.write([], type, message, 4),
       config,
+      globalConfig,
     );
 
   let testConsole;