#113 #114 #116: Fix for skip tests and collect logs validation.

README.md

@@ -308,6 +308,10 @@ directory. You should add `it.only` to the test case you are working on to speed
 
 ## Release Notes
 
+- Fix issue `cy:intercept` not between the allowed configuration options. [issue](https://github.com/archfz/cypress-terminal-report/issues/113)
+- Fix issue with plugin breaking cypress with skipped tests. [issue1](https://github.com/archfz/cypress-terminal-report/issues/116) [issue2](https://github.com/archfz/cypress-terminal-report/issues/114)
+  Update cypress to 8.3.0 in tests to confirm support.
+
 #### 3.3.0
 
 - Added support for logging command from skipped tests. [issue](https://github.com/archfz/cypress-terminal-report/issues/111)

src/collector/LogCollectExtendedControl.js

@@ -276,7 +276,11 @@ module.exports = class LogCollectExtendedControl extends LogCollectBaseControl {
     // Logs commands if test was manually skipped.
     Cypress.mocha.getRunner().on('pending', function (test) {
       if (self.collectorState.getCurrentTest()) {
-        sendLogsToPrinterForATest(self.collectorState.getCurrentTest());
+        // In case of fully skipped tests we might not yet have a log stack.
+        if (!self.collectorState.hasLogsCurrentStack()) {
+          self.collectorState.addNewLogStack();
+        }
+        sendLogsToPrinterForATest(test);
       }
     });
   }

src/collector/LogCollectSimpleControl.js

@@ -40,23 +40,42 @@ module.exports = class LogCollectSimpleControl extends LogCollectBaseControl {
       return this.prepareLogs(logStackIndex, {mochaRunnable, testState, testTitle, testLevel});
     };
 
-    // Need to wait for command log update debounce.
-    cy.wait(wait, {log: false})
-      .then(() => {
-        cy.task(
-          CONSTANTS.TASK_NAME,
-          {
+    if (options.noQueue) {
+      Promise.resolve().then(() => {
+        Cypress.backend('task', {
+          task: CONSTANTS.TASK_NAME,
+          arg: {
             spec: spec,
             test: testTitle,
             messages: prepareLogs(),
             state: testState,
             level: testLevel,
             consoleTitle: options.consoleTitle,
             isHook: options.isHook,
-          },
-          {log: false}
-        );
-      });
+          }
+        })
+          // For some reason cypress throws empty error although the task indeed works.
+          .catch((error) => {/* noop */})
+      }).catch(console.error);
+    } else {
+      // Need to wait for command log update debounce.
+      cy.wait(wait, {log: false})
+        .then(() => {
+          cy.task(
+            CONSTANTS.TASK_NAME,
+            {
+              spec: spec,
+              test: testTitle,
+              messages: prepareLogs(),
+              state: testState,
+              level: testLevel,
+              consoleTitle: options.consoleTitle,
+              isHook: options.isHook,
+            },
+            {log: false}
+          );
+        });
+    }
   }
 
   registerState() {
@@ -86,7 +105,11 @@ module.exports = class LogCollectSimpleControl extends LogCollectBaseControl {
     // Logs commands if test was manually skipped.
     Cypress.mocha.getRunner().on('pending', function (test) {
       if (self.collectorState.getCurrentTest()) {
-        self.sendLogsToPrinter(self.collectorState.getCurrentLogStackIndex(), self.collectorState.getCurrentTest());
+        // In case of fully skipped tests we might not yet have a log stack.
+        if (!self.collectorState.hasLogsCurrentStack()) {
+          self.collectorState.addNewLogStack();
+        }
+        self.sendLogsToPrinter(self.collectorState.getCurrentLogStackIndex(), self.collectorState.getCurrentTest(), {noQueue: true});
       }
     });
   }

src/installLogsCollector.schema.json

@@ -20,6 +20,7 @@
           "cy:log",
           "cy:xhr",
           "cy:request",
+          "cy:intercept",
           "cy:route",
           "cy:command"
         ]

test/cypress/integration/skipTest.spec.js

@@ -0,0 +1,37 @@
+describe('Describe 1', () => {
+  before(() => {
+    cy.visit(`https://www.google.co.uk`);
+  });
+
+  it.skip('Skipped test 1', () => {
+    cy.log('first skip');
+  });
+
+  it('Test 2', () => {
+    cy.log('test 2');
+  });
+});
+
+describe('Describe 2', () => {
+  beforeEach(() => {
+    cy.visit(`https://www.google.co.uk`);
+  });
+
+  // The tests gets hang here
+  it.skip('Skipped test 2', () => {
+    cy.log('Skipped test 2');
+  });
+
+  //This second skipped test seems to be the culprit
+  it.skip('Skipped test 3', () => {
+    cy.log('Skipped test 3');
+  });
+
+  it('Test 3', () => {
+    cy.log('Test 3');
+  });
+
+  it('Test 4', () => {
+    cy.log('Test 4');
+  });
+});

test/cypress/support/index.js

@@ -72,6 +72,11 @@ if (env.supportBadConfig == '1') {
     shouldNotBeHere: ""
   };
 }
+if (env.supportGoodConfig == '1') {
+  config = {
+    collectTypes: ['cons:log','cons:info', 'cons:warn', 'cons:error', 'cy:log', 'cy:xhr', 'cy:request', 'cy:route', 'cy:intercept', 'cy:command']
+  };
+}
 if (env.enableExtendedCollector == '1') {
   config.enableExtendedCollector = true;
 }

test/package.json

@@ -24,7 +24,7 @@
     "@babel/preset-env": "^7.4.5",
     "@babel/register": "^7.13.16",
     "chai": "^4.2.0",
-    "cypress": "8.1.0",
+    "cypress": "8.3.0",
     "cypress-cucumber-preprocessor": "^4.0.1",
     "fs-extra": "^9.0.1",
     "glob": "^7.1.6",

test/specs/extendedController.spec.js

@@ -191,7 +191,7 @@ describe('Extended controller.', () => {
     });
   }).timeout(60000);
 
-  it('Should work correctly with skip.', async function () {
+  it('Should work correctly with dynamic skip.', async function () {
     await runTest(commandBase(['enableExtendedCollector=1'], ['dynamicSkip.spec.js']), (error, stdout, stderr) => {
       expect(clean(stdout)).to.contain(`- test3
           cy:log ${ICONS.info}  before
@@ -200,4 +200,18 @@ describe('Extended controller.', () => {
           cy:log ${ICONS.info}  test3 3`);
     });
   }).timeout(60000);
+
+  it('Should work correctly with skipped tests.', async function () {
+    await runTest(commandBase(['enableExtendedCollector=1'], ['skipTest.spec.js']), (error, stdout, stderr) => {
+      expect(clean(stdout, true)).to.contain(`  Describe 1
+    - Skipped test 1
+    ✓ Test 2
+
+  Describe 2
+    - Skipped test 2
+    - Skipped test 3
+    ✓ Test 3
+    ✓ Test 4`);
+    });
+  }).timeout(60000);
 });

test/specs/misc.spec.js

@@ -1,7 +1,7 @@
 import {
   ICONS,
   runTest,
-  commandBase, logLastRun,
+  commandBase, logLastRun, clean,
 } from "../utils";
 
 const {expect} = require('chai');
@@ -75,4 +75,18 @@ describe('Misc.', () => {
     });
   }).timeout(30000);
 
+  it('Should work correctly with skipped tests.', async function () {
+    await runTest(commandBase([''], ['skipTest.spec.js']), (error, stdout, stderr) => {
+      expect(clean(stdout, true)).to.contain(`  Describe 1
+    - Skipped test 1
+    ✓ Test 2
+
+  Describe 2
+    - Skipped test 2
+    - Skipped test 3
+    ✓ Test 3
+    ✓ Test 4`);
+    });
+  }).timeout(60000);
+
 });

test/specs/validation.spec.js

@@ -27,6 +27,12 @@ describe('Validation.', () => {
     });
   }).timeout(60000);
 
+  it('Should not fail with proper config.', async () => {
+    await runTest(commandBase(['supportGoodConfig=1'], ['happyFlow.spec.js']), (error, stdout, stderr) => {
+      expect(stdout).to.not.contain(`cypress-terminal-report: Invalid plugin install options:`);
+    });
+  }).timeout(60000);
+
   it('Should print proper validation error on invalid support install options.', async () => {
     await runTest(commandBase(['supportBadConfig=1'], ['happyFlow.spec.js']), (error, stdout, stderr) => {
       expect(stdout).to.contain(`cypress-terminal-report: Invalid plugin install options:`);