Update tests for foreign currency check

[Sha-yol]

Add minimal tests for foreign currency check functionality.
Due to the caching, this required modifying the older test_main.py, otherwise an HTTP request would be made on importing main.

[Sha-yol]

I'm far from an expert myself, but caching complicates the testing. I haven't reloaded main in the other tests, but when testing the caching itself, I think it's better to reload it every time. Open for more suggestions down the line :)

…

On Fri, Oct 18, 2024, 13:58 Adyanth Hosavalike ***@***.***> wrote: ***@***.**** approved this pull request. ------------------------------ In tests/test_account_currency.py <#14 (comment)> : > +# Reload the main module in each test to ensure a clean slate +def reload_main(): + with patch('main.callApi') as mock_call_api: + mock_call_api.return_value.json.return_value = {'data': []} + import main + importlib.reload(main) + return main Is this the best way to do this? I am not familiar enough to comment. — Reply to this email directly, view it on GitHub <#14 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AHU22SSM3SXHL6ZGKIZTFBDZ4DSVHAVCNFSM6AAAAABQFJDMHSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNZXG43DKMRQHA> . You are receiving this because you authored the thread.Message ID: ***@***.***>