-
Notifications
You must be signed in to change notification settings - Fork 0
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
JANUS unit tests #31
Comments
The current tests take about 7 minutes on my system, which is too long too be useful. I think not only splitting them up, but also making a smaller test data set that runs in a shorter amount of time (couple of seconds) would be nice. |
One cycle of instellation alone takes 45 seconds, and the code does 7 cycles 😅 |
It's not entirely clear how the test can be broken into smaller pieces, it's mostly lots of setup, a few lines of code, and then clean-up. What we can do is parametrize the tests. I'm going to have a go at this to get a bit of a feel for the code. |
It would be good if any tests of JANUS use different platforms or distributions. We are more likely to catch issues such as: FormingWorlds/MORS#22 |
The tests we have for now are global examples and we might think of breaking those into smaller pieces so that it makes bugs easier to track.
The text was updated successfully, but these errors were encountered: