222 successfully fixed the bug produced once the Chamber of Deputies introduced a new column (cpf) to their datasets, but it broke the test suite.
What was done to achieve this purpose?
This PR continues #222 following the TDD methodology. It included the cpf column in the fixture and adjusted code and tests to the new reality. Also, this PR includes the version bump (ccce009) to trigger a new deploy to PyPI once the commit hits the master branch.
How to test if it really works?
The test suite should pass and the resulting CSV file should not have a cpf column.
Who can help reviewing it?
@sergiomario
TODO
[x] Code review and test this PR
[x] Merge this PR to @sergiomario's fix-reimbursements-cleaner branch (that is to say, incorporate these commits to #222)
[x] Merge #222 once these commits get to @sergiomario's fix-reimbursements-cleaner branch
What is the purpose of this Pull Request?
222 successfully fixed the bug produced once the Chamber of Deputies introduced a new column (
cpf) to their datasets, but it broke the test suite.What was done to achieve this purpose? This PR continues #222 following the TDD methodology. It included the
cpfcolumn in the fixture and adjusted code and tests to the new reality. Also, this PR includes the version bump (ccce009) to trigger a new deploy to PyPI once the commit hits themasterbranch.How to test if it really works? The test suite should pass and the resulting CSV file should not have a
cpfcolumn.Who can help reviewing it? @sergiomario
TODO
fix-reimbursements-cleanerbranch (that is to say, incorporate these commits to #222)fix-reimbursements-cleanerbranch