bvanelli / actualpy

Python API implementation for Actual server - reference https://actualbudget.org/
20 stars 4 forks source link

allow set self signed certs #59

Closed jlvcm closed 2 weeks ago

bvanelli commented 3 weeks ago

Tests are also failing, try:

diff --git a/tests/test_api.py b/tests/test_api.py
index e16805e..402b28d 100644
--- a/tests/test_api.py
+++ b/tests/test_api.py
@@ -33,6 +33,7 @@ def test_rename_delete_budget_without_file():
 @patch("requests.post", return_value=RequestsMock({"status": "error", "reason": "proxy-not-trusted"}))
 def test_api_login_unknown_error(_post):
     actual = Actual.__new__(Actual)
+    actual.cert = True
     actual.api_url = "localhost"
     with pytest.raises(AuthorizationError, match="Something went wrong on login"):
         actual.login("foo")
codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 96.95%. Comparing base (31f0e5d) to head (2c259c0). Report is 1 commits behind head on main.

Files Patch % Lines
actual/api/__init__.py 92.85% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #59 +/- ## ======================================= Coverage 96.95% 96.95% ======================================= Files 13 13 Lines 1838 1839 +1 ======================================= + Hits 1782 1783 +1 Misses 56 56 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

bvanelli commented 2 weeks ago

Merged, will come on the next release.

Thanks for your contribution.