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/04/23 17:06:25 UTC

[jira] [Commented] (ACCUMULO-2718) [FindBugs] Confusing both setup() and setUp() methods in AuditMessageIT

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

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

Commit 1f2420c79d318a080f1e7db9e5818cc71fc9a5b0 in accumulo's branch refs/heads/findbugs from [~ctubbsii]
[ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1f2420c ]

ACCUMULO-2718 Make two test method names vary by more than case


> [FindBugs] Confusing both setup() and setUp() methods in AuditMessageIT
> -----------------------------------------------------------------------
>
>                 Key: ACCUMULO-2718
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2718
>             Project: Accumulo
>          Issue Type: Sub-task
>          Components: test
>            Reporter: Christopher Tubbs
>            Assignee: Christopher Tubbs
>            Priority: Minor
>              Labels: findbugs
>             Fix For: 1.6.1, 1.7.0
>
>
> FindBugs found a very confusing method in a subclass, AuditMessageIT (rank 4 bug). It's a small and trivial change and has no bearing on the test itself, but it gets us FindBugs validation coverage for all bugs through rank 4, so it's worth changing.



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