Update check message on completion

[samuel-w]

Other messages claim to complete without checking the returncode, for example BorgExtractThread. Should we change that?

Fixes #671

[m3nu]

Good idea. Now that we have 2 areas for status updates, we could add more messages on success/failure based on return code without hiding the last error log message.

[codecov-io]

Codecov Report

Merging #672 (f3f0a97) into master (8814624) will increase coverage by 2.63%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #672      +/-   ##
==========================================
+ Coverage   70.86%   73.49%   +2.63%     
==========================================
  Files          53       53              
  Lines        3504     3505       +1     
==========================================
+ Hits         2483     2576      +93     
+ Misses       1021      929      -92     

Impacted Files	Coverage Δ
src/vorta/borg/check.py	70.00% <0.00%> (-3.69%)	⬇️
src/vorta/models.py	81.12% <0.00%> (+0.51%)	⬆️
src/vorta/borg/borg_thread.py	79.28% <0.00%> (+1.77%)	⬆️
src/vorta/config.py	100.00% <0.00%> (+5.55%)	⬆️
src/vorta/network_status/abc.py	88.23% <0.00%> (+5.88%)	⬆️
src/vorta/utils.py	60.86% <0.00%> (+6.52%)	⬆️
src/vorta/keyring/abc.py	88.23% <0.00%> (+8.82%)	⬆️
src/vorta/notifications.py	84.84% <0.00%> (+13.63%)	⬆️
src/vorta/network_status/darwin.py	61.01% <0.00%> (+27.11%)	⬆️
src/vorta/autostart.py	96.96% <0.00%> (+33.33%)	⬆️
... and 1 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8814624...f3f0a97. Read the comment docs.