MAINT: Add more pytest idiom to parser tests by gfyoung · Pull Request #23712 · pandas-dev/pandas (original) (raw)
@jreback @jbrockmendel : Here is the plan:
c_parser_only.py
is now separate fromtest_parsers.py
, so you can just runpytest test_c_parser_only.py
(the fixtures get automatically injected per the newly createdconftest.py
). The new format of those tests is 100% functional (no classes), which is morepytest
-idiomatic.- If this format looks good, I will convert all other files (e.g.
header.py
,comment.py
,mangle_dupe_cols.py
similarly in subsequent PR's (we will be in this weird intermediate state withtest_parsers.py
andconftest.py
co-existing).
Once that is done, we no longer need test_parsers.py
. All tests will be nice and modular and not all clumped into one giant test class as they are currently.
Hopefully that clears things up?