Skip to content

Maintenance: Increase unit test coverage #1096

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

Closed
1 of 2 tasks
jeromevdl opened this issue Mar 18, 2023 · 4 comments
Closed
1 of 2 tasks

Maintenance: Increase unit test coverage #1096

jeromevdl opened this issue Mar 18, 2023 · 4 comments

Comments

@jeromevdl
Copy link
Contributor

jeromevdl commented Mar 18, 2023

Summary

Based on https://app.codecov.io/github/aws-powertools/powertools-lambda-java, increase the unit test coverage on modules which are under 80%: core, logging, validation, parameters & serialization

Why is this needed?

As powertools for java becomes more popular, we need to make sure we are well covered with unit tests. Objective is not to increase the coverage for the numbers but to make sure there is nothing left untested.

Which area does this relate to?

Tests

Solution

No response

Acknowledgment

@eldimi
Copy link
Contributor

eldimi commented Jun 21, 2023

I will start working on this. It is not possible to set myself as an assignee for the time being.
Also, including the updated code coverage location

@jeromevdl
Copy link
Contributor Author

@eldimi, any help needed, any progress so far?

@eldimi
Copy link
Contributor

eldimi commented Jul 4, 2023

@eldimi, any help needed, any progress so far?

I just added this PR for review

@jeromevdl jeromevdl moved this to Working on it in Powertools for AWS Lambda (Java) Jul 9, 2023
@jeromevdl
Copy link
Contributor Author

jeromevdl commented Jul 10, 2023

closed with #1264, great improvement of the coverage: https://app.codecov.io/gh/aws-powertools/powertools-lambda-java/tree/main

@jeromevdl jeromevdl moved this from Working on it to Shipped in Powertools for AWS Lambda (Java) Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Shipped
Development

No branches or pull requests

2 participants