Could you please fix the codestyle issues and update this to the latest version of staging? Also, please look over the tests again. There is for example no check for the return value of decrypt().
These test are not testing so much of the class, I would close this PR. Further more at the testing code sprint we removed all the placeholder test code because we have had a lot of needs to be implemented "test", so this PR can't be merged anyway.
Closing based on comments.
Thanks for your contribution anyway. If you want to propose a updated and more complete unit test, feel free to create a new PR.
brianteeman
- 
nicksavov
- 
That's fantastic thanks.