Skip to content

Commit 0c82f8c

Browse files
author
Tom Verran
committed
Add some missing assertEquals
1 parent 9c52cde commit 0c82f8c

File tree

1 file changed

+6
-6
lines changed
  • natchez-extras-dogstatsd/src/test/scala/com/ovoenergy/natchez/extras/dogstatsd

1 file changed

+6
-6
lines changed

natchez-extras-dogstatsd/src/test/scala/com/ovoenergy/natchez/extras/dogstatsd/DatadogTest.scala

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ class DatadogTest extends ScalaCheckSuite {
4141

4242
test("Serialisation should allow through dots") {
4343
Prop.forAll(Gen.alphaNumStr, Gen.alphaNumStr) { (pref, suf) =>
44-
filterName(s"$pref.$suf") == s"$pref.$suf".dropWhile(!_.isLetter)
44+
assertEquals(filterName(s"$pref.$suf"), s"$pref.$suf".dropWhile(!_.isLetter))
4545
}
4646
}
4747

@@ -51,10 +51,10 @@ class DatadogTest extends ScalaCheckSuite {
5151

5252
test("Serialisation should generate correct counters and histograms with no tags") {
5353
Prop.forAll(Arbitrary.arbLong.arbitrary) { l =>
54-
makeString(serialiseCounter(Metric("foo", Map.empty), l)) == s"foo:$l|c" &&
55-
makeString(serialiseHistogram(Metric("foo", Map.empty), l)) == s"foo:$l|h|@1.0" &&
56-
makeString(serialiseGauge(Metric("foo", Map.empty), l)) == s"foo:$l|g" &&
57-
makeString(serialiseDistribution(Metric("foo", Map.empty), l)) == s"foo:$l|d|@1.0"
54+
assertEquals(makeString(serialiseCounter(Metric("foo", Map.empty), l)), s"foo:$l|c")
55+
assertEquals(makeString(serialiseHistogram(Metric("foo", Map.empty), l)), s"foo:$l|h|@1.0")
56+
assertEquals(makeString(serialiseGauge(Metric("foo", Map.empty), l)), s"foo:$l|g")
57+
assertEquals(makeString(serialiseDistribution(Metric("foo", Map.empty), l)), s"foo:$l|d|@1.0")
5858
}
5959
}
6060

@@ -98,7 +98,7 @@ class DatadogTest extends ScalaCheckSuite {
9898
test("Config should separate the global prefix from the metric name with a dot") {
9999
Prop.forAll(string) { prefix =>
100100
val config = Config(localAddress, Some(prefix), Map("bar" -> "boz"))
101-
applyConfig(Metric("foo", Map.empty), config).name == s"$prefix.foo"
101+
assertEquals(applyConfig(Metric("foo", Map.empty), config).name, s"$prefix.foo")
102102
}
103103
}
104104
}

0 commit comments

Comments
 (0)