Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Make sure registrations are closed
[controller.git]
/
opendaylight
/
md-sal
/
sal-remoterpc-connector
/
src
/
test
/
java
/
org
/
opendaylight
/
controller
/
remote
/
rpc
/
OpsRegistrarTest.java
diff --git
a/opendaylight/md-sal/sal-remoterpc-connector/src/test/java/org/opendaylight/controller/remote/rpc/OpsRegistrarTest.java
b/opendaylight/md-sal/sal-remoterpc-connector/src/test/java/org/opendaylight/controller/remote/rpc/OpsRegistrarTest.java
index 59db730d0152be3289361960bd3da96a98e1ccc8..1bbb22ed7a02874a9bd1acf2e893736ff4ee7d01 100644
(file)
--- a/
opendaylight/md-sal/sal-remoterpc-connector/src/test/java/org/opendaylight/controller/remote/rpc/OpsRegistrarTest.java
+++ b/
opendaylight/md-sal/sal-remoterpc-connector/src/test/java/org/opendaylight/controller/remote/rpc/OpsRegistrarTest.java
@@
-154,6
+154,8
@@
public class OpsRegistrarTest {
inOrder.verify(rpcService).registerRpcImplementation(any(RemoteRpcImplementation.class),
eq(secondEndpoint.getRpcs()));
inOrder.verify(rpcService).registerRpcImplementation(any(RemoteRpcImplementation.class),
eq(secondEndpoint.getRpcs()));
+ // verify first registration is closed
+ inOrder.verify(oldReg).close();
verifyNoMoreInteractions(rpcService, oldReg, newReg);
}
verifyNoMoreInteractions(rpcService, oldReg, newReg);
}
@@
-175,7
+177,7
@@
public class OpsRegistrarTest {
eq(secondActionEndpoint.getActions()));
// verify first registration is closed
eq(secondActionEndpoint.getActions()));
// verify first registration is closed
-
// inOrder.verify(old
Reg).close();
+
inOrder.verify(oldAction
Reg).close();
verifyNoMoreInteractions(actionService, oldActionReg, newActionReg);
}
verifyNoMoreInteractions(actionService, oldActionReg, newActionReg);
}