From 4d0e37594cb223b091ba2ec126eabe89ac8c13f8 Mon Sep 17 00:00:00 2001
From: Ivan Topolnjak <ivantopo@gmail.com>
Date: Sun, 28 Dec 2014 07:59:59 +0100
Subject: ! core,akka: separate all akka instrumentation into it's own
 kamon-akka module, related to #136.

All Akka-related instrumentation and code has been moved to the kamon-akka module,
including the filters for actor, dispatcher and router metrics. Also the following
changes are included:

  - Router Metrics are now working properly, related to #139.
  - Cleanup the log output for this module, related to #142.
  - Some minor cleanups in various tests.

This PR breaks the reporting modules which will need to wait for #141 to be ready to come back
to life.
---
 .../akka/ActorCellInstrumentationSpec.scala        | 125 +++++++++++++++++++++
 1 file changed, 125 insertions(+)
 create mode 100644 kamon-akka/src/test/scala/kamon/instrumentation/akka/ActorCellInstrumentationSpec.scala

(limited to 'kamon-akka/src/test/scala/kamon/instrumentation/akka/ActorCellInstrumentationSpec.scala')

diff --git a/kamon-akka/src/test/scala/kamon/instrumentation/akka/ActorCellInstrumentationSpec.scala b/kamon-akka/src/test/scala/kamon/instrumentation/akka/ActorCellInstrumentationSpec.scala
new file mode 100644
index 00000000..06a232bd
--- /dev/null
+++ b/kamon-akka/src/test/scala/kamon/instrumentation/akka/ActorCellInstrumentationSpec.scala
@@ -0,0 +1,125 @@
+/* ===================================================
+ * Copyright © 2013 the kamon project <http://kamon.io/>
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ========================================================== */
+package kamon.instrumentation.akka
+
+import akka.actor.{ Actor, ActorSystem, Props }
+import akka.pattern.{ ask, pipe }
+import akka.routing._
+import akka.testkit.{ TestKitBase, ImplicitSender, TestKit }
+import akka.util.Timeout
+import com.typesafe.config.ConfigFactory
+import kamon.trace.TraceRecorder
+import org.scalatest.{ BeforeAndAfterAll, WordSpecLike }
+
+import scala.concurrent.duration._
+
+class ActorCellInstrumentationSpec extends TestKitBase with WordSpecLike with ImplicitSender with BeforeAndAfterAll {
+  implicit lazy val system: ActorSystem = ActorSystem("actor-cell-instrumentation-spec")
+  implicit val executionContext = system.dispatcher
+
+  "the message passing instrumentation" should {
+    "propagate the TraceContext using bang" in new EchoActorFixture {
+      val testTraceContext = TraceRecorder.withNewTraceContext("bang-reply") {
+        ctxEchoActor ! "test"
+        TraceRecorder.currentContext
+      }
+
+      expectMsg(testTraceContext)
+    }
+
+    "propagate the TraceContext using tell" in new EchoActorFixture {
+      val testTraceContext = TraceRecorder.withNewTraceContext("tell-reply") {
+        ctxEchoActor.tell("test", testActor)
+        TraceRecorder.currentContext
+      }
+
+      expectMsg(testTraceContext)
+    }
+
+    "propagate the TraceContext using ask" in new EchoActorFixture {
+      implicit val timeout = Timeout(1 seconds)
+      val testTraceContext = TraceRecorder.withNewTraceContext("ask-reply") {
+        // The pipe pattern use Futures internally, so FutureTracing test should cover the underpinnings of it.
+        (ctxEchoActor ? "test") pipeTo (testActor)
+        TraceRecorder.currentContext
+      }
+
+      expectMsg(testTraceContext)
+    }
+
+    "propagate the TraceContext to actors behind a simple router" in new EchoSimpleRouterFixture {
+      val testTraceContext = TraceRecorder.withNewTraceContext("router-reply") {
+        router.route("test", testActor)
+        TraceRecorder.currentContext
+      }
+
+      expectMsg(testTraceContext)
+    }
+
+    "propagate the TraceContext to actors behind a pool router" in new EchoPoolRouterFixture {
+      val testTraceContext = TraceRecorder.withNewTraceContext("router-reply") {
+        pool ! "test"
+        TraceRecorder.currentContext
+      }
+
+      expectMsg(testTraceContext)
+    }
+
+    "propagate the TraceContext to actors behind a group router" in new EchoGroupRouterFixture {
+      val testTraceContext = TraceRecorder.withNewTraceContext("router-reply") {
+        group ! "test"
+        TraceRecorder.currentContext
+      }
+
+      expectMsg(testTraceContext)
+    }
+  }
+
+  override protected def afterAll(): Unit = shutdown()
+
+  trait EchoActorFixture {
+    val ctxEchoActor = system.actorOf(Props[TraceContextEcho])
+  }
+
+  trait EchoSimpleRouterFixture {
+    val router = {
+      val routees = Vector.fill(5) {
+        val r = system.actorOf(Props[TraceContextEcho])
+        ActorRefRoutee(r)
+      }
+      Router(RoundRobinRoutingLogic(), routees)
+    }
+  }
+
+  trait EchoPoolRouterFixture {
+    val pool = system.actorOf(RoundRobinPool(nrOfInstances = 5).props(Props[TraceContextEcho]), "pool-router")
+  }
+
+  trait EchoGroupRouterFixture {
+    val routees = Vector.fill(5) {
+      system.actorOf(Props[TraceContextEcho])
+    }
+
+    val group = system.actorOf(RoundRobinGroup(routees.map(_.path.toStringWithoutAddress)).props(), "group-router")
+  }
+}
+
+class TraceContextEcho extends Actor {
+  def receive = {
+    case msg: String ⇒ sender ! TraceRecorder.currentContext
+  }
+}
+
-- 
cgit v1.2.3