Use also instead of tap

propensive · Dec 6, 2023 · 672f0c1 · 672f0c1
1 parent 51451cc
commit 672f0c1
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/core/report.scala b/src/core/report.scala
@@ -168,22 +168,21 @@ class TestReport(using Environment):
   private val details: scm.SortedMap[TestId, scm.ArrayBuffer[DebugInfo]] =
     scm.TreeMap[TestId, scm.ArrayBuffer[DebugInfo]]().withDefault(_ => scm.ArrayBuffer[DebugInfo]())
 
-  def declareSuite(suite: TestSuite): TestReport = this.tap: _ =>
+  def declareSuite(suite: TestSuite): TestReport = this.also:
     resolve(suite.parent).tests(suite.id) = ReportLine.Suite(suite)
 
-  def addBenchmark(testId: TestId, benchmark: Benchmark): TestReport = this.tap: _ =>
+  def addBenchmark(testId: TestId, benchmark: Benchmark): TestReport = this.also:
     val benchmarks = resolve(testId.suite).tests
     benchmarks.getOrElseUpdate(testId, ReportLine.Bench(testId, benchmark))
 
-  def addOutcome(testId: TestId, outcome: Outcome): TestReport = this.tap: _ =>
+  def addOutcome(testId: TestId, outcome: Outcome): TestReport = this.also:
     val tests = resolve(testId.suite).tests
 
     (tests.getOrElseUpdate(testId, ReportLine.Test(testId, scm.ArrayBuffer[Outcome]())): @unchecked) match
       case ReportLine.Test(_, buf) => buf.append(outcome)
 
   def addDebugInfo(testId: TestId, info: DebugInfo): TestReport =
-    this.tap: _ =>
-      details(testId) = details(testId).append(info)
+    this.also(details(testId) = details(testId).append(info))
 
   enum Status:
     case Pass, Fail, Throws, CheckThrows, Mixed, Suite, Bench