[Engine-devel] Change in ovirt-engine[master]: restapi: non admin user api - Filtering user queries (#78308...

[Itamar Heim]

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

On 06/04/2012 05:24 PM, asaf at 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<asaf at 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<asaf at redhat.com>
> Gerrit-Reviewer: Asaf Shakarchi<asaf at redhat.com>
> Gerrit-Reviewer: Michael Pasternak<mpastern at redhat.com>
> Gerrit-Reviewer: Ori Liel<oliel at redhat.com>
> _______________________________________________
> Engine-commits mailing list
> Engine-commits at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-commits