Use @lru_cache-clearing Pytest fixture at test module level, not in function signatures
#462
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
akaihola
changed the title
lru_cache clear fixture at test module level@lru_cache-clearing Pytest fixture at test module level, not in function signatures
akaihola
force-pushed
the
module-global-fixture
branch
from
f8f36ee to
fbaea9a
Compare
akaihola
force-pushed
the
module-global-fixture
branch
from
fbaea9a to
09089f4
Compare
akaihola
force-pushed
the
module-global-fixture
branch
from
09089f4 to
9d3e49e
Compare
akaihola
force-pushed
the
module-global-fixture
branch
from
9d3e49e to
e78979b
Compare
akaihola
force-pushed
the
module-global-fixture
branch
from
e78979b to
0c20863
Compare
|
Obsolete, fixed earlier in another PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Test function signatures are cleaner if the
find_project_root_cache_clearfixture is omitted and instead added at the module level. It's not that expensive, so doesn't matter if it's used also for test functions which don't require it.