You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2011/03/01 23:45:37 UTC

[jira] Commented: (CASSANDRA-2242) Improve BRAFTest

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

Jonathan Ellis commented on CASSANDRA-2242:
-------------------------------------------

Let's not use @Test(expected=X) in tests that also contain other assertions -- the test will pass if an exception is thrown at the "wrong" time, too.

> Improve BRAFTest
> ----------------
>
>                 Key: CASSANDRA-2242
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2242
>             Project: Cassandra
>          Issue Type: Test
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.4
>
>         Attachments: CASSANDRA-2242.patch
>
>
> BRAF is insufficiently tested (see CASSANDRA-2241).  I'd like to get this up to 100% line coverage.  (Not sure what it's actually at right now, since "ant codecoverage" doesn't work.)

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira