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

Henning Schmiedehausen commented on SHINDIG-808:
------------------------------------------------

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.

One (recommended) solution would be to create a Shindig test support Maven 
module which contains these base classes. This has been proposed a while back 
but I haven't seen it happen yet.

I don't like test-related classes polluting the release code base. People tend 
to think that test related stuff is available in the actual codebase and start 
to use it. So Shindig has now a dependency on hamcrest which is definitely a 
test related tool. 

> 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