[ 
https://issues.apache.org/jira/browse/SHINDIG-808?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12658892#action_12658892
 ] 

Vincent Siveton commented on SHINDIG-808:
-----------------------------------------

bq: This would require moving the class out of the src/test/java tree into 
src/main/java as the test classes are not put into the dependency jars. 

Of course no! You could use a type for dependency, something like the following:

parent
{noformat}
  <dependencyManagement>
      <dependency>
        <groupId>org.apache.shindig</groupId>
        <artifactId>shindig-common</artifactId>
        <version>1.1-SNAPSHOT</version>
        <type>test-jar</type>
      </dependency>
...
  </dependencyManagement>
{noformat}

shindig-common:
{noformat}
      <plugin>
        <groupId>org.apache.maven.plugins</groupId>
        <artifactId>maven-jar-plugin</artifactId>
        <executions>
          <execution>
            <goals>
              <goal>test-jar</goal>
            </goals>
          </execution>
        </executions>
      </plugin>
{noformat}

shindig-gadgets
{noformat}
    <dependency>
        <groupId>org.apache.shindig</groupId>
        <artifactId>shindig-common</artifactId>
      <type>test-jar</type>
      <scope>test</scope>
    </dependency>
{noformat}

> Duplicate EasyMockTestCase class
> --------------------------------
>
>                 Key: SHINDIG-808
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-808
>             Project: Shindig
>          Issue Type: Improvement
>    Affects Versions: trunk, 1.0.0-rc1
>            Reporter: Vincent Siveton
>            Priority: Trivial
>
> We have two EasyMockTestCase classes:
> http://svn.apache.org/repos/asf/incubator/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/EasyMockTestCase.java
> http://svn.apache.org/repos/asf/incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/EasyMockTestCase.java
> I propose to merge them and put the new class in common test.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to