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)