Updated the database test to catch more bugs and get up to 100% coverage in the database package. Also fixed an issue with the RSAKey fromJSON method that didn't work correctly.
Why
More coverage is always better, I personally felt we we're missing a few useful test cases for the database so I added them.
I will resolve the merge conflicts on this PR as soon as #93 (simple PR that has already been reviewed and approved) and #96 (important fix) are merged into develop 🙂
Relevant Issues: -
Related Pull Requests: -
What
Updated the database test to catch more bugs and get up to 100% coverage in the
database
package. Also fixed an issue with the RSAKey fromJSON method that didn't work correctly.Why
More coverage is always better, I personally felt we we're missing a few useful test cases for the database so I added them.
How
Run the tests locally to verify they work.
Alternative implementation
n/a
Notes
n/a