Fixed videoOnFailOnly option applying to skipped tests

LironEr / cypress-mochawesome-reporter

Zero config Mochawesome reporter for Cypress with screenshots and videos

MIT License

158 stars 49 forks source link

Fixed videoOnFailOnly option applying to skipped tests #150

Closed joakim-sch closed 1 year ago

joakim-sch commented 1 year ago

Fixed videoOnFailOnly option adding videos to tests skipped by mocha .skip flag.

Added tests to verify videos being added correctly.

Resolves #149

joakim-sch commented 1 year ago

@LironEr The tests are checking for a Pending test because it seems that when using the .skip mocha flag it adds the test to the report as "Pending"...

LironEr commented 1 year ago

Thanks @joakim-sch ! (: