Review comments

[serboctor]

• As per pep8 convention doc-strings should use triple double quotes not single quotes. Please fix it here and every where else.
• This is a bit redundant, I'd use self.ovc directly. It is already short enough.
• Move the create before the VDCUser struct so all the main schema arguments are successive and clear.
• I dont like that one function in the tests will test different scenarios, the test functions should be smaller and test one specific case, that way when the tests fail, you'll know where the problem is exactly. For example, here the test_update function tests two cases, the case where it is a read only account and the update raises an error and the normal successful funciton. These should be two separate test functions: 1) test_update_readonly_account and 2) test_update. The same applies for all the other tests of all the templates. @zaibon do you have a different input regarding this?
• Incorrect indentation here
• I think this was added by mistake and needs to be removed.

[serboctor]

Also, in the readme of the templates, infront of template, I usually write the template uid not the template name which I think is nicer and makes it easier for people to just copy it and use it. Check here as an example https://github.com/zero-os/0-templates/blob/master/templates/container/README.md

[zaibon]

@zaibon do you have a different input regarding this?

No that is the correct way of doing it