chiefy / vaulted

nodejs based wrapper for HashiCorp's Vault HTTP API
https://chiefy.github.io/vaulted
MIT License
47 stars 6 forks source link

chore(tests): Test and document core #11

Closed kenjones-cisco closed 8 years ago

kenjones-cisco commented 8 years ago

Implement test cases for the core library and documentation of each of the public methods.

As a result of adding test cases for all modules, all API methods will now return a promise that will be resolved or rejected. And several bug fixes or inconsistencies were resolved.

The init method was made more user-friendly such that if the Vault has already been initialized the Vault is returned instead of rejecting the request.

Because the internal state of the Vault of critical and the ability to repeatedly restart the Vault is required, internal state management has been made to be recoverable.

closes #9

chiefy commented 8 years ago

@kenjones-cisco sorry, I finally got a few free minutes. Is this RTM? If so, I say pull the switch! Thinking we should bump to 2.0.0 since there's a lot of changes.

kenjones-cisco commented 8 years ago

This one is ready. Then I have another 7 or so features ready to submit a a PR for. After merging them all, a bump to 2.0 makes sense.