Skip to content

Commit d7c5d7a

Browse files
committed
JUnit5 assertThrows SerializationProxyTest
1 parent a09dab7 commit d7c5d7a

1 file changed

Lines changed: 11 additions & 2 deletions

File tree

test/src/test/java/org/apache/commons/proxy2/serialization/SerializationProxyTest.java

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
import static org.junit.Assert.assertEquals;
2020
import static org.junit.Assert.assertTrue;
21+
import static org.junit.jupiter.api.Assertions.assertThrows;
2122

2223
import java.io.Serializable;
2324

@@ -31,6 +32,7 @@
3132
import org.junit.After;
3233
import org.junit.Before;
3334
import org.junit.Test;
35+
import org.junit.jupiter.api.function.Executable;
3436

3537
public class SerializationProxyTest extends AbstractProxyFactoryAgnosticTest
3638
{
@@ -91,14 +93,21 @@ public void clearProxyFactory()
9193
PROXY_FACTORY.remove();
9294
}
9395

94-
@Test(expected = SerializationException.class)
96+
@Test
9597
public void testNaive()
9698
{
9799
final Provider proxy = proxyFactory.createInterceptorProxy(null, implementProvider("foo"), Provider.class,
98100
Serializable.class);
99101
assertEquals("foo", proxy.getObject().getValue());
100102
assertTrue(Serializable.class.isInstance(proxy));
101-
SerializationUtils.roundtrip((Serializable) proxy);
103+
// FIXME Simplification once upgraded to Java 1.8
104+
final Executable testMethod = new Executable() {
105+
@Override
106+
public void execute() throws Throwable {
107+
SerializationUtils.roundtrip((Serializable) proxy);
108+
}
109+
};
110+
assertThrows(SerializationException.class, testMethod);
102111
}
103112

104113
@Test

0 commit comments

Comments
 (0)