Add test case for custom UnitSpeed

brantonb · May 15, 2024 · c284c98 · c284c98
1 parent 018441a
commit c284c98
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/Sources/RunningPace/UnitSpeedExtensions.swift b/Sources/RunningPace/UnitSpeedExtensions.swift
@@ -8,7 +8,7 @@ public extension UnitSpeed {
                                                converter: UnitConverterInverse(constant: 16.666666666667))
 }
 
-private class UnitConverterInverse: UnitConverter {
+class UnitConverterInverse: UnitConverter {
     let constant: Double
 
     init(constant: Double) {

diff --git a/Tests/RunningPaceTests/OperatorsTests.swift b/Tests/RunningPaceTests/OperatorsTests.swift
@@ -1,5 +1,5 @@
 import XCTest
-import RunningPace
+@testable import RunningPace
 
 final class OperatorsTests: XCTestCase {
 
@@ -64,6 +64,8 @@ final class OperatorsTests: XCTestCase {
     }
 
     func testMultiplyingSpeedByDurationHasProperUnits() {
+        let secondsPerMeter = UnitSpeed(symbol: "\(UnitDuration.seconds.symbol)/\(UnitLength.meters.symbol)",
+                                        converter: UnitConverterInverse(constant: 0.277778))
         let conversions: [(unitSpeed: UnitSpeed, unitLength: UnitLength)] = [
             (.minutesPerMile, .miles),
             (.milesPerHour, .miles),
@@ -73,6 +75,9 @@ final class OperatorsTests: XCTestCase {
 
             (.knots, .nauticalMiles),
             (.metersPerSecond, .meters),
+
+            // Default should be kilometers for any unknown types like this custom one
+            (secondsPerMeter, .kilometers),
         ]
 
         for (unitSpeed, unitLength) in conversions {
@@ -211,4 +216,8 @@ private func assertEqualDurations(_ expected: Duration, _ actual: Duration) {
     let actualSeconds = actual.converted(to: .seconds).value
     XCTAssertEqual(expectedSeconds, actualSeconds, accuracy: tolerance)
 }
+
+private extension UnitSpeed {
+    static let secondsPerMeter = UnitSpeed(symbol: "\(UnitDuration.seconds.symbol)/\(UnitLength.meters.symbol)",
+                                           converter: UnitConverterInverse(constant: 0.277778))
 }