accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-4432) Write test to assert that all ITs are categorized
Date Wed, 31 Aug 2016 03:42:20 GMT

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

Christopher Tubbs commented on ACCUMULO-4432:
---------------------------------------------

Checkstyle can analyze the type hierarchy. That's why it has to run after the compile phase.
Some tasks are source checks only. Others require the class files. That said, I checked, and
there doesn't appear to be any existing checkstyle module to do this, except RegEx, which
would be terribly ugly.

Checkstyle modules aren't as easy to write as maven plugins, so I might code a maven plugin
up later this week. If you have specific configuration options to use as requirements, let
me know.

> Write test to assert that all ITs are categorized
> -------------------------------------------------
>
>                 Key: ACCUMULO-4432
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4432
>             Project: Accumulo
>          Issue Type: Task
>          Components: test
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>             Fix For: 1.7.3, 1.8.1, 2.0.0
>
>
> Since we switched to using Categories and groups to determine which tests are run, we've
introduced the possibility that we have tests which are not categorized (by accident) and
they would subsequently never be run by the build.
> [~busbey] was a fine enough gent this morning and pointed me to HBase's [TestInterfaceAudienceAnnotations|https://github.com/apache/hbase/blob/master/hbase-client/src/test/java/org/apache/hadoop/hbase/TestInterfaceAudienceAnnotations.java]
test class which does a similar thing for HBase. I'm thinking that we could write a similar
unit test for Accumulo to prevent this edge-case.



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

Mime
View raw message