-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21877/
-----------------------------------------------------------

(Updated May 23, 2014, 8:59 p.m.)


Review request for sentry and Gregory Chanan.


Changes
-------

Addressed Greg's comments.
Renamed runAllQueryChecks -> checkNonAdminUsers


Bugs: SENTRY-230
    https://issues.apache.org/jira/browse/SENTRY-230


Repository: sentry


Description
-------

Sentry tests should include failure scenarios for cases where a user who has 
doc level permissions on a collection but has no index level permissions to 
access the collection.

Cases covered:
User w/ doc permissions set + w/o Index level permissions -> QUERY FAIL
User w/o doc permissions + w/o index level permissions -> QUERY FAIL


Diffs (updated)
-----

  
sentry-tests/sentry-tests-solr/src/test/java/org/apache/sentry/tests/e2e/solr/TestDocLevelOperations.java
 31ecd5b685e61fb260a9335e751121a3a3f700d5 
  
sentry-tests/sentry-tests-solr/src/test/resources/solr/sentry/test-authz-provider.ini
 96ab0d456ce4003b18c4ad4c779ebdaca4a9acfc 

Diff: https://reviews.apache.org/r/21877/diff/


Testing
-------

Ran the e2e tests


Thanks,

Vamsee Yarlagadda

Reply via email to