patched tests to use 1.4 compatible synatx and also to allow for the fact that maven...

git-svn-id: http://anonsvn.jboss.org/repos/javassist/trunk@536 30ef5769-5b8d-40dd-aea6-55b5d6557bb3

diff --git a/src/test/test/javassist/proxy/ProxyCacheGCTest.java b/src/test/test/javassist/proxy/ProxyCacheGCTest.java
index 062872b72fc9b6850756f627df57a6aed838a020..379fefc204d531901da0506289f39e44bda41682 100644 (file)
--- a/src/test/test/javassist/proxy/ProxyCacheGCTest.java
+++ b/src/test/test/javassist/proxy/ProxyCacheGCTest.java
@@ -38,6 +38,7 @@ public class ProxyCacheGCTest extends TestCase
     public void testCacheGC()
     {
         ClassLoader oldCL = Thread.currentThread().getContextClassLoader();
+        try {
         ProxyFactory.useCache = false;
         for (int i = 0; i < REPETITION_COUNT; i++) {
             ClassLoader newCL = new TestLoader();
@@ -48,6 +49,9 @@ public class ProxyCacheGCTest extends TestCase
                 Thread.currentThread().setContextClassLoader(oldCL);
             }
         }
+        } finally {
+            ProxyFactory.useCache = true;
+        }
     }
 
     /**

diff --git a/src/test/test/javassist/proxy/ProxyFactoryCompatibilityTest.java b/src/test/test/javassist/proxy/ProxyFactoryCompatibilityTest.java
index 069d0c9c3e2d7a1836ea69adfd8a9a5b83649dc4..5b72d82e1475222588591ebef806ec61391f3db9 100644 (file)
--- a/src/test/test/javassist/proxy/ProxyFactoryCompatibilityTest.java
+++ b/src/test/test/javassist/proxy/ProxyFactoryCompatibilityTest.java
@@ -41,6 +41,7 @@ public class ProxyFactoryCompatibilityTest extends TestCase
 
     public void testFactoryCompatibility() throws Exception
     {
+        System.out.println("ProxyFactory.useCache = " + ProxyFactory.useCache);
         // create a factory which, by default, uses caching
         ProxyFactory factory = new ProxyFactory();
         factory.setSuperclass(TestClass.class);

diff --git a/src/test/test/javassist/proxy/ProxySerializationTest.java b/src/test/test/javassist/proxy/ProxySerializationTest.java
index e79bcb481e8e15ea0f4d780f26dfac3457355d63..28125de01a7350095b8b172cb58d074a3e2f9d76 100644 (file)
--- a/src/test/test/javassist/proxy/ProxySerializationTest.java
+++ b/src/test/test/javassist/proxy/ProxySerializationTest.java
@@ -30,8 +30,8 @@ public class ProxySerializationTest extends TestCase
 
         try {
             String name = "proxytest_1";
-            Constructor constructor = proxyClass.getConstructor(String.class);
-            TestClass proxy = (TestClass)constructor.newInstance(name);
+            Constructor constructor = proxyClass.getConstructor(new Class[] {String.class});
+            TestClass proxy = (TestClass)constructor.newInstance(new Object[] {name});
             ((ProxyObject)proxy).setHandler(handler);
             ByteArrayOutputStream bos = new ByteArrayOutputStream();
             ObjectOutputStream out = new ObjectOutputStream(bos);
@@ -41,10 +41,10 @@ public class ProxySerializationTest extends TestCase
             ByteArrayInputStream bis = new ByteArrayInputStream(bytes);
             ObjectInputStream in = new ObjectInputStream(bis);
             TestClass newProxy = (TestClass)in.readObject();
-            // inherited fields should have been deserialized
-            assert(newProxy.getName() == null);
+            // inherited fields should not have been deserialized
+            assertTrue("new name should be null", newProxy.getName() == null);
             // since we are reading into the same JVM the new proxy should have the same class as the old proxy
-            assert(newProxy.getClass() == proxy.getClass());
+            assertTrue("classes should be equal", newProxy.getClass() == proxy.getClass());
         } catch (Exception e) {
             e.printStackTrace();
             fail();
@@ -57,8 +57,8 @@ public class ProxySerializationTest extends TestCase
 
         try {
             String name = "proxytest_2";
-            Constructor constructor = proxyClass.getConstructor(String.class);
-            TestClass proxy = (TestClass)constructor.newInstance(name);
+            Constructor constructor = proxyClass.getConstructor(new Class[] {String.class});
+            TestClass proxy = (TestClass)constructor.newInstance(new Object[] {name});
             ((ProxyObject)proxy).setHandler(handler);
             ByteArrayOutputStream bos = new ByteArrayOutputStream();
             ProxyObjectOutputStream out = new ProxyObjectOutputStream(bos);
@@ -69,9 +69,9 @@ public class ProxySerializationTest extends TestCase
             ProxyObjectInputStream in = new ProxyObjectInputStream(bis);
             TestClass newProxy = (TestClass)in.readObject();
             // inherited fields should have been deserialized
-            assert(proxy.getName() == newProxy.getName());
+            assertTrue("names should be equal", proxy.getName().equals(newProxy.getName()));
             // since we are reading into the same JVM the new proxy should have the same class as the old proxy
-            assert(newProxy.getClass() == proxy.getClass());
+            assertTrue("classes should still be equal", newProxy.getClass() == proxy.getClass());
         } catch (Exception e) {
             e.printStackTrace();
             fail();