please send a status of what works, what not, and gaps to close.
thanks

On 06/04/2012 05:24 PM, a...@redhat.com wrote:
Asaf Shakarchi has submitted this change and it was merged.

Change subject: restapi: non admin user api - Filtering user queries (#783087)
......................................................................


restapi: non admin user api - Filtering user queries (#783087)

https://bugzilla.redhat.com/783087

- A global support for filtering content based on user permissions for all
   query executions based on the 'filter' HTTP header.
   Note: The 'filter' attribute header is stateless and must be specified by 
the client per HTTP request.

- Added a unit test for BackendResourceTest currently for testing the 'filter' 
HTTP header.

- Abstracted base test class from resource centric base test class

- Changed TestHelper.getMethod to seek for 'isMethod()' if 'getMethod()' wasn't 
found.

Change-Id: I8e5e23cbb8e883f6005f69be46fa528ac6e7518c
Signed-off-by: Asaf Shakarchi<a...@redhat.com>
---
M 
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendResource.java
M 
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BaseBackendResource.java
A 
backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendBaseTest.java
M 
backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendResourceTest.java
M 
backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java
A 
backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendResourceTest.java
M 
backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmResourceTest.java
7 files changed, 490 insertions(+), 387 deletions(-)

Approvals:
   Asaf Shakarchi: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8e5e23cbb8e883f6005f69be46fa528ac6e7518c
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Asaf Shakarchi<a...@redhat.com>
Gerrit-Reviewer: Asaf Shakarchi<a...@redhat.com>
Gerrit-Reviewer: Michael Pasternak<mpast...@redhat.com>
Gerrit-Reviewer: Ori Liel<ol...@redhat.com>
_______________________________________________
Engine-commits mailing list
engine-comm...@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-commits

_______________________________________________
Engine-devel mailing list
Engine-devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel

Reply via email to