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

Improve testing coverage #6

Open
Tracked by #2
egonw opened this issue Jan 1, 2024 · 3 comments
Open
Tracked by #2

Improve testing coverage #6

egonw opened this issue Jan 1, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request WP1

Comments

@egonw
Copy link
Member

egonw commented Jan 1, 2024

Description:

Second, we wish to improve the maintainability of the library and improve the code coverage of
the (unit) testing to at least 50% for all modules (statistics available at app.codecov.io/gh/cdk/cdk).
@egonw egonw added enhancement New feature or request WP1 labels Jan 1, 2024
@egonw egonw mentioned this issue Jan 1, 2024
3 tasks
@egonw
Copy link
Member Author

egonw commented Jun 11, 2024

I had a look at the org.openscience.cdk.controller package (in the extra module), but that one is hard to test, I think: https://app.codecov.io/gh/cdk/cdk/tree/master/misc%2Fcontrol%2Fsrc%2Fmain%2Fjava%2Forg%2Fopenscience%2Fcdk%2Fcontroller

@egonw
Copy link
Member Author

egonw commented Jun 11, 2024

This one worked better. No new code needed, just making more "maven-like": cdk/cdk#1081

Mostly increased counted :)

image

See https://app.codecov.io/gh/cdk/cdk/pull/1081/indirect-changes

@egonw
Copy link
Member Author

egonw commented Jul 14, 2024

new PR: cdk/cdk#1095

image

@egonw egonw self-assigned this Jul 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request WP1
Projects
None yet
Development

No branches or pull requests

1 participant