Skip to content
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

WDIO - Spec retry + minor tweaks + tests #21

Closed
wants to merge 12 commits into from

Conversation

unickq
Copy link
Collaborator

@unickq unickq commented Jan 23, 2025

This pull request includes several changes to improve the testing setup and enhance the reporting functionality. The most important changes include updating the Node.js version, switching from Jest to Vitest for testing, and enhancing the generate-report module to include debug options and handle previous reports.

Improvements to testing setup:

  • .github/workflows/main.yaml: Updated to use Node.js 22.x and added steps to run tests and publish test reports.
  • package.json: Switched from Jest to Vitest for testing and added a new script for continuous integration testing. [1] [2]

Enhancements to reporting functionality:

  • src/generate-report.ts: Added a debug option to the ReporterConfigOptions interface, included logic to read and handle previous reports, and improved the handling of test retries and flaky tests. [1] [2] [3] [4] [5] [6] [7]

Other changes:

Or

@unickq unickq changed the title WDIO - Spec retry + minor tweaks WDIO - Spec retry + minor tweaks + tests Jan 27, 2025
@unickq
Copy link
Collaborator Author

unickq commented Jan 27, 2025

@Ma11hewThomas , I added some tests, please have a look

@Ma11hewThomas
Copy link
Collaborator

Thanks @unickq!

I'm really happy with the changes overall, can you take a look at the pending review for flaky tests logic and fix the failing tests in the pipeline - I'll then be happy to merge

Thanks for contributing!

@unickq
Copy link
Collaborator Author

unickq commented Jan 31, 2025

@Ma11hewThomas , can you enable runs of Actions for forked PRs?

@unickq unickq force-pushed the nick/upd-reporter branch from 81d57c6 to 5a2e361 Compare January 31, 2025 17:20
This reverts commit 5a2e361.
@unickq unickq force-pushed the nick/upd-reporter branch from 9b5efcd to 78ec03d Compare January 31, 2025 17:35
@unickq unickq force-pushed the nick/upd-reporter branch from 78ec03d to 5285053 Compare January 31, 2025 17:37
@unickq
Copy link
Collaborator Author

unickq commented Jan 31, 2025

I'll reopen it later if I make it work :(

@unickq unickq closed this Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants