From 0ad117a9255ccdbf26ce30222508070545579056 Mon Sep 17 00:00:00 2001 From: Diego Date: Thu, 8 Jun 2017 23:27:09 -0300 Subject: * Introduce Java 8 Unsafe intrinsics in LogAdder (lock addq and lock xchg) * Introduce atomic variant of LongAdder::sumThenReset -> LongAdder::sumAndReset * Remove LongMaxUpdater in favor to AtomicLongMaxUpdater --- .../kamon/metric/instrument/CounterSpec.scala | 94 ++++++++++++---------- .../metric/instrument/MinMaxCounterSpec.scala | 90 +++++++++++++++++++++ 2 files changed, 140 insertions(+), 44 deletions(-) create mode 100644 kamon-core/src/test/scala/kamon/metric/instrument/MinMaxCounterSpec.scala (limited to 'kamon-core/src/test/scala/kamon/metric/instrument') diff --git a/kamon-core/src/test/scala/kamon/metric/instrument/CounterSpec.scala b/kamon-core/src/test/scala/kamon/metric/instrument/CounterSpec.scala index d76b3613..b4e3fe96 100644 --- a/kamon-core/src/test/scala/kamon/metric/instrument/CounterSpec.scala +++ b/kamon-core/src/test/scala/kamon/metric/instrument/CounterSpec.scala @@ -1,46 +1,52 @@ +/* ========================================================================================= + * Copyright © 2013-2017 the kamon project + * + * 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.metric.instrument -//import kamon.LogInterceptor -//import kamon.metric.Entity -//import kamon.testkit.DefaultInstrumentFactory -//import org.scalatest.{Matchers, WordSpec} -//import uk.org.lidalia.slf4jext.Level -//import uk.org.lidalia.slf4jtest.TestLoggerFactory -// -//class CounterSpec extends WordSpec with Matchers with LogInterceptor with DefaultInstrumentFactory { -// implicit val testLogger = TestLoggerFactory.getTestLogger(classOf[LongAdderCounter]) -//// -//// "a Counter" should { -//// -//// "allow unit and bundled increments" in { -//// val counter = buildCounter("unit-increments") -//// counter.increment() -//// counter.increment() -//// counter.increment(40) -//// -//// counter.snapshot().value shouldBe(42) -//// } -//// -//// "warn the user and ignore attempts to decrement the counter" in { -//// val counter = buildCounter("attempt-to-decrement") -//// counter.increment(100) -//// counter.increment(100) -//// counter.increment(100) -//// -//// interceptLog(Level.WARN) { -//// counter.increment(-10L) -//// }.head.getMessage() shouldBe(s"Ignored attempt to decrement counter [attempt-to-decrement] on entity [$defaultEntity]") -//// -//// counter.snapshot().value shouldBe(300) -//// } -//// -//// "reset the internal state to zero after taking snapshots" in { -//// val counter = buildCounter("reset-after-snapshot") -//// counter.increment() -//// counter.increment(10) -//// -//// counter.snapshot().value shouldBe(11) -//// counter.snapshot().value shouldBe(0) -//// } -//// } -//} +import kamon.testkit.DefaultInstrumentFactory +import org.scalatest.{Matchers, WordSpec} + +class CounterSpec extends WordSpec with Matchers with DefaultInstrumentFactory { + + "a Counter" should { + + "allow unit and bundled increments" in { + val counter = buildCounter("unit-increments") + counter.increment() + counter.increment() + counter.increment(40) + + counter.snapshot().value shouldBe 42 + } + + "warn the user and ignore attempts to decrement the counter" in { + val counter = buildCounter("attempt-to-decrement") + counter.increment(100) + counter.increment(100) + counter.increment(100) + + counter.snapshot().value shouldBe 300 + } + + "reset the internal state to zero after taking snapshots" in { + val counter = buildCounter("reset-after-snapshot") + counter.increment() + counter.increment(10) + + counter.snapshot().value shouldBe 11 + counter.snapshot().value shouldBe 0 + } + } +} diff --git a/kamon-core/src/test/scala/kamon/metric/instrument/MinMaxCounterSpec.scala b/kamon-core/src/test/scala/kamon/metric/instrument/MinMaxCounterSpec.scala new file mode 100644 index 00000000..3de2f385 --- /dev/null +++ b/kamon-core/src/test/scala/kamon/metric/instrument/MinMaxCounterSpec.scala @@ -0,0 +1,90 @@ +/* ========================================================================================= + * Copyright © 2013-2017 the kamon project + * + * 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.metric.instrument + +import kamon.metric.Bucket +import kamon.testkit.DefaultInstrumentFactory +import org.scalatest.{Matchers, WordSpec} + +case class TemporalBucket(value: Long, frequency: Long) extends Bucket + +class MinMaxCounterSpec extends WordSpec with Matchers with DefaultInstrumentFactory { + + "a MinMaxCounter" should { + + "track ascending tendencies" in { + val mmCounter = buildMinMaxCounter("track-ascending") + mmCounter.increment() + mmCounter.increment(3) + mmCounter.increment() + + mmCounter.sample() + + val snapshot = mmCounter.snapshot() + + snapshot.distribution.min should be(0) + snapshot.distribution.max should be(5) + } + + "track descending tendencies" in { + val mmCounter = buildMinMaxCounter("track-descending") + mmCounter.increment(5) + mmCounter.decrement() + mmCounter.decrement(3) + mmCounter.decrement() + + mmCounter.sample() + + val snapshot = mmCounter.snapshot() + + snapshot.distribution.min should be(0) + snapshot.distribution.max should be(5) + } + + "reset the min and max to the current value after taking a snapshot" in { + val mmCounter = buildMinMaxCounter("reset-min-max-to-current") + + mmCounter.increment(5) + mmCounter.decrement(3) + + mmCounter.sample() + + val firstSnapshot = mmCounter.snapshot() + + firstSnapshot.distribution.min should be(0) + firstSnapshot.distribution.max should be(5) + + mmCounter.sample() + + val secondSnapshot = mmCounter.snapshot() + + secondSnapshot.distribution.min should be(2) + secondSnapshot.distribution.max should be(2) + } + + "report zero as the min and current values if the current value fell bellow zero" in { + val mmCounter = buildMinMaxCounter("report-zero") + + mmCounter.decrement(3) + + val snapshot = mmCounter.snapshot() + + snapshot.distribution.min should be(0) + snapshot.distribution.max should be(0) + } + } +} \ No newline at end of file -- cgit v1.2.3