TST/REF: Add more pytest idiom to tests/tslib

[gfyoung]

I was planning to correct individual files, but then I began to realize that the fixes were involving more and more files (was shifting tests around) in the directory, to the point that I just modified them all. 🙂

[codecov]

Codecov Report

Merging #24587 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #24587      +/-   ##
==========================================
- Coverage   92.38%   92.38%   -0.01%     
==========================================
  Files         166      166              
  Lines       52478    52478              
==========================================
- Hits        48483    48482       -1     
- Misses       3995     3996       +1

Flag	Coverage Δ
#multiple	90.81% <ø> (ø)	⬆️
#single	43.05% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
pandas/util/testing.py	88% <0%> (-0.1%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 62506ca...646af2f. Read the comment docs.

[codecov]

Codecov Report

Merging #24587 into master will decrease coverage by 49.32%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #24587       +/-   ##
===========================================
- Coverage   92.36%   43.03%   -49.33%     
===========================================
  Files         166      166               
  Lines       52497    52497               
===========================================
- Hits        48489    22593    -25896     
- Misses       4008    29904    +25896

Flag	Coverage Δ
#multiple	?
#single	43.03% <ø> (-0.01%)	⬇️

Impacted Files	Coverage Δ
pandas/io/formats/latex.py	0% <0%> (-100%)	⬇️
pandas/core/categorical.py	0% <0%> (-100%)	⬇️
pandas/io/sas/sas_constants.py	0% <0%> (-100%)	⬇️
pandas/tseries/plotting.py	0% <0%> (-100%)	⬇️
pandas/tseries/converter.py	0% <0%> (-100%)	⬇️
pandas/io/formats/html.py	0% <0%> (-99.37%)	⬇️
pandas/core/groupby/categorical.py	0% <0%> (-95.46%)	⬇️
pandas/io/sas/sas7bdat.py	0% <0%> (-91.17%)	⬇️
pandas/io/sas/sas_xport.py	0% <0%> (-90.15%)	⬇️
pandas/core/tools/numeric.py	10.44% <0%> (-89.56%)	⬇️
... and 124 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0039158...16c4d84. Read the comment docs.

[jreback]

looks good. some comments.

[jbrockmendel]

Is this just getting rid of a level of indentation by removing the classes? I get that they don't serve much functional purpose, but do find them helpful for organization. Is there a preferred idiom for organization?

[gfyoung]

@jbrockmendel : In general, small, modular, functional test cases would be preferred I think (both for consistency with pytest idiom and readability, especially the short and modular part). I agree that test classes can be useful, especially when we are sharing test cases amongst multiple files or the test file is massive and has several super distinct collections of tests.

That being said, for this situation, I did not really find that to be the case for tests/tslibs. Note that this PR attempts to reorganize the tests a little as well as add a lot more parameterization (in interests of the short and modular).

I don’t think we will have a unified style for tests. We might prefer functional, but I wouldn’t force it everywhere if the (semantic) organization does not dictate it to be so.

[gfyoung]

@jreback : Addressed all comments, and everything is green. PTAL.

[jreback]

thanks @gfyoung