vorta icon indicating copy to clipboard operation
vorta copied to clipboard

Issue/2198 fix

Open gargolito opened this issue 9 months ago • 0 comments

keyring/kwallet.py: Modified VortaKWallet5Keyring to use enums for results and added error handling. More logging. Reorganized methods, added more doc strings, added pytests

Description

keyring/kwallet.py: Modified VortaKWallet5Keyring to use enums for results and added error handling. More logging. Reorganized methods, added more doc strings, added pytests

Related Issue

Issue #2198

Motivation and Context

See Issue 2198 Added option to prompt user for wallet name if none can be found or determined.

How Has This Been Tested?

Pytest unit tests and mocks, used QT 6.9 libraries in Kubuntu Linux. (Noble 24.04), and "works-on-my-machine" test :)

Types of changes

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [x] I have read the CONTRIBUTING guide.
  • [x] My code follows the code style of this project.
  • [ ] My change requires a change to the documentation.
  • [ ] I have updated the documentation accordingly.
  • [x] I have added tests to cover my changes.
  • [x] All new and existing tests passed.

I provide my contribution under the terms of the license of this repository and I affirm the Developer Certificate of Origin.

gargolito avatar May 30 '25 18:05 gargolito