TST/CLN: Parameterize test in sparse/test_indexing

[gfyoung]

Title is self-explanatory.

[pep8speaks]

Hello @gfyoung! Thanks for submitting the PR.

• There are no PEP8 issues in the file pandas/tests/sparse/test_indexing.py !

[gfyoung]

I expect to have Azure fail until #23182 is merged.

[jschendel]

LGTM in general. A few small comments (also want to try out this new "suggestion" feature).

[codecov]

Codecov Report

Merging #23213 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #23213   +/-   ##
=======================================
  Coverage   92.19%   92.19%           
=======================================
  Files         169      169           
  Lines       50954    50954           
=======================================
  Hits        46975    46975           
  Misses       3979     3979

Flag	Coverage Δ
#multiple	90.61% <ø> (ø)	⬆️
#single	42.27% <ø> (ø)	⬆️

---

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 1546c35...5b68ad4. Read the comment docs.

[gfyoung]

A few small comments (also want to try out this new "suggestion" feature).

In the context of pandas, the fact that each applied suggestion is a new commit means we're going to trigger CI for each applied suggestion, which is not ideal unfortunately. Interesting feature though...

[jschendel]

My only gripe with the feature is that it is going to fire new commits every time.

Yeah, and doesn't look like there's a way to do multi-line edits. Probably good if there's a single one line change that needs to be made, but otherwise might not be worthwhile atm.

Each commit (at least for this repository) is going to trigger CI IIUC.

Is CI smart enough to cancel the job if a newer commit has been made? I've seen it do so in some similar scenarios before, but not sure what the exact logic around that is.

[jreback]

CI do cancel older builds if newer ones appear
though only if they didn’t start

[gfyoung]

CI do cancel older builds if newer ones appear
though only if they didn’t start

Ideally, though Circle was having issues with that...

[gfyoung]

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

[TomAugspurger]

Thanks!