-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add --used-fixtures
command
#23
base: main
Are you sure you want to change the base?
Add --used-fixtures
command
#23
Conversation
@jllorencetti can we accept this PR? This really good feature |
@malthejorgensen could you please rebase your branch with main? |
5505c49
to
97fdb2b
Compare
@jairhenrique I've rebased this branch as well as updated the base branch from The code is quite old so I'm not 100% whether it works with the changes made in the base branch. |
Codecov Report
@@ Coverage Diff @@
## main #23 +/- ##
==========================================
+ Coverage 66.66% 68.15% +1.48%
==========================================
Files 1 1
Lines 126 157 +31
Branches 28 35 +7
==========================================
+ Hits 84 107 +23
- Misses 35 41 +6
- Partials 7 9 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
97fdb2b
to
16f472f
Compare
Tests / Python 3.11 (pull_request) was failing due to linting/ |
@malthejorgensen this changes looks good for me! |
This new
--used-fixtures
command lists fixtures used in the tests that are run.There might already be a builtin pytest command to do this, but I couldn't find it, so this might be useful for the next person passing by.
About the PR
I have expanded some of the existing tests, but somewhat naïvely (there's probably a nicer way to test the new command?). Also, I didn't know what to do for the
doctest
-tests.