Updates to the per-slice CSV

[artoonie]

Closes #871. Makes several changes as outlined there.

[yezr]

Testing

• confirmed candidate order in tabulate by precinct and tabulate by batch
• confirmed elect/eliminate in tabulate by precinct and tabulate by batch
• confirmed removal of both thresholds in tabulate by precinct and tabulate by batch
• confirmed additional text at the bottom of summary.csv for precinct files and batch files

I don't see any tabulate by batch files currently in tests and I think we should include at least one. I tried to do this by using the contest that had already changed (Minneapolis Mayor 2013), turning on "Tabulate by Batch", and set the Batch Column setting in the CVR config to 1 in order to just re-use the precinct ids as batch ids. That way they should be equivalent except for the minor tabulate by batch/precinct differences. I was then gonna check in one batch summary file, the updated config and change the equivalent test to expect more files.

However, when running the tabulation with Precinct Column and Batch Column set to 1 I got warnings that every row didn't have a batch id. I looked it the code and that's because the StreamingCvrReader only allows a cell to be one of precinct id, batch id, ballot id or an actual ranking - never more than one of those things.

I changed Batch Column to 4 to use the extra count column that ES&S CVRs spit out. Effectively, every batch id = 1. That resulted in this file that exactly matches the full contest summary.csv except for the tabulate by batch specific edits, as expected.
2024-09-06_15-15-03_1_batch_summary.csv

I think we need to check one batch slice file in. What I did here seems kind of clunky. We could use the smaller contest I made to test tabulate by batch earlier
CSV CVRs - tab by batch test.csv

Other than adding in at least one batch summary the changes LGTM! Ready for a live code review!

[yezr]

code review comments addressed

• two tabulate by batch files have been added by @artoonie
• addActionRows logic now explicit inline, removed boolean in function call
• explicit check that the candidates we have within the slice match the candidate list sent to us from the Tabulator

Tested that the batch tabulation contest runs on my machine. LGTM! approved

[yezr]

LGTM

[yezr]

approving #883 and #884 merge