Skip to content
This repository has been archived by the owner on Mar 24, 2021. It is now read-only.

Commit

Permalink
Add failing test to recreate EVL upload error
Browse files Browse the repository at this point in the history
If a cell is an empty string, it is correctly removed and there's no
attempt to convert it to an integer. Recently we've seen a sneaky
whitespace which isn't treated the same. This causes an upload failure.
This test modification reproduces the problem.
  • Loading branch information
Paul Furley committed Dec 18, 2013
1 parent 2e098e2 commit b8cb626
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tests/contrib/test_evl_upload_filters.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ def test_converts_missing_failure_as_zero(self):
self.ignore_rows(4) +
[["No tax-disc failure", 0, '', 0, 2]] +
[["No sorn failure", 1, 20, 0, '']] +
[["Failure with whitespace in value", 1, 20, 0, ' ']] +
[[None, "None in first column means end of failures list"]]
]

Expand Down

0 comments on commit b8cb626

Please sign in to comment.