Skip to content

Commit 0d1cb7c

Browse files
committed
fix display unit
1 parent ba329da commit 0d1cb7c

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

Sources/Prometheus/MetricTypes/Summary.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,8 @@ public class PromSummary<NumType: DoubleRepresentable, Labels: SummaryLabels>: P
139139

140140
// Updated for SwiftMetrics 2.0 to be unit agnostic if displayUnit is set or default to nanoseconds.
141141
private func format(_ v: Double) -> Double {
142-
let displayUnitScale = self.displayUnit?.scaleFromNanoseconds ?? 1
142+
let displayUnit = lock.withLock { self.displayUnit }
143+
let displayUnitScale = displayUnit?.scaleFromNanoseconds ?? 1
143144
return v / Double(displayUnitScale)
144145
}
145146

Tests/SwiftPrometheusTests/SummaryTests.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,8 @@ final class SummaryTests: XCTestCase {
3636

3737
func testSummary() {
3838
let summary = Timer(label: "my_summary")
39-
39+
summary.handler.preferDisplayUnit(.nanoseconds)
40+
4041
summary.recordNanoseconds(1)
4142
summary.recordNanoseconds(2)
4243
summary.recordNanoseconds(4)

0 commit comments

Comments
 (0)