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.

Sign up for GitHub

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

Jump to bottom

Test that dependencies are actually used, not just loaded #205

Open
IanButterworth opened this issue Oct 2, 2023 · 1 comment
Open

Test that dependencies are actually used, not just loaded #205

IanButterworth opened this issue Oct 2, 2023 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed new test

Comments

@IanButterworth
Copy link

I appreciate this is probably hard to do, but just to collect thoughts..

https://github.com/julia-actions/julia-treeshake exists which tests that dependencies are used during tests via code coverage.

It can be fooled by __init__'s.

Might there be a cleverer/more efficient way to do this?
@lgoettgens lgoettgens added enhancement New feature or request help wanted Extra attention is needed new test labels Oct 3, 2023
@lgoettgens
Copy link
Collaborator

That seems like a great test idea. However, I would only include it into Aqua (at least for the default tests) if it runs in a short time at doesn't need to run the whole testsuite of the package.

If somebody has an idea on how to approach this, please let me know or provide a (partial) PR yourself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed new test
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@IanButterworth @lgoettgens