Change in ovirt-engine[master]: tools: Adds tests of argument values in engine-manage-domain...

yzaslavs at redhat.com yzaslavs at redhat.com
Sun Jun 30 03:06:52 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: tools: Adds tests of argument values in engine-manage-domains
......................................................................


tools: Adds tests of argument values in engine-manage-domains

Current implementation tests only if required arguments are entered on
command line, but it doesn't test if their values has been entered.
This patch adds test, that if an argument requires value, its value
has to be also entered on command line.

Change-Id: Ie6c41e209539b14b4499f424c54584401b6312e4
Bug-Url: https://bugzilla.redhat.com/927949
Signed-off-by: Martin Perina <mperina at redhat.com>
---
M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java
M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomainsResultEnum.java
2 files changed, 18 insertions(+), 1 deletion(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


-- 
To view, visit http://gerrit.ovirt.org/15050
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie6c41e209539b14b4499f424c54584401b6312e4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list