dorkbox/Network
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Sending a class over the network always requires explicit registration now (previously, RMI message implementations did not require explicit registration)

Browse Source
This commit is contained in:
nathan 2019-01-09 23:41:18 +01:00
parent ff350991a4
commit 05f3272f96
5 changed files with 16 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
test/dorkbox/network/rmi/RmiSendObjectOverrideMethodTest.java
View File

@ -106,9 +106,10 @@ class RmiSendObjectOverrideMethodTest extends BaseTest {
final boolean isUDP = configuration.udpPort > 0;
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
configuration.serialization.registerRmi(TestObject.class, TestObjectImpl.class);
configuration.serialization.registerRmi(OtherObject.class, OtherObjectImpl.class);
configuration.serialization.register(OtherObjectImpl.class); // registered because this class is sent over the wire
Server server = new Server(configuration);
server.setIdleTimeout(0);
@ -138,9 +139,12 @@ class RmiSendObjectOverrideMethodTest extends BaseTest {
configuration = new Configuration();
config.apply(configuration);
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
configuration.serialization.registerRmi(TestObject.class, TestObjectImpl.class);
configuration.serialization.registerRmi(OtherObject.class, OtherObjectImpl.class);
configuration.serialization.register(OtherObjectImpl.class); // registered because this class is sent over the wire
Client client = new Client(configuration);
client.setIdleTimeout(0);

7
test/dorkbox/network/rmi/RmiSendObjectTest.java
View File

@ -90,9 +90,11 @@ class RmiSendObjectTest extends BaseTest {
Configuration configuration = new Configuration();
config.apply(configuration);
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
configuration.serialization.registerRmi(TestObject.class, TestObjectImpl.class);
configuration.serialization.registerRmi(OtherObject.class, OtherObjectImpl.class);
configuration.serialization.register(OtherObjectImpl.class); // registered because this class is sent over the wire
@ -124,9 +126,10 @@ class RmiSendObjectTest extends BaseTest {
configuration = new Configuration();
config.apply(configuration);
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
configuration.serialization.registerRmi(TestObject.class, TestObjectImpl.class);
configuration.serialization.registerRmi(OtherObject.class, OtherObjectImpl.class);
configuration.serialization.register(OtherObjectImpl.class); // registered because this class is sent over the wire
Client client = new Client(configuration);

4
test/dorkbox/network/rmi/RmiTest.java
View File

@ -218,7 +218,7 @@ class RmiTest extends BaseTest {
Configuration configuration = new Configuration();
config.apply(configuration);
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
register(configuration.serialization);
// for Client -> Server RMI (ID 1)
@ -275,7 +275,7 @@ class RmiTest extends BaseTest {
configuration = new Configuration();
config.apply(configuration);
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
register(configuration.serialization);
// for Client -> Server RMI (ID 1)

4
test/dorkbox/network/rmi/multiJVM/TestClient.java
View File

@ -78,10 +78,8 @@ class TestClient
configuration.udpPort = 2001;
configuration.host = "localhost";
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
RmiTest.register(configuration.serialization);
configuration.serialization.registerRmi(TestCow.class);
try {
final Client client = new Client(configuration);

3
test/dorkbox/network/rmi/multiJVM/TestServer.java
View File

@ -21,8 +21,9 @@ class TestServer
configuration.tcpPort = 2000;
configuration.udpPort = 2001;
configuration.serialization = Serialization.DEFAULT(true, true, false, null);
configuration.serialization = Serialization.DEFAULT();
RmiTest.register(configuration.serialization);
configuration.serialization.registerRmi(TestCow.class, TestCowImpl.class);
Server server = null;