Mailing-List: contact issues-help@geode.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@geode.incubator.apache.org
Date: Mon, 11 Jul 2016 17:42:11 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@geode.incubator.apache.org
Message-ID: <JIRA.12972566.1464134183000.81029.1468258931425@Atlassian.JIRA>
In-Reply-To: <JIRA.12972566.1464134183000@Atlassian.JIRA>
References: <JIRA.12972566.1464134183000@Atlassian.JIRA> <JIRA.12972566.1464134183412@arcas>
Subject: [jira] [Commented] (GEODE-1452) Annotate disabled test methods with
 @Ignore
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Mon, 11 Jul 2016 17:42:23 -0000


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

ASF GitHub Bot commented on GEODE-1452:
---------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-geode/pull/184


> 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)