Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request help wanted Extra attention is needed new test
Projects
None yet
Development

No branches or pull requests

2 participants