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

Kirk Lund commented on GEODE-1452:
----------------------------------

Commit for GEODE-837 added @Ignore to many disabled tests. Need to sweep 
through all tests again to make sure everything has been annotated.

> Annotate disabled test methods with @Ignore
> -------------------------------------------
>
>                 Key: GEODE-1452
>                 URL: https://issues.apache.org/jira/browse/GEODE-1452
>             Project: Geode
>          Issue Type: Test
>          Components: tests
>            Reporter: Kirk Lund
>            Assignee: Kirk Lund
>
> Some of the older tests have test methods which were disabled by renaming the 
> test method from JUnit 3 syntax to something like "public void _testMethod" 
> or "public void disabled_testMethod".
> Most of these tests were then updated to JUnit 4, but the disabled tests 
> remain without @Ignore or @Test annotations.
> We should annotate these test methods with @Ignore and @Test so they are 
> correctly counted as skipped tests. Tickets should then be filed to analyze 
> whether these tests should be fixed or deleted.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to