You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ri...@apache.org on 2008/07/17 18:14:09 UTC

svn commit: r677629 - in /incubator/qpid/trunk/qpid/java: common/src/main/java/org/apache/qpid/util/ systests/src/main/java/org/apache/qpid/test/framework/distributedtesting/ systests/src/main/java/org/apache/qpid/test/utils/

Author: ritchiem
Date: Thu Jul 17 09:14:08 2008
New Revision: 677629

URL: http://svn.apache.org/viewvc?rev=677629&view=rev
Log:
Moved the Reflection Wrapping code used by the system tests to the system test. If they are left in common then we must include the common directory when using the systest-testing frameworks no matter the version of Qpid being tested. Whilst this is probably not a problem as long as the classpath is carefully set up it does introduce another potential uncertainty to any test failure.

Added:
    incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtils.java
      - copied, changed from r677525, incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtils.java
    incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtilsException.java
      - copied, changed from r677525, incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtilsException.java
Removed:
    incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtils.java
    incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtilsException.java
Modified:
    incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/framework/distributedtesting/TestClient.java
    incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ConversationFactory.java

Modified: incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/framework/distributedtesting/TestClient.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/framework/distributedtesting/TestClient.java?rev=677629&r1=677628&r2=677629&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/framework/distributedtesting/TestClient.java (original)
+++ incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/framework/distributedtesting/TestClient.java Thu Jul 17 09:14:08 2008
@@ -27,8 +27,8 @@
 import org.apache.qpid.test.framework.TestUtils;
 import org.apache.qpid.test.framework.clocksynch.ClockSynchThread;
 import org.apache.qpid.test.framework.clocksynch.UDPClockSynchronizer;
-import org.apache.qpid.util.ReflectionUtils;
-import org.apache.qpid.util.ReflectionUtilsException;
+import org.apache.qpid.test.utils.ReflectionUtils;
+import org.apache.qpid.test.utils.ReflectionUtilsException;
 
 import org.apache.qpid.junit.extensions.SleepThrottle;
 import org.apache.qpid.junit.extensions.util.ParsedProperties;

Modified: incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ConversationFactory.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ConversationFactory.java?rev=677629&r1=677628&r2=677629&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ConversationFactory.java (original)
+++ incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ConversationFactory.java Thu Jul 17 09:14:08 2008
@@ -21,7 +21,7 @@
 package org.apache.qpid.test.utils;
 
 import org.apache.log4j.Logger;
-import org.apache.qpid.util.ReflectionUtils;
+import org.apache.qpid.test.utils.ReflectionUtils;
 
 import javax.jms.*;
 

Copied: incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtils.java (from r677525, incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtils.java)
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtils.java?p2=incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtils.java&p1=incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtils.java&r1=677525&r2=677629&rev=677629&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtils.java (original)
+++ incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtils.java Thu Jul 17 09:14:08 2008
@@ -18,7 +18,7 @@
  * under the License.
  *
  */
-package org.apache.qpid.util;
+package org.apache.qpid.test.utils;
 
 import java.lang.reflect.Constructor;
 import java.lang.reflect.InvocationTargetException;

Copied: incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtilsException.java (from r677525, incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtilsException.java)
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtilsException.java?p2=incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtilsException.java&p1=incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtilsException.java&r1=677525&r2=677629&rev=677629&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/util/ReflectionUtilsException.java (original)
+++ incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/ReflectionUtilsException.java Thu Jul 17 09:14:08 2008
@@ -18,7 +18,7 @@
  * under the License.
  *
  */
-package org.apache.qpid.util;
+package org.apache.qpid.test.utils;
 
 /**
  * Wraps a checked exception that occurs when {@link ReflectionUtils} encounters checked exceptions using standard