Fix dbrestore on SQLite

[Lokkook]

Adding @rgaiacs changes about restoring to sqlite db with newlines

Bug fix

Description

This change fix a pb when restoring fields that contains newlines.

[johnthagen]

Related to #254 #258

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.22%. Comparing base (8f1307e) to head (3106325).
Report is 13 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #383      +/-   ##
==========================================
- Coverage   91.40%   91.22%   -0.19%     
==========================================
  Files          19       19              
  Lines         873      912      +39     
  Branches      157      164       +7     
==========================================
+ Hits          798      832      +34     
- Misses         40       43       +3     
- Partials       35       37       +2     

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

[Archmonger]

@johnthagen You mentioned something about the functional tests on this repo being iffy. Do you think that might be the reason why the test_create_dump_with_newline in this PR doesn't appear to run (according to the codecov hits)?

Unrelated note, I confirmed at least one issue on Windows appears to be related to gpg support. See dbbackup/tests/utils/add_public_gpg.

[johnthagen]

You mentioned something about the functional tests on this repo being iffy.

I don't fully recall.

Unrelated note, I confirmed at least one issue on Windows appears to be related to gpg support.

Maybe *nix specific tests like gpg could be marked as skipped on Windows: https://docs.pytest.org/en/7.1.x/how-to/skipping.html#skipping-test-functions

[Archmonger]

I'd need to confirm whether they should be skipped or not. It's very possible these tests failing is an indication that encryption support is broken on Windows.

[Archmonger]

@Lokkook @rgaiacs Are either of you interested in resolving the merge conflicts on this PR?

[rgaiacs]

I have been working with Django in the last couple of years. Sorry for not be able to resolve the conflicts.

[Lokkook]

@Lokkook @rgaiacs Are either of you interested in resolving the merge conflicts on this PR?

Hello ! I did it. I hope I did it well :) Tell me !

[Archmonger]

This can be merged after hitting full codecov. Seems like there's two logical branches that aren't being tested.

if line_str.startswith("INSERT") and not line_str.endswith(");\n"):
    sql_is_complete = False
    continue

if not sql_is_complete and line_str.endswith(");\n"):
    sql_is_complete = True

[Archmonger]

This looks good, thank you for the PR!