mirror
/
swift-metrics
mirror of https://github.com/apple/swift-metrics.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Change Timer.record(duration: Duration) signature to avoid source breakage

This commit is contained in:
Hamzah Malik 2024-05-14 14:40:17 +01:00
parent eb18581491
commit d3ae9de68d
Failed to extract signature
2 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Sources/Metrics/Metrics.swift
View File

@ -82,11 +82,9 @@ extension Timer {
/// ///
/// - Parameters: /// - Parameters:
/// - duration: The `Duration` to record. /// - duration: The `Duration` to record.
///
/// - Throws: `TimerError.durationToIntOverflow` if conversion from `Duration` to `Int64` of Nanoseconds overflowed.
@available(macOS 13, iOS 16, tvOS 15, watchOS 8, *) @available(macOS 13, iOS 16, tvOS 15, watchOS 8, *)
@inlinable @inlinable
public func record(_ duration: Duration) { public func record(duration: Duration) {
// `Duration` doesn't have a nice way to convert it nanoseconds or seconds, // `Duration` doesn't have a nice way to convert it nanoseconds or seconds,
// and manual conversion can overflow. // and manual conversion can overflow.
let seconds = duration.components.seconds.multipliedReportingOverflow(by: 1_000_000_000) let seconds = duration.components.seconds.multipliedReportingOverflow(by: 1_000_000_000)

4
Tests/MetricsTests/MetricsTests.swift
View File

@ -111,11 +111,11 @@ class MetricsExtensionsTests: XCTestCase {
let duration = Duration(secondsComponent: 3, attosecondsComponent: 123_000_000_000_000_000) let duration = Duration(secondsComponent: 3, attosecondsComponent: 123_000_000_000_000_000)
let nanoseconds = duration.components.seconds * 1_000_000_000 + duration.components.attoseconds / 1_000_000_000 let nanoseconds = duration.components.seconds * 1_000_000_000 + duration.components.attoseconds / 1_000_000_000
timer.record(duration) timer.record(duration: duration)
// Record a Duration that would overflow, // Record a Duration that would overflow,
// expect Int64.max to be recorded. // expect Int64.max to be recorded.
timer.record(Duration(secondsComponent: 10_000_000_000, attosecondsComponent: 123)) timer.record(duration: Duration(secondsComponent: 10_000_000_000, attosecondsComponent: 123))
let testTimer = try metrics.expectTimer(timer) let testTimer = try metrics.expectTimer(timer)
XCTAssertEqual(testTimer.values.count, 2, "expected number of entries to match") XCTAssertEqual(testTimer.values.count, 2, "expected number of entries to match")