search

aws-powertools / powertools-lambda-typescript

Powertools is a developer toolkit to implement Serverless best practices and increase developer velocity.
https://docs.powertools.aws.dev/lambda/typescript/latest/
MIT No Attribution
1.58k stars 139 forks source link

test(idempotency): migrate to vitest #3124

Closed dreamorosi closed 1 month ago

dreamorosi commented 1 month ago

Summary

Changes

Please provide a summary of what's being changed

This PR updates the unit tests for the Idempoency package so that they use vitest instead of Jest.

The e2e tests for the utility will be addressed in a future PR. I also removed a number of test cases that were redundant while leaving coverage unaffected. There's still room for further improvement but I decided to not address it now for the sake of keeping the PR focused.

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

Issue number: closes #3123

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.

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
4.5% Duplication on New Code

See analysis details on SonarCloud

dreamorosi commented 1 month ago

Sorry, I had to make another commit to remove a leftover test I had left commented out.

@am29d, @leandrodamascena, if you could click approve again please.