Github action summary added

[jayedul]

Description of the Change

Mochawesome reporter added for Cypress test report

Closes #124

Changelog Entry

Added - Mochawesome reporter added for Cypress test report

Credits

Props @jayedul

Checklist:

• I agree to follow this project's Code of Conduct.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests pass.

[peterwilsoncc]

@jayedul I noticed you marked this ready and then reverted to draft. I can see the data being included (my last commit testing it deliberately breaks a few things) but I'm not sure if I should be seeing the screen shot you've provided in the description on the summary tab.

[jayedul]

Thank you @peterwilsoncc for this commit c4193f1

I was wondering why all the E2E tests were not running after adding GH summary commands. That's why I reverted to draft to check later. As I can see this commit resolved the issue, this PR is ready to review now. And I'll have to use this fix in some other PRs that has this same issue.

I'm not sure if I should be seeing the screen shot you've provided in the description on the summary tab.

You should if you run the following commands in your local system sequentially after running E2E tests.
npx mochawesome-merge ./tests/cypress/reports/*.json -o tests/cypress/reports/mochawesome.json
rm -rf ./tests/cypress/reports/mochawesome-*.json
npx mochawesome-json-to-md -p ./tests/cypress/reports/mochawesome.json -o ./tests/cypress/reports/mochawesome.md
npx mochawesome-report-generator tests/cypress/reports/mochawesome.json -o tests/cypress/reports/

It will generate mochawesome.html file in the directory ./tests/cypress/reports that you can open in any browser.

However for Github actions, I guess nothing else to do apart from checking if the scripts passed successfully. Or maybe @faisal-alvi can suggest something as he has some similar PR merged like this one 10up/simple-local-avatars#174

[peterwilsoncc]

This LGTM, thanks!

I pushed a minor change as noted inline.

I've tested an action run with broken tests, but have since force pushed the relevant commit out of the history to avoid noise.

[peterwilsoncc]

I pushed this in 550308b to ensure the summary is displayed for both test passes and failures.