Change in ovirt-engine-sdk[master]: Check types of locator parameters

Code Review gerrit at ovirt.org
Mon Jan 23 19:22:16 UTC 2017


>From Ondra Machacek <omachace at redhat.com>:

Ondra Machacek has submitted this change and it was merged.

Change subject: Check types of locator parameters
......................................................................


Check types of locator parameters

Currently the types of the parameters of locator methods aren't checked.
This patch addresses that issue, so that the parameters will be checked
and useful exceptions will be generated.

Change-Id: I8b170c5ee917701b56960d1ffe23d3833d8df833
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java
M sdk/tests/test_check_types.py
2 files changed, 20 insertions(+), 0 deletions(-)

Approvals:
  Ondra Machacek: Looks good to me, approved
  Juan Hernandez: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/70999
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8b170c5ee917701b56960d1ffe23d3833d8df833
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine-sdk
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list