Skip to content

Conversation

@dreamorosi
Copy link
Contributor

Summary

Changes

Please provide a summary of what's being changed

This PR upgrades vitest and its coverage module to the latest major version and revisits all the code paths that were previously ignored by either updating them to the new ignore syntax or removing the ignore comment entirely.

In the process, I also had to update a couple tests that were not being awaited correctly.

Thanks to #4679 and @sdangol for the prework on this, I opted for a new PR and branch since the other had conflicts and this was faster.

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed

Issue number: closes #4746


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@dreamorosi dreamorosi requested a review from svozza November 14, 2025 16:19
@dreamorosi dreamorosi self-assigned this Nov 14, 2025
@pull-request-size pull-request-size bot added the size/L PRs between 100-499 LOC label Nov 14, 2025
@boring-cyborg boring-cyborg bot added batch This item relates to the Batch Processing Utility commons This item relates to the Commons Utility dependencies Changes that touch dependencies, e.g. Dependabot, etc. event-handler This item relates to the Event Handler Utility idempotency This item relates to the Idempotency Utility logger This item relates to the Logger Utility metrics This item relates to the Metrics Utility parameters This item relates to the Parameters Utility tests PRs that add or change tests labels Nov 14, 2025
@dreamorosi dreamorosi requested a review from sdangol November 14, 2025 16:28
@dreamorosi dreamorosi requested a review from svozza November 14, 2025 16:44
@sonarqubecloud
Copy link

@dreamorosi dreamorosi merged commit d7303e8 into main Nov 14, 2025
37 checks passed
@dreamorosi dreamorosi deleted the chore/update_vitest branch November 14, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

batch This item relates to the Batch Processing Utility commons This item relates to the Commons Utility dependencies Changes that touch dependencies, e.g. Dependabot, etc. event-handler This item relates to the Event Handler Utility idempotency This item relates to the Idempotency Utility logger This item relates to the Logger Utility metrics This item relates to the Metrics Utility parameters This item relates to the Parameters Utility size/L PRs between 100-499 LOC tests PRs that add or change tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maintenance: upgrade vitest to next major

2 participants