-
Notifications
You must be signed in to change notification settings - Fork 124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Cypress reports for skipped tests (fixes #948) #949
Add Cypress reports for skipped tests (fixes #948) #949
Conversation
e783148
to
d22c6ab
Compare
9ed848c
to
cf736c3
Compare
Hey, @RobertBattaglia! Thank you for the contribution. Could you please run |
done! |
d222a08
to
f1cf54d
Compare
@RobertBattaglia could you please rebase it? |
f1cf54d
to
307a1f5
Compare
done, still haven't been able to figure out why that unit test is failing, I am having trouble running the tests locally |
Hey, @RobertBattaglia! We're working hard now on the new version, so it was much simpler to mo to cherry-pick your commits and solve the conflicts. I closed the PR because I had opened a new one (#1029). Thank you for the contributions! |
Context
Attempt to report Cypress skipped tests. Since skipped tests don't execute the
afterEach
hook, create a start and end message for each skipped test and executeallureReportTest
task in theafter
hook.Checklist