CSV upload tests should call the route directly

[mbarton]

At the moment test_csv_upload is a mixture of calling read_csv and upload_csv. It would be better to make them true e2e tests and call the route directly so that we catch any problems in the code that shows the error messages to the user.

Probably can't do this before #347 as that would provide the mechanism to read back the errors in the tests