search

askonomm / shh

A CLI password manager designed for efficiency.
MIT License
22 stars 1 forks source link

Implement tags #15

Closed AbhinavOmprakash closed 2 years ago

AbhinavOmprakash commented 2 years ago

Closes #11

I have tested things, and they seem to work. But I may have missed a few things. so please please test them before merging

P.S. we need to add automated tests

askonomm commented 2 years ago

Re: automated tests. Do you know how to do that for CLI apps? I've only seen browser-based automated tests before.

AbhinavOmprakash commented 2 years ago

You could use monkey patching to patch the IO functions to take input from the program and test it that way. Just a guess, since I've not done something like this.

askonomm commented 2 years ago

You could use monkey patching to patch the IO functions to take input from the program and test it that way. Just a guess, since I've not done something like this.

I'll look into that over the weekend, thanks for the suggestion!