You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2014/06/13 20:51:02 UTC

[jira] [Commented] (ACCUMULO-2549) ShellTest sometimes fails because of order of authentication strings returned

    [ https://issues.apache.org/jira/browse/ACCUMULO-2549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14030991#comment-14030991 ] 

ASF subversion and git services commented on ACCUMULO-2549:
-----------------------------------------------------------

Commit e6171e6b2a293ac5561e20147e005df2d0e6c06a in accumulo's branch refs/heads/1.6.1-SNAPSHOT from [~mallen]
[ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=e6171e6 ]

ACCUMULO-2549 Fixes ShellTest

Authorizations were coming back from getauths in an unexpected
order.  This patch changes the test to expect not a specific
string a but a list of expected authorization entries.

Signed-off-by: Mike Drob <md...@cloudera.com>


> ShellTest sometimes fails because of order of authentication strings returned
> -----------------------------------------------------------------------------
>
>                 Key: ACCUMULO-2549
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2549
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.5.1
>            Reporter: Michael Allen
>            Priority: Minor
>             Fix For: 1.5.2, 1.6.1, 1.7.0
>
>         Attachments: ACCUMULO-2549-Fixes-ShellTest.patch
>
>
> {{org.apache.accumulo.core.util.shell.ShellTest}} sometimes fails for me when run from the Maven command line.  Tracing down, I see that line 154 (1.5.1 version) contains the following
> {noformat}
>     exec("getauths", true, "y,z,x");
> {noformat}
> This test expects that the set of authorizations returns from the {{getauths}} command always come in the particular order "y,z,x".  For whatever reason, on my machine, they came out in a different order ("x,y,z" for the record).
> My proposed fix (patch shortly) would be to have the test look for those individual label strings returned within the authorization string.  I think this is a more accurate and future proof test than what's in there now.



--
This message was sent by Atlassian JIRA
(v6.2#6252)