Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Merge "added feature topology manager shell"
[controller.git]
/
opendaylight
/
md-sal
/
sal-distributed-datastore
/
src
/
test
/
java
/
org
/
opendaylight
/
controller
/
cluster
/
datastore
/
ShardTransactionTest.java
diff --git
a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ShardTransactionTest.java
b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ShardTransactionTest.java
index 2d9ae93d9ee1d044b2809996b88d1183c16672f8..7884eeccdae7874f8d19c9090bf08d755cc991be 100644
(file)
--- a/
opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ShardTransactionTest.java
+++ b/
opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ShardTransactionTest.java
@@
-26,8
+26,11
@@
import org.opendaylight.controller.cluster.datastore.modification.Modification;
import org.opendaylight.controller.cluster.datastore.modification.WriteModification;
import org.opendaylight.controller.md.cluster.datastore.model.TestModel;
import org.opendaylight.controller.md.sal.dom.store.impl.InMemoryDOMDataStore;
import org.opendaylight.controller.cluster.datastore.modification.WriteModification;
import org.opendaylight.controller.md.cluster.datastore.model.TestModel;
import org.opendaylight.controller.md.sal.dom.store.impl.InMemoryDOMDataStore;
-import org.opendaylight.yangtools.yang.data.api.InstanceIdentifier;
+import org.opendaylight.yangtools.yang.data.api.
Yang
InstanceIdentifier;
import org.opendaylight.yangtools.yang.data.impl.schema.ImmutableNodes;
import org.opendaylight.yangtools.yang.data.impl.schema.ImmutableNodes;
+import org.opendaylight.yangtools.yang.model.api.SchemaContext;
+
+import java.util.Collections;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
@@
-39,31
+42,33
@@
public class ShardTransactionTest extends AbstractActorTest {
private static final InMemoryDOMDataStore store =
new InMemoryDOMDataStore("OPER", storeExecutor);
private static final InMemoryDOMDataStore store =
new InMemoryDOMDataStore("OPER", storeExecutor);
+ private static final SchemaContext testSchemaContext = TestModel.createTestContext();
+
static {
static {
- store.onGlobalContextUpdated(
TestModel.createTestContext()
);
+ store.onGlobalContextUpdated(
testSchemaContext
);
}
@Test
public void testOnReceiveReadData() throws Exception {
new JavaTestKit(getSystem()) {{
}
@Test
public void testOnReceiveReadData() throws Exception {
new JavaTestKit(getSystem()) {{
- final ActorRef shard = getSystem().actorOf(Shard.props("config"));
+ final ActorRef shard = getSystem().actorOf(Shard.props("config"
, Collections.EMPTY_MAP
));
final Props props =
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, testSchemaContext
);
final ActorRef subject = getSystem().actorOf(props, "testReadData");
new Within(duration("1 seconds")) {
protected void run() {
subject.tell(
final ActorRef subject = getSystem().actorOf(props, "testReadData");
new Within(duration("1 seconds")) {
protected void run() {
subject.tell(
- new ReadData(
InstanceIdentifier.builder().build()
),
+ new ReadData(
YangInstanceIdentifier.builder().build()).toSerializable(
),
getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
- if (in
instanceof ReadDataReply
) {
-
if (((ReadDataReply) in).getNormalizedNode(
)
-
!= null) {
+ if (in
.getClass().equals(ReadDataReply.SERIALIZABLE_CLASS)
) {
+
if (ReadDataReply.fromSerializable(testSchemaContext,YangInstanceIdentifier.builder().build(), in
)
+
.getNormalizedNode()
!= null) {
return "match";
}
return null;
return "match";
}
return null;
@@
-86,23
+91,24
@@
public class ShardTransactionTest extends AbstractActorTest {
@Test
public void testOnReceiveReadDataWhenDataNotFound() throws Exception {
new JavaTestKit(getSystem()) {{
@Test
public void testOnReceiveReadDataWhenDataNotFound() throws Exception {
new JavaTestKit(getSystem()) {{
- final ActorRef shard = getSystem().actorOf(Shard.props("config"));
+ final ActorRef shard = getSystem().actorOf(Shard.props("config"
, Collections.EMPTY_MAP
));
final Props props =
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, testSchemaContext
);
final ActorRef subject = getSystem().actorOf(props, "testReadDataWhenDataNotFound");
new Within(duration("1 seconds")) {
protected void run() {
subject.tell(
final ActorRef subject = getSystem().actorOf(props, "testReadDataWhenDataNotFound");
new Within(duration("1 seconds")) {
protected void run() {
subject.tell(
- new ReadData(TestModel.TEST_PATH),
+ new ReadData(TestModel.TEST_PATH)
.toSerializable()
,
getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
- if (in instanceof ReadDataReply) {
- if (((ReadDataReply) in).getNormalizedNode()
+ if (in.getClass().equals(ReadDataReply.SERIALIZABLE_CLASS)) {
+ if (ReadDataReply.fromSerializable(testSchemaContext,TestModel.TEST_PATH, in)
+ .getNormalizedNode()
== null) {
return "match";
}
== null) {
return "match";
}
@@
-159,9
+165,9
@@
public class ShardTransactionTest extends AbstractActorTest {
@Test
public void testOnReceiveWriteData() throws Exception {
new JavaTestKit(getSystem()) {{
@Test
public void testOnReceiveWriteData() throws Exception {
new JavaTestKit(getSystem()) {{
- final ActorRef shard = getSystem().actorOf(Shard.props("config"));
+ final ActorRef shard = getSystem().actorOf(Shard.props("config"
, Collections.EMPTY_MAP
));
final Props props =
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, TestModel.createTestContext()
);
final ActorRef subject =
getSystem().actorOf(props, "testWriteData");
final ActorRef subject =
getSystem().actorOf(props, "testWriteData");
@@
-169,13
+175,13
@@
public class ShardTransactionTest extends AbstractActorTest {
protected void run() {
subject.tell(new WriteData(TestModel.TEST_PATH,
protected void run() {
subject.tell(new WriteData(TestModel.TEST_PATH,
- ImmutableNodes.containerNode(TestModel.TEST_QNAME)),
+ ImmutableNodes.containerNode(TestModel.TEST_QNAME)
, TestModel.createTestContext()).toSerializable(
),
getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
- if (in
instanceof WriteDataReply
) {
+ if (in
.getClass().equals(WriteDataReply.SERIALIZABLE_CLASS)
) {
return "match";
} else {
throw noMatch();
return "match";
} else {
throw noMatch();
@@
-197,9
+203,9
@@
public class ShardTransactionTest extends AbstractActorTest {
@Test
public void testOnReceiveMergeData() throws Exception {
new JavaTestKit(getSystem()) {{
@Test
public void testOnReceiveMergeData() throws Exception {
new JavaTestKit(getSystem()) {{
- final ActorRef shard = getSystem().actorOf(Shard.props("config"));
+ final ActorRef shard = getSystem().actorOf(Shard.props("config"
, Collections.EMPTY_MAP
));
final Props props =
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, testSchemaContext
);
final ActorRef subject =
getSystem().actorOf(props, "testMergeData");
final ActorRef subject =
getSystem().actorOf(props, "testMergeData");
@@
-207,13
+213,13
@@
public class ShardTransactionTest extends AbstractActorTest {
protected void run() {
subject.tell(new MergeData(TestModel.TEST_PATH,
protected void run() {
subject.tell(new MergeData(TestModel.TEST_PATH,
- ImmutableNodes.containerNode(TestModel.TEST_QNAME)),
+ ImmutableNodes.containerNode(TestModel.TEST_QNAME)
, testSchemaContext).toSerializable(
),
getRef());
getRef());
- final String out = new ExpectMsg<String>("match hint") {
+ final String out = new ExpectMsg<String>(
duration("500 milliseconds"),
"match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
- if (in
instanceof MergeDataReply
) {
+ if (in
.getClass().equals(MergeDataReply.SERIALIZABLE_CLASS)
) {
return "match";
} else {
throw noMatch();
return "match";
} else {
throw noMatch();
@@
-236,21
+242,21
@@
public class ShardTransactionTest extends AbstractActorTest {
@Test
public void testOnReceiveDeleteData() throws Exception {
new JavaTestKit(getSystem()) {{
@Test
public void testOnReceiveDeleteData() throws Exception {
new JavaTestKit(getSystem()) {{
- final ActorRef shard = getSystem().actorOf(Shard.props("config"));
+ final ActorRef shard = getSystem().actorOf(Shard.props("config"
, Collections.EMPTY_MAP
));
final Props props =
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, TestModel.createTestContext()
);
final ActorRef subject =
getSystem().actorOf(props, "testDeleteData");
new Within(duration("1 seconds")) {
protected void run() {
final ActorRef subject =
getSystem().actorOf(props, "testDeleteData");
new Within(duration("1 seconds")) {
protected void run() {
- subject.tell(new DeleteData(TestModel.TEST_PATH), getRef());
+ subject.tell(new DeleteData(TestModel.TEST_PATH)
.toSerializable()
, getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
- if (in
instanceof DeleteDataReply
) {
+ if (in
.getClass().equals(DeleteDataReply.SERIALIZABLE_CLASS)
) {
return "match";
} else {
throw noMatch();
return "match";
} else {
throw noMatch();
@@
-273,21
+279,21
@@
public class ShardTransactionTest extends AbstractActorTest {
@Test
public void testOnReceiveReadyTransaction() throws Exception {
new JavaTestKit(getSystem()) {{
@Test
public void testOnReceiveReadyTransaction() throws Exception {
new JavaTestKit(getSystem()) {{
- final ActorRef shard = getSystem().actorOf(Shard.props("config"));
+ final ActorRef shard = getSystem().actorOf(Shard.props("config"
, Collections.EMPTY_MAP
));
final Props props =
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, TestModel.createTestContext()
);
final ActorRef subject =
getSystem().actorOf(props, "testReadyTransaction");
new Within(duration("1 seconds")) {
protected void run() {
final ActorRef subject =
getSystem().actorOf(props, "testReadyTransaction");
new Within(duration("1 seconds")) {
protected void run() {
- subject.tell(new ReadyTransaction(), getRef());
+ subject.tell(new ReadyTransaction()
.toSerializable()
, getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
- if (in
instanceof ReadyTransactionReply
) {
+ if (in
.getClass().equals(ReadyTransactionReply.SERIALIZABLE_CLASS)
) {
return "match";
} else {
throw noMatch();
return "match";
} else {
throw noMatch();
@@
-309,9
+315,9
@@
public class ShardTransactionTest extends AbstractActorTest {
@Test
public void testOnReceiveCloseTransaction() throws Exception {
new JavaTestKit(getSystem()) {{
@Test
public void testOnReceiveCloseTransaction() throws Exception {
new JavaTestKit(getSystem()) {{
- final ActorRef shard = getSystem().actorOf(Shard.props("config"));
+ final ActorRef shard = getSystem().actorOf(Shard.props("config"
, Collections.EMPTY_MAP
));
final Props props =
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, TestModel.createTestContext()
);
final ActorRef subject =
getSystem().actorOf(props, "testCloseTransaction");
final ActorRef subject =
getSystem().actorOf(props, "testCloseTransaction");
@@
-320,12
+326,12
@@
public class ShardTransactionTest extends AbstractActorTest {
new Within(duration("2 seconds")) {
protected void run() {
new Within(duration("2 seconds")) {
protected void run() {
- subject.tell(new CloseTransaction(), getRef());
+ subject.tell(new CloseTransaction()
.toSerializable()
, getRef());
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
- if (in
instanceof CloseTransactionReply
) {
+ if (in
.getClass().equals(CloseTransactionReply.SERIALIZABLE_CLASS)
) {
return "match";
} else {
throw noMatch();
return "match";
} else {
throw noMatch();