[jira] [Commented] (ACCUMULO-4058) mvn test on ScannerContextIT will fail if test jar is not already built.

["Christopher Tubbs (JIRA)" <ji...@apache.org>]

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

Christopher Tubbs commented on ACCUMULO-4058:
---------------------------------------------

Is this worth pursuing? This is a build test. Do we need to support running these outside of the build? If we do, then the class itself should create the jar in code, on demand. But, I'm not convinced that's worth supporting. The test is already tied to Maven, using the target directories, etc.

At the very least, I don't think this is "Critical" as it is currently marked.

> mvn test on ScannerContextIT will fail if test jar is not already built.
> ------------------------------------------------------------------------
>
>                 Key: ACCUMULO-4058
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4058
>             Project: Accumulo
>          Issue Type: Bug
>          Components: test
>            Reporter: Josh Elser
>            Priority: Critical
>              Labels: newbie
>             Fix For: 1.8.1
>
>
> https://github.com/apache/accumulo/blob/master/test/pom.xml#L230
> {code}
>         <plugin>
>           <groupId>org.apache.maven.plugins</groupId>
>           <artifactId>maven-jar-plugin</artifactId>
>           <executions>
>             <execution>
>               <goals>
>                 <goal>test-jar</goal>
>               </goals>
>               <phase>pre-integration-test</phase>
>               <configuration>
>                 <finalName>TestIterators</finalName>
>                 <classifier />
>                 <includes>
>                   <include>org/apache/accumulo/test/functional/ValueReversingIterator.class</include>
>                 </includes>
>               </configuration>
>             </execution>
>           </executions>
>         </plugin>
> {code}
> Binding this to pre-integration-test instead of test-compile (or something else) will likely break the test that requires this special jar if the test is manually invoked. This is a common pattern among devs (although {{mvn verify -Dit.test}} is more correct).



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