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

["Josh Elser (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ACCUMULO-4058?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Josh Elser updated ACCUMULO-4058:
---------------------------------
    Priority: Minor  (was: Critical)

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