votingworks / arlo

GNU Affero General Public License v3.0
141 stars 25 forks source link

Catch all failures to parse Dominion interpretation values as integers #1886

Closed arsalansufi closed 7 months ago

arsalansufi commented 7 months ago

Overview

Welp, as soon as we rolled out https://github.com/votingworks/arlo/pull/1885, we encountered a Dominion CVR file with a value that couldn't be parsed but that also didn't contain a percent:

invalid literal for int() with base 10: '0 (0)'

I'm changing course and now catching all failures to parse Dominion interpretation values as integers:

screenshot-1
screenshot-2

Testing