Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Initial support for multiple-shards
[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 e4d8e1b23a4acf354e7fc68ab9849c0e46d8549e..68cee1fcaccd91c51c18f6cfab65e45da494bc39 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
@@
-28,6
+28,7
@@
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.impl.schema.ImmutableNodes;
import org.opendaylight.controller.md.sal.dom.store.impl.InMemoryDOMDataStore;
import org.opendaylight.yangtools.yang.data.api.InstanceIdentifier;
import org.opendaylight.yangtools.yang.data.impl.schema.ImmutableNodes;
+import org.opendaylight.yangtools.yang.model.api.SchemaContext;
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,8
+40,10
@@
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
}
@Test
@@
-48,22
+51,22
@@
public class ShardTransactionTest extends AbstractActorTest {
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
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(InstanceIdentifier.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,InstanceIdentifier.builder().build(), in
)
+
.getNormalizedNode()
!= null) {
return "match";
}
return null;
return "match";
}
return null;
@@
-88,21
+91,22
@@
public class ShardTransactionTest extends AbstractActorTest {
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
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";
}
@@
-161,7
+165,7
@@
public class ShardTransactionTest extends AbstractActorTest {
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
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,7
+173,7
@@
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") {
getRef());
final String out = new ExpectMsg<String>("match hint") {
@@
-199,7
+203,7
@@
public class ShardTransactionTest extends AbstractActorTest {
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
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,10
+211,10
@@
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) {
if (in instanceof MergeDataReply) {
// do not put code outside this method, will run afterwards
protected String match(Object in) {
if (in instanceof MergeDataReply) {
@@
-238,14
+242,14
@@
public class ShardTransactionTest extends AbstractActorTest {
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
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
final String out = new ExpectMsg<String>("match hint") {
// do not put code outside this method, will run afterwards
@@
-275,7
+279,7
@@
public class ShardTransactionTest extends AbstractActorTest {
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
- ShardTransaction.props(store.newReadWriteTransaction(), shard);
+ ShardTransaction.props(store.newReadWriteTransaction(), shard
, TestModel.createTestContext()
);
final ActorRef subject =
getSystem().actorOf(props, "testReadyTransaction");
final ActorRef subject =
getSystem().actorOf(props, "testReadyTransaction");
@@
-311,7
+315,7
@@
public class ShardTransactionTest extends AbstractActorTest {
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
final Props props =
new JavaTestKit(getSystem()) {{
final ActorRef shard = getSystem().actorOf(Shard.props("config"));
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");
@@
-355,6
+359,4
@@
public class ShardTransactionTest extends AbstractActorTest {
}};
}
}};
}
-
-
}
}