Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Do not duplicate OSGi dependencyManagement
[controller.git]
/
opendaylight
/
md-sal
/
sal-akka-raft
/
src
/
test
/
java
/
org
/
opendaylight
/
controller
/
cluster
/
raft
/
behaviors
/
CandidateTest.java
diff --git
a/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/CandidateTest.java
b/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/CandidateTest.java
index c76368370506af15ab4b7b567672514514e41690..0dc68c2461c2235b22e663b39ad51220e96c80b5 100644
(file)
--- a/
opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/CandidateTest.java
+++ b/
opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/CandidateTest.java
@@
-3,12
+3,16
@@
package org.opendaylight.controller.cluster.raft.behaviors;
import akka.actor.ActorRef;
import akka.actor.Props;
import akka.testkit.JavaTestKit;
import akka.actor.ActorRef;
import akka.actor.Props;
import akka.testkit.JavaTestKit;
-import junit.framework.Assert;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.Map;
+import org.junit.Assert;
import org.junit.Before;
import org.junit.Test;
import org.junit.Before;
import org.junit.Test;
+import org.opendaylight.controller.cluster.raft.DefaultConfigParamsImpl;
import org.opendaylight.controller.cluster.raft.MockRaftActorContext;
import org.opendaylight.controller.cluster.raft.RaftActorContext;
import org.opendaylight.controller.cluster.raft.MockRaftActorContext;
import org.opendaylight.controller.cluster.raft.RaftActorContext;
-import org.opendaylight.controller.cluster.raft.R
aftState
;
+import org.opendaylight.controller.cluster.raft.R
eplicatedLogEntry
;
import org.opendaylight.controller.cluster.raft.base.messages.ElectionTimeout;
import org.opendaylight.controller.cluster.raft.messages.AppendEntries;
import org.opendaylight.controller.cluster.raft.messages.AppendEntriesReply;
import org.opendaylight.controller.cluster.raft.base.messages.ElectionTimeout;
import org.opendaylight.controller.cluster.raft.messages.AppendEntries;
import org.opendaylight.controller.cluster.raft.messages.AppendEntriesReply;
@@
-16,10
+20,6
@@
import org.opendaylight.controller.cluster.raft.messages.RequestVote;
import org.opendaylight.controller.cluster.raft.messages.RequestVoteReply;
import org.opendaylight.controller.cluster.raft.utils.DoNothingActor;
import org.opendaylight.controller.cluster.raft.messages.RequestVoteReply;
import org.opendaylight.controller.cluster.raft.utils.DoNothingActor;
-import java.util.Collections;
-import java.util.HashMap;
-import java.util.Map;
-
import static org.junit.Assert.assertEquals;
public class CandidateTest extends AbstractRaftActorBehaviorTest {
import static org.junit.Assert.assertEquals;
public class CandidateTest extends AbstractRaftActorBehaviorTest {
@@
-80,12
+80,12
@@
public class CandidateTest extends AbstractRaftActorBehaviorTest {
public void testThatAnElectionTimeoutIsTriggered(){
new JavaTestKit(getSystem()) {{
public void testThatAnElectionTimeoutIsTriggered(){
new JavaTestKit(getSystem()) {{
- new Within(
duration("1 seconds"
)) {
+ new Within(
DefaultConfigParamsImpl.HEART_BEAT_INTERVAL.$times(6
)) {
protected void run() {
Candidate candidate = new Candidate(createActorContext(getTestActor()));
protected void run() {
Candidate candidate = new Candidate(createActorContext(getTestActor()));
- final Boolean out = new ExpectMsg<Boolean>(
duration("1 seconds"
), "ElectionTimeout") {
+ final Boolean out = new ExpectMsg<Boolean>(
DefaultConfigParamsImpl.HEART_BEAT_INTERVAL.$times(6
), "ElectionTimeout") {
// do not put code outside this method, will run afterwards
protected Boolean match(Object in) {
if (in instanceof ElectionTimeout) {
// do not put code outside this method, will run afterwards
protected Boolean match(Object in) {
if (in instanceof ElectionTimeout) {
@@
-108,10
+108,10
@@
public class CandidateTest extends AbstractRaftActorBehaviorTest {
Candidate candidate =
new Candidate(raftActorContext);
Candidate candidate =
new Candidate(raftActorContext);
- Raft
State raftState
=
+ Raft
ActorBehavior raftBehavior
=
candidate.handleMessage(candidateActor, new ElectionTimeout());
candidate.handleMessage(candidateActor, new ElectionTimeout());
- Assert.assert
Equals(RaftState.Leader, raftState
);
+ Assert.assert
True(raftBehavior instanceof Leader
);
}
@Test
}
@Test
@@
-122,10
+122,10
@@
public class CandidateTest extends AbstractRaftActorBehaviorTest {
Candidate candidate =
new Candidate(raftActorContext);
Candidate candidate =
new Candidate(raftActorContext);
- Raft
State raftState
=
+ Raft
ActorBehavior raftBehavior
=
candidate.handleMessage(candidateActor, new ElectionTimeout());
candidate.handleMessage(candidateActor, new ElectionTimeout());
- Assert.assert
Equals(RaftState.Candidate, raftSt
ate);
+ Assert.assert
True(raftBehavior instanceof Candid
ate);
}
@Test
}
@Test
@@
-136,9
+136,9
@@
public class CandidateTest extends AbstractRaftActorBehaviorTest {
Candidate candidate =
new Candidate(raftActorContext);
Candidate candidate =
new Candidate(raftActorContext);
- Raft
State state
OnFirstVote = candidate.handleMessage(peerActor1, new RequestVoteReply(0, true));
+ Raft
ActorBehavior behavior
OnFirstVote = candidate.handleMessage(peerActor1, new RequestVoteReply(0, true));
- Assert.assert
Equals(RaftState.Leader, stateOnFirstVote
);
+ Assert.assert
True(behaviorOnFirstVote instanceof Leader
);
}
}
@@
-150,12
+150,12
@@
public class CandidateTest extends AbstractRaftActorBehaviorTest {
Candidate candidate =
new Candidate(raftActorContext);
Candidate candidate =
new Candidate(raftActorContext);
- Raft
State state
OnFirstVote = candidate.handleMessage(peerActor1, new RequestVoteReply(0, true));
+ Raft
ActorBehavior behavior
OnFirstVote = candidate.handleMessage(peerActor1, new RequestVoteReply(0, true));
- Raft
State state
OnSecondVote = candidate.handleMessage(peerActor2, new RequestVoteReply(0, true));
+ Raft
ActorBehavior behavior
OnSecondVote = candidate.handleMessage(peerActor2, new RequestVoteReply(0, true));
- Assert.assert
Equals(RaftState.Candidate, stateOnFirstVo
te);
- Assert.assert
Equals(RaftState.Leader, stateOnSecondVote
);
+ Assert.assert
True(behaviorOnFirstVote instanceof Candida
te);
+ Assert.assert
True(behaviorOnSecondVote instanceof Leader
);
}
}
@@
-168,7
+168,7
@@
public class CandidateTest extends AbstractRaftActorBehaviorTest {
Candidate candidate = new Candidate(createActorContext(getTestActor()));
Candidate candidate = new Candidate(createActorContext(getTestActor()));
- candidate.handleMessage(getTestActor(), new AppendEntries(0, "test", 0,0,Collections.
EMPTY_LIST, 0
));
+ candidate.handleMessage(getTestActor(), new AppendEntries(0, "test", 0,0,Collections.
<ReplicatedLogEntry>emptyList(), 0, -1
));
final Boolean out = new ExpectMsg<Boolean>(duration("1 seconds"), "AppendEntriesResponse") {
// do not put code outside this method, will run afterwards
final Boolean out = new ExpectMsg<Boolean>(duration("1 seconds"), "AppendEntriesResponse") {
// do not put code outside this method, will run afterwards