#241: Ensure print logs to console on never works always.

README.md

@@ -369,6 +369,8 @@ directory. You should add `it.only` to the test case you are working on to speed
 
 ## Release Notes
 
+- Fix `never` on `printLogsToConsole` works in all cases. [issue](https://github.com/archfz/cypress-terminal-report/issues/241)
+
 #### 6.0.1
 
 - Fix `txt` output processor not logging command timings. [merge-request](https://github.com/archfz/cypress-terminal-report/pull/236) by [AlexGuironnetRTE](https://github.com/AlexGuironnetRTE)

src/installLogsPrinter.js

@@ -101,9 +101,11 @@ function installLogsPrinter(on, options = {}) {
       }
 
       if (
-        (options.printLogsToConsole === "onFail" && data.state !== "passed")
-        || options.printLogsToConsole === "always"
-        || isHookAndShouldLog
+        options.printLogsToConsole !== "never" && (
+          options.printLogsToConsole === "always"
+          || (options.printLogsToConsole === "onFail" && data.state !== "passed")
+          || isHookAndShouldLog
+        )
       ) {
         logToTerminal(terminalMessages, options, data);
       }

test/specs/extendedController.spec.js

@@ -136,6 +136,12 @@ describe('Extended controller.', () => {
     });
   }).timeout(60000);
 
+  it('Should not display logs from before all hooks when printing to console is never.', async function () {
+    await runTest(commandBase(['printSuccessfulHookLogs=1', 'enableExtendedCollector=1', 'printLogsToConsoleNever=1'], ['beforeLogs.spec.js']), (error, stdout, stderr) => {
+      expect(clean(stdout, true)).to.not.contain(`cy:log ${ICONS.info}  some before command`);
+    });
+  }).timeout(60000);
+
   it('Should display logs from after all hooks if they fail.', async function () {
     this.retries(3);