The socrata/socrata_upload.py script is relatively stable, so I don't think it urgently needs unit tests, but I do suspect some of its abstractions may be suboptimal and I bet unit tests would help guide us in a gentle refactor to make it easier to maintain. They would also give me more confidence making major changes in the future, should that need to happen.
See https://github.com/ccao-data/alerts/pull/5/changes for an example of a PR that adds a bunch of unit tests using pytest.
The
socrata/socrata_upload.pyscript is relatively stable, so I don't think it urgently needs unit tests, but I do suspect some of its abstractions may be suboptimal and I bet unit tests would help guide us in a gentle refactor to make it easier to maintain. They would also give me more confidence making major changes in the future, should that need to happen.See https://github.com/ccao-data/alerts/pull/5/changes for an example of a PR that adds a bunch of unit tests using pytest.