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 Fri, 09 Sep 2016 19:05:22 GMT

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

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

Agreed on just having it in 2.0.0. I'm not sure it's worth fixing in older branches. It was
mainly important if there were going to be pre-defined profiles and defaults whose union was
not a universal set. So, some tests would never be able to run, no matter what config/profile
one used.

Now that the defaults run everything, and the profiles which attempt to pre-define a particular
mutually exclusive set don't exist, this it's probably not that important in the earlier branches.

The earlier branches can run the plugin in an auto-activated jdk8 profile when building with
Java 8, so it'll run with Java 8, but won't otherwise cause problems. (I forgot that was an
easy thing to do.)

> 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