TST: Using more fixtures in of tests/base/test_ops.py

[SaturnFromTitan]

part of #23877

• tests added / passed
• passes black pandas
• passes git diff upstream/master -u -- "*.py" | flake8 --diff

[jbrockmendel]

the diff looks a little mangled; is this all basically un-indent?

[SaturnFromTitan]

@jbrockmendel Not entirely. I had to extend the tests or add some extra conditions because the fixtures are more versatile than the original test values

[WillAyd]

Can these just be split into separate tests for index / series? One-offs are fine but this seems to branch a lot within the test based on the type of object, which makes the purpose of the test less clear

[SaturnFromTitan]

@WillAyd I guess you're referring to test_memory_usage as the others don't really branch for logic just relevant to indices or series. I put the series-specific logic of this test into two new ones.

I kept narrow_series and series_with_simple_index separated as I don't want to create another combined fixture with a very narrow use case.

Please have another look

[jreback]

thanks @SaturnFromTitan