accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-2242) Add PowerMock to Security testing.
Date Thu, 23 Jan 2014 14:42:38 GMT

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

Bill Havanki commented on ACCUMULO-2242:
----------------------------------------

Hi Dave! It looks like we're using 1.5, so hopefully bumping up to 1.5.3 won't be onerous.
Thanks very much for the tip!

Despite me having used Powermock in one or two places so far in the code, I generally prefer
reworking the design of the classes under test so that we don't need the crazy capabilities
Powermock provides, like mocking static methods. So, hopefully whoever works this issue can
see if it's feasible and worthwhile to do that instead. If not, no big deal; there's a huge
reliance on static methods throughout Accumulo anyway.

> Add PowerMock to Security testing.
> ----------------------------------
>
>                 Key: ACCUMULO-2242
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2242
>             Project: Accumulo
>          Issue Type: Bug
>          Components: test
>    Affects Versions: 1.4.5, 1.5.0, 1.6.0
>            Reporter: Mike Drob
>            Priority: Minor
>             Fix For: 1.7.0
>
>
> From the TODO in AddressUtilTest.java:
> {noformat}
> replace all of this with Powermock on the Security class
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message