Add file_path param to download_file() to directly download files to disk

[MHHukiewitz]

Instead of keeping them in memory. Addresses #37.

[codecov]

Codecov Report

Attention: Patch coverage is 96.72131% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 84.17%. Comparing base (ea4acdc) to head (69b9cf0).

Files	Patch %	Lines
tests/unit/conftest.py	93.10%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #115      +/-   ##
==========================================
+ Coverage   83.67%   84.17%   +0.50%     
==========================================
  Files          27       27              
  Lines        1084     1131      +47     
  Branches      181      188       +7     
==========================================
+ Hits          907      952      +45     
- Misses        175      176       +1     
- Partials        2        3       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MHHukiewitz]

@hoh I think codecov is incorrectly configured. It currently also measures test file coverage... Which doesn't make sense.

[hoh]

@hoh I think codecov is incorrectly configured. It currently also measures test file coverage... Which doesn't make sense.

Why not ? Tests that don't run do not make sense.

[MHHukiewitz]

Why not ? Tests that don't run do not make sense.

Because I just added some cleanup checks after downloading to the file, in order to check the automatic directory and file creation every time correctly

[hoh]

The approach for dual use of the download_file seems very confusing, see my comment on the topic.

This branch is re-implementing in it a lot of patterns that are already covered by the Python standard library.

[1yam]

LGTM

Concerns have been addressed, reviewed by Lyam.