only create the routees once they can be looked-up, see #3406

- move the creation of the RoutedActorCell’s route into
  ActorCell.start(); it used to be done in the constructor
- this requires “val route” to turn into a volatile private var

Thanks to Patrik for finding it!
This commit is contained in:
Roland 2013-05-30 12:44:08 +02:00
parent 9c89f170d2
commit 0af123aa6d
3 changed files with 65 additions and 4 deletions

View file

@ -0,0 +1,51 @@
/**
* Copyright (C) 2009-2013 Typesafe Inc. <http://www.typesafe.com>
*/
package akka.routing
import akka.testkit.AkkaSpec
import akka.actor.Props
import akka.actor.Actor
import akka.actor.ActorRef
import akka.actor.LocalActorRef
import scala.concurrent.duration._
class RouteeCreationSpec extends AkkaSpec {
"Creating Routees" must {
"result in visible routees" in {
val N = 100
system.actorOf(Props(new Actor {
testActor ! system.actorFor(self.path)
def receive = Actor.emptyBehavior
}).withRouter(RoundRobinRouter(N)))
for (i 1 to N) {
expectMsgType[ActorRef] match {
case _: LocalActorRef // fine
case x fail(s"routee $i was a ${x.getClass}")
}
}
}
"allow sending to context.parent" in {
val N = 100
system.actorOf(Props(new Actor {
context.parent ! "one"
def receive = {
case "one" testActor forward "two"
}
}).withRouter(RoundRobinRouter(N)))
val gotit = receiveWhile(messages = N) {
case "two" lastSender.toString
}
expectNoMsg(100.millis)
if (gotit.size != N) {
fail(s"got only ${gotit.size} from \n${gotit mkString "\n"}")
}
}
}
}