-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add precincts reporting percent override #52
Conversation
why does the test use a fixture and not an actual page? also, can we add a unit test for this? |
@lennybronner No particular reason, it's just better to use a fixture than have your test hit a live API. Yes, we can add a unit test using that same fixture. |
Unit test added. |
Why does expected votes go away when we override? |
@lennybronner Expected votes only appears when the precinct is fully reporting. It is set to the turnout. |
Description
This PR adds an override to allow specifying a reporting status for precincts in a specific county. To use, you add
--precinctsReportingPct='{"bacon":0}'
, specifying the county IDs and reporting percents, to the CLI. This sets all the subunits to the same reporting percent, as well as overall result, so the above example will set all subunits in Bacon county to0
as well as the overall percent for Bacon county itself.Jira Ticket
ELEX-1959
Test Steps
After
Before