PasswordState icon indicating copy to clipboard operation
PasswordState copied to clipboard

Improve testing on the module and enhance security

Open trir262 opened this issue 6 years ago • 0 comments

I removed insecure TLS from the module, refactored the testing capabilities by adding tests per function.

Description

  • Removed insecure TLS support
  • Modified plural functions to singular
  • Updated documentation for singular functions
  • Refactored and improved testing part of the library

Related Issue

Fixes #13 Fixes #14 Fixes #15 Fixes #16

Motivation and Context

I want to further improve the module because we use PasswordState in our organization, however we can only use winapi. I want to include the possibility to add it to the module as well.

How Has This Been Tested?

Implemented improved testing framework, all tests returned success

Screenshots (if appropriate):

Types of changes

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

Checklist:

  • [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 read the CONTRIBUTING document.
  • [x] I have added tests to cover my changes.
  • [x] All new and existing tests passed.

trir262 avatar Dec 28 '19 21:12 trir262