fix: disable console history in tests #882
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What I did
Prevents IPython from spawning history collection threads when running the test suite
fixes: #881
How I did it
Add an auto-used fixture to patch an
APE_TESTING
environment variable in while testing. The IPython embedded instance launches with history disabled if this environment variable is set.How to verify it
If the tests weren't failing from "too many open files", you can change the soft limit on your machine with this.
This should produce a failure without these changes
Checklist