search

cdhunt / SecretManagement.1Password

This is a SecretManagement extension for 1Password.
MIT License
22 stars 14 forks source link

Tests #24

Closed theaquamarine closed 2 years ago

theaquamarine commented 2 years ago

They currently rely on the vault being signed in/unlocked/etc, but it's enough for Invoke-Pester to work.

Most of the ideas in https://github.com/cdhunt/SecretManagement.1Password/issues/12 are implemented- I left byte[] and hashtable tests for Set-Secret out since they hit the same issue as String and would be virtually identical.

cdhunt commented 2 years ago

Thank you for the PRs, @theaquamarine! Could you consolidate the changes (#24, #25, #26, #27) into one PR?

theaquamarine commented 2 years ago

Sure, I've combined them all in to this branch, enabled tests for the features fixed in the other branches, and also fixed Get-SecretInfo's filtering.

cdhunt commented 2 years ago

Thank you for the contibution. Sorry for the delay.