Misc bug fixes and features, continued the refactor

[DownstreamDataTeam]

Description of change

• added currency field to loan_accounts
• added payment_details field to deposit_transactions
• added status field to tasks stream
• added sorting to gl_journal_entries stream to fix missing record at large extractions (very big offset)
• fixed loan accounts being skipped if updated while the extraction is running
• replaced endpoint_config dict with class properties
• created unit tests for auxiliary functions
• added logging to refactored code
• implemented unit tests and functional tests for loan_accounts and loan_repayments streams
• refactored deposit_accounts and cards streams (including unit tests)

Manual QA steps

• compared Tap output before and after the changes
• ran unit tests and Tap Tester

Rollback steps

• revert this branch

[cmerrick]

Hi @DownstreamDataTeam, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

[dmosorast]

This conflicts with the setup.py already in use. Pip should pick up these requirements (except for mock, which is a test requirement) when running pip install -e . (with the -e meaning installing from source)

[dmosorast]

This should be only installed during test running, in setup.py, we'd usually put this into a 'test' extras, much like the 'dev' one specified here: https://github.com/DownstreamDataTeam/tap-mambu/blob/release/37/setup.py#L16-L22

[dmosorast]

These should be under a test profile in setup.py, as mentioned above. To keep it at only one way to install dependencies in the repo.

For running the tests with the gitlab-ci config (or circleci, which I can update after merge), you can use pip install -e .[test] to install the test profile dependencies along with the tap's regular dependencies.

[singer-bot]

You did it @DownstreamDataTeam!

Thank you for signing the Singer Contribution License Agreement.