You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by "Gregory Chanan (JIRA)" <ji...@apache.org> on 2013/11/05 00:38:17 UTC
[jira] [Commented] (SENTRY-60) TestSolrAuthzBinding does not
properly test negative cases
[ https://issues.apache.org/jira/browse/SENTRY-60?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13813417#comment-13813417 ]
Gregory Chanan commented on SENTRY-60:
--------------------------------------
Trivial patch; I ran the test again with the patch applied and it succeeds.
> TestSolrAuthzBinding does not properly test negative cases
> ----------------------------------------------------------
>
> Key: SENTRY-60
> URL: https://issues.apache.org/jira/browse/SENTRY-60
> Project: Sentry
> Issue Type: Bug
> Affects Versions: 1.3.0
> Reporter: Gregory Chanan
> Assignee: Gregory Chanan
> Fix For: 1.3.0
>
> Attachments: SENTRY-60.patch
>
>
> {code}
> expectAuthException(SolrAuthzBinding binding, Subject subject,
> Collection collection, EnumSet<SearchModelAction> action) throws Exception {
> {code}
> 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.
--
This message was sent by Atlassian JIRA
(v6.1#6144)