This is odd. The test failures seem unrelated to the changes.

It does seem to somehow be related to this PR's changes.

Just rebased #536 and got no CI errors.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.44%. Comparing base (8f1307e) to head (25a4bb7).
Report is 18 commits behind head on master.

@@            Coverage Diff             @@
##           master     #540      +/-   ##
==========================================
+ Coverage   91.40%   91.44%   +0.03%     
==========================================
  Files          19       19              
  Lines         873      912      +39     
  Branches      157      154       -3     
==========================================
+ Hits          798      834      +36     
- Misses         40       43       +3     
  Partials       35       35              

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

I got confused by the sqlite errors, but those seem to be expected/ok.

I couldn't track down the problem completely, but there seems to be some modification of the USER setting that breaks the interaction with get() and a default value.

At some point during the tests, self.settings.get("USER","") returns None which could only happen if None was assigned explicitely. I couldn't find that place in the code, though.

Could be this LOC, not sure though

https://github.com/jazzband/django-dbbackup/blob/master/dbbackup%2Ftests%2Ftest_connectors%2Ftest_postgresql.py#L49

Will go ahead and merge this since it's a step up from current behavior. If someone in the future needs self.settings.get("USER","") then they will need to debug it within their PR.