[SENTRY-60] TestSolrAuthzBinding does not properly test negative cases

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3.0
• Fix Version/s: 1.3.0
• Assignee: Gregory Chanan
• Reporter: Gregory Chanan
• Created: 04/Nov/13 23:35
• Updated: 06/Nov/13 01:20
• Resolved: 06/Nov/13 01:19

Description

 
expectAuthException(SolrAuthzBinding binding, Subject subject,
      Collection collection, EnumSet<SearchModelAction> action) throws Exception {

is supposed to check that an exception happens for the given binding, subject, collection, and actions, but it ignores the parameters. This didn't cause a test failure because the params it uses cause exceptions to be generated anyway.