diff options
Diffstat (limited to 'kamon-spray/src/test/scala/kamon/spray/ServerRequestInstrumentationSpec.scala')
-rw-r--r-- | kamon-spray/src/test/scala/kamon/spray/ServerRequestInstrumentationSpec.scala | 106 |
1 files changed, 84 insertions, 22 deletions
diff --git a/kamon-spray/src/test/scala/kamon/spray/ServerRequestInstrumentationSpec.scala b/kamon-spray/src/test/scala/kamon/spray/ServerRequestInstrumentationSpec.scala index 1c5b56cd..7edbbe11 100644 --- a/kamon-spray/src/test/scala/kamon/spray/ServerRequestInstrumentationSpec.scala +++ b/kamon-spray/src/test/scala/kamon/spray/ServerRequestInstrumentationSpec.scala @@ -1,22 +1,23 @@ -/* =================================================== +/* + * ========================================================================================= * 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 + * 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 + * 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. - * ========================================================== */ + * 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.spray import _root_.spray.httpx.RequestBuilding -import akka.testkit.{ TestProbe, TestKit } +import akka.testkit.{ TestKitBase, TestProbe } import akka.actor.ActorSystem import org.scalatest.{ Matchers, WordSpecLike } import kamon.Kamon @@ -25,25 +26,58 @@ import spray.http.HttpHeaders.RawHeader import spray.http.{ HttpResponse, HttpRequest } import kamon.metrics.{ TraceMetrics, Metrics } import kamon.metrics.Subscriptions.TickMetricSnapshot +import com.typesafe.config.ConfigFactory +import kamon.metrics.TraceMetrics.ElapsedTime -class ServerRequestInstrumentationSpec extends TestKit(ActorSystem("spec")) with WordSpecLike with Matchers with RequestBuilding +class ServerRequestInstrumentationSpec extends TestKitBase with WordSpecLike with Matchers with RequestBuilding with ScalaFutures with PatienceConfiguration with TestServer { + implicit lazy val system: ActorSystem = ActorSystem("client-pipelining-segment-strategy-instrumentation-spec", ConfigFactory.parseString( + """ + |akka { + | loglevel = ERROR + |} + | + |kamon { + | metrics { + | tick-interval = 2 seconds + | + | filters = [ + | { + | trace { + | includes = [ "*" ] + | excludes = [] + | } + | } + | ] + | } + | + | spray { + | client { + | segment-collection-strategy = internal + | } + | } + |} + """.stripMargin)) + "the spray server request tracing instrumentation" should { - "reply back with the same trace token header provided in the request" in { + "include the trace-token header in responses when the automatic-trace-token-propagation is enabled" in { + enableAutomaticTraceTokenPropagation() + val (connection, server) = buildClientConnectionAndServer val client = TestProbe() - client.send(connection, Get("/").withHeaders(RawHeader("X-Trace-Token", "reply-trace-token"))) + client.send(connection, Get("/").withHeaders(RawHeader(Kamon(Spray).traceTokenHeaderName, "propagation-enabled"))) server.expectMsgType[HttpRequest] server.reply(HttpResponse(entity = "ok")) val response = client.expectMsgType[HttpResponse] - response.headers should contain(RawHeader("X-Trace-Token", "reply-trace-token")) - + response.headers should contain(RawHeader(Kamon(Spray).traceTokenHeaderName, "propagation-enabled")) } - "reply back with a automatically assigned trace token if none was provided with the request" in { + "reply back with an automatically assigned trace token if none was provided with the request and automatic-trace-token-propagation is enabled" in { + enableAutomaticTraceTokenPropagation() + val (connection, server) = buildClientConnectionAndServer val client = TestProbe() @@ -52,28 +86,56 @@ class ServerRequestInstrumentationSpec extends TestKit(ActorSystem("spec")) with server.reply(HttpResponse(entity = "ok")) val response = client.expectMsgType[HttpResponse] - response.headers.filter(_.name == "X-Trace-Token").size should be(1) + response.headers.filter(_.name == Kamon(Spray).traceTokenHeaderName).size should be(1) } + "not include the trace-token header in responses when the automatic-trace-token-propagation is disabled" in { + disableAutomaticTraceTokenPropagation() + + val (connection, server) = buildClientConnectionAndServer + val client = TestProbe() + + client.send(connection, Get("/").withHeaders(RawHeader(Kamon(Spray).traceTokenHeaderName, "propagation-disabled"))) + server.expectMsgType[HttpRequest] + server.reply(HttpResponse(entity = "ok")) + val response = client.expectMsgType[HttpResponse] + + response.headers should not contain (RawHeader(Kamon(Spray).traceTokenHeaderName, "propagation-disabled")) + } + "open and finish a trace during the lifetime of a request" in { val (connection, server) = buildClientConnectionAndServer val client = TestProbe() val metricListener = TestProbe() Kamon(Metrics)(system).subscribe(TraceMetrics, "*", metricListener.ref, permanently = true) + metricListener.expectMsgType[TickMetricSnapshot] client.send(connection, Get("/open-and-finish")) server.expectMsgType[HttpRequest] server.reply(HttpResponse(entity = "ok")) client.expectMsgType[HttpResponse] - metricListener.fishForMessage() { - case snapshot @ TickMetricSnapshot(_, _, metrics) ⇒ metrics.keys.exists(_.name.contains("open-and-finish")) - case other ⇒ false + val tickSnapshot = metricListener.expectMsgType[TickMetricSnapshot] + val traceMetrics = tickSnapshot.metrics.find { case (k, v) ⇒ k.name.contains("open-and-finish") } map (_._2.metrics) + traceMetrics should not be empty + + traceMetrics map { metrics ⇒ + metrics(ElapsedTime).numberOfMeasurements should be(1L) } } } + def enableAutomaticTraceTokenPropagation(): Unit = setIncludeTraceToken(true) + def disableAutomaticTraceTokenPropagation(): Unit = setIncludeTraceToken(false) + + def setIncludeTraceToken(include: Boolean): Unit = { + val target = Kamon(Spray)(system) + val field = target.getClass.getDeclaredField("includeTraceToken") + field.setAccessible(true) + field.set(target, include) + } + } |