add configuration flag to immediately print from original console method (#49)

Co-authored-by: Simon Müller <[email protected]>

Author: SimonMueller

README.md

@@ -194,6 +194,14 @@ failOnConsole({
 })
 ```
 
+### shouldPrintMessage
+
+Use this to print the message immediately when called not awaiting the test to finish. This is useful to show the message if there are
+other or earlier test failures which will result in the fail on console error to be hidden by jest.
+
+- Type: `boolean`
+- Default: `false`
+
 ## License
 
 [MIT](https://github.com/ValentinH/jest-fail-on-console/blob/master/LICENSE)

index.d.ts

@@ -52,6 +52,9 @@ declare namespace init {
       methodName: ConsoleMethodName,
       context: { group: string; groups: string[] }
     ) => boolean
+
+    /** @default false */
+    shouldPrintMessage?: boolean
   }
 }
 

index.js

@@ -24,6 +24,7 @@ const init = ({
   skipTest,
   silenceMessage,
   allowMessage,
+  shouldPrintMessage = false,
 } = {}) => {
   const flushUnexpectedConsoleCalls = (methodName, unexpectedConsoleCallStacks) => {
     if (unexpectedConsoleCallStacks.length > 0) {
@@ -72,6 +73,10 @@ const init = ({
         return
       }
 
+      if (shouldPrintMessage) {
+        originalMethod(format, ...args)
+      }
+
       // Capture the call stack now so we can warn about it later.
       // The call stack has helpful information for the test author.
       // Don't throw yet though b'c it might be accidentally caught and suppressed.

tests/fixtures/print-message-with-other-failure/index.js

@@ -0,0 +1,4 @@
+module.exports = () => {
+  console.error('my error message that I do not control');
+  throw new Error('some other error');
+}

tests/fixtures/print-message-with-other-failure/index.test.js

@@ -0,0 +1,7 @@
+const consoleError = require('.')
+
+describe('print message flag with other test failure', () => {
+  it('does not throw', () => {
+    expect(consoleError).not.toThrow()
+  })
+})

tests/fixtures/print-message-with-other-failure/jest.config.js

@@ -0,0 +1,3 @@
+module.exports = {
+  setupFilesAfterEnv: ['./jest.setup.js'],
+}

tests/fixtures/print-message-with-other-failure/jest.setup.js

@@ -0,0 +1,6 @@
+const failOnConsole = require('../../..')
+
+failOnConsole({
+  shouldFailOnError: true,
+  shouldPrintMessage: true,
+})

tests/fixtures/print-message/index.js

@@ -0,0 +1,3 @@
+module.exports = () => {
+  console.error('my error message that I do not control')
+}

tests/fixtures/print-message/index.test.js

@@ -0,0 +1,7 @@
+const consoleError = require('.')
+
+describe('print message flag', () => {
+  it('does not throw', () => {
+    expect(consoleError).not.toThrow()
+  })
+})

tests/fixtures/print-message/jest.config.js

@@ -0,0 +1,3 @@
+module.exports = {
+  setupFilesAfterEnv: ['./jest.setup.js'],
+}