From 9cf3489411e35a737c2d2de63f50677d5ce8a4a8 Mon Sep 17 00:00:00 2001 From: woxtu Date: Mon, 24 Jul 2023 12:31:52 +0000 Subject: [PATCH] Fix `UserDefaults.double(forKey:)` behavior --- Sources/Foundation/UserDefaults.swift | 9 ++++++ Tests/Foundation/Tests/TestUserDefaults.swift | 30 +++++++++++++++++++ 2 files changed, 39 insertions(+) diff --git a/Sources/Foundation/UserDefaults.swift b/Sources/Foundation/UserDefaults.swift index a5972439f7..fda2a9a912 100644 --- a/Sources/Foundation/UserDefaults.swift +++ b/Sources/Foundation/UserDefaults.swift @@ -235,6 +235,15 @@ open class UserDefaults: NSObject { if let bVal = aVal as? Double { return bVal } + if let bVal = aVal as? Bool { + return NSNumber(value: bVal).doubleValue + } + if let bVal = aVal as? Int { + return NSNumber(value: bVal).doubleValue + } + if let bVal = aVal as? Float { + return NSNumber(value: bVal).doubleValue + } if let bVal = aVal as? String { return NSString(string: bVal).doubleValue } diff --git a/Tests/Foundation/Tests/TestUserDefaults.swift b/Tests/Foundation/Tests/TestUserDefaults.swift index 1b5705b9f0..6b46675be9 100644 --- a/Tests/Foundation/Tests/TestUserDefaults.swift +++ b/Tests/Foundation/Tests/TestUserDefaults.swift @@ -36,6 +36,9 @@ class TestUserDefaults : XCTestCase { ("test_setValue_FloatFromInt", test_setValue_FloatFromInt ), ("test_setValue_FloatFromDouble", test_setValue_FloatFromDouble ), ("test_setValue_FloatFromString", test_setValue_FloatFromString ), + ("test_setValue_DoubleFromBool", test_setValue_DoubleFromBool ), + ("test_setValue_DoubleFromInt", test_setValue_DoubleFromInt ), + ("test_setValue_DoubleFromFloat", test_setValue_DoubleFromFloat ), ("test_setValue_DoubleFromString", test_setValue_DoubleFromString ), ("test_setValue_StringFromBool", test_setValue_StringFromBool ), ("test_setValue_StringFromInt", test_setValue_StringFromInt ), @@ -310,6 +313,33 @@ class TestUserDefaults : XCTestCase { XCTAssertEqual(defaults.float(forKey: "key1"), 1234) } + func test_setValue_DoubleFromBool() { + let defaults = UserDefaults.standard + + // Register a boolean default value. UserDefaults.double(forKey:) is supposed to return the converted Double value + defaults.set(true, forKey: "key1") + + XCTAssertEqual(defaults.double(forKey: "key1"), 1) + } + + func test_setValue_DoubleFromInt() { + let defaults = UserDefaults.standard + + // Register an integer default value. UserDefaults.double(forKey:) is supposed to return the converted Double value + defaults.set(42, forKey: "key1") + + XCTAssertEqual(defaults.double(forKey: "key1"), 42) + } + + func test_setValue_DoubleFromFloat() { + let defaults = UserDefaults.standard + + // Register a float default value. UserDefaults.double(forKey:) is supposed to return the converted Double value + defaults.set(12.34 as Float, forKey: "key1") + + XCTAssertEqual(defaults.double(forKey: "key1"), Double(12.34 as Float)) + } + func test_setValue_DoubleFromString() { let defaults = UserDefaults.standard