You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
dreamorosi opened this issue
Aug 16, 2024
· 2 comments
· Fixed by #2938
Assignees
Labels
batchThis item relates to the Batch Processing UtilitycompletedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)testsPRs that add or change tests
In #2935 we have added Vitest to the workspace, as part of this effort we should move the Batch Processing unit tests to use it.
The activity should be considered complete once the tests can be run: in 1/ CI environment, 2/ locally as a workspace, 3/ locally as standalone package, and 4/ locally in the pre-push commit hook.
As part of this work, we should also disable default coverage reporting.
Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET
Future readers
Please react with 👍 and your use case to help us understand customer demand.
The text was updated successfully, but these errors were encountered:
dreamorosi
added
internal
PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)
confirmed
The scope is clear, ready for implementation
tests
PRs that add or change tests
batch
This item relates to the Batch Processing Utility
labels
Aug 16, 2024
batchThis item relates to the Batch Processing UtilitycompletedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)testsPRs that add or change tests
Summary
In #2935 we have added Vitest to the workspace, as part of this effort we should move the Batch Processing unit tests to use it.
The activity should be considered complete once the tests can be run: in 1/ CI environment, 2/ locally as a workspace, 3/ locally as standalone package, and 4/ locally in the pre-push commit hook.
As part of this work, we should also disable default coverage reporting.
Why is this needed?
So that we can continue the migration to Vitest.
Which area does this relate to?
Tests
Solution
No response
Acknowledgment
Future readers
Please react with 👍 and your use case to help us understand customer demand.
The text was updated successfully, but these errors were encountered: