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 the package testing process #148

Open
ismael-lajaaiti opened this issue Mar 22, 2024 · 1 comment
Open

Improve the package testing process #148

ismael-lajaaiti opened this issue Mar 22, 2024 · 1 comment
Assignees

Comments

@ismael-lajaaiti
Copy link
Collaborator

ismael-lajaaiti commented Mar 22, 2024

Originally posted by @iago-lito in #147 (comment)

Yikes! Consider taking time to set up a proper sandbox environment in this case, or you'll spend ages running ] test over and over. :(
I start my sandbox script with things like:

using Test
using Random
using Revise
using EcologicalNetworksDynamics
include("PATH/TO/REPO/test/test_failures.jl")
include("PATH/TO/REPO/test/dedicate_test_failures.jl")
const EN = EcologicalNetworksDynamics # Required by some tests.

So I can then run tests line by line in a corresponding REPL.

Unfortunately, this is not exactly stable or exposed yet. But maybe it should eventually make its way to the CONTRIBUTING file :\

Also, feel free to (temporarily) comment out tests from ./test/runtests.jl or specify unique test files to run here.

@iago-lito iago-lito self-assigned this Mar 22, 2024
@iago-lito
Copy link
Collaborator

Good, thank you :) Unfortunately, the tests structure is very unstable yet and it cannot be exposed. There is no plan to stabilize it in the near future, and the hack described above will likely not work for all tests. What we can maybe do for now is to use this thread to provide help/support whenever someone has trouble writing tests for the package?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants