Change in ovirt-engine[master]: core: Fix GetAllVmPoolsAttachedToUserQueryTest

amureini at redhat.com amureini at redhat.com
Sun Jan 27 09:03:31 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Fix GetAllVmPoolsAttachedToUserQueryTest
......................................................................


core: Fix GetAllVmPoolsAttachedToUserQueryTest

GetAllVmPoolsAttachedToUserQueryTest was broken by change-id
I59346bc9f1cf6d948c4401ddbd7bb365c67b5b05 which changed the query from a
user query to an admin query.

This patch removes the parts of the test that are only relevant to user
queries (i.e., permission filtering) so that it will only check the
expected behavior.

Change-Id: I5883d47c4ca074c83913449ecd80972139a7dcf2
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetAllVmPoolsAttachedToUserQueryTest.java
1 file changed, 6 insertions(+), 27 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5883d47c4ca074c83913449ecd80972139a7dcf2
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: ofri masad <omasad at redhat.com>



More information about the Engine-commits mailing list