Fix 5.9 tests

pointfreeco · Jul 12, 2023 · 4a87bb7 · 4a87bb7
1 parent a146af9
commit 4a87bb7
Showing 1 changed file with 57 additions and 64 deletions.
diff --git a/Tests/CustomDumpTests/DumpTests.swift b/Tests/CustomDumpTests/DumpTests.swift
@@ -709,71 +709,64 @@ final class DumpTests: XCTestCase {
 
   func testKeyPath() {
     var dump = ""
+    #if swift(>=5.9)
+      if #available(macOS 13.3, iOS 16.4, watchOS 9.4, tvOS 16.4, *) {
+        dump = ""
+        customDump(\UserClass.name, to: &dump)
+        XCTAssertNoDifference(
+          dump,
+          #"""
+          \UserClass.name
+          """#
+        )
+
+        dump = ""
+        customDump(\Pair.driver.name, to: &dump)
+        XCTAssertNoDifference(
+          dump,
+          #"""
+          \Pair.driver.name
+          """#
+        )
+
+        dump = ""
+        customDump(\User.name.count, to: &dump)
+        XCTAssertNoDifference(
+          dump,
+          #"""
+          \User.name.count
+          """#
+        )
 
-    // NB: This code path marks the expectation of relying on SE-0369's
-    // `AnyKeyPath.debugDescription`, which currently has a crash related to dynamic member lookup:
-    // https://github.com/apple/swift/issues/64865
-    //
-    // #if swift(>=5.8)
-    //   if #available(macOS 13.3, iOS 16.4, watchOS 9.4, tvOS 16.4, *) {
-    //     dump = ""
-    //     customDump(\UserClass.name, to: &dump)
-    //     XCTAssertNoDifference(
-    //       dump,
-    //       #"""
-    //       \UserClass.name
-    //       """#
-    //     )
-    //
-    //     dump = ""
-    //     customDump(\Pair.driver.name, to: &dump)
-    //     XCTAssertNoDifference(
-    //       dump,
-    //       #"""
-    //       \Pair.driver.name
-    //       """#
-    //     )
-    //
-    //     dump = ""
-    //     customDump(\User.name.count, to: &dump)
-    //     XCTAssertNoDifference(
-    //       dump,
-    //       #"""
-    //       \User.name.count
-    //       """#
-    //     )
-    //
-    //     dump = ""
-    //     customDump(\(x: Double, y: Double).x, to: &dump)
-    //     XCTAssertNoDifference(
-    //       dump,
-    //       #"""
-    //       \(x: Double, y: Double).x
-    //       """#
-    //     )
-    //
-    //     dump = ""
-    //     customDump(\Item.$isInStock, to: &dump)
-    //     XCTAssertNoDifference(
-    //       dump,
-    //       #"""
-    //       \Item.$isInStock
-    //       """#
-    //     )
-    //
-    //     // NB: This currently crashes when using Swift's `debugDescription`:
-    //     // https://github.com/apple/swift/issues/64865
-    //     dump = ""
-    //     customDump(\Wrapped<String>.count, to: &dump)
-    //     XCTAssertNoDifference(
-    //       dump,
-    //       #"""
-    //       \Wrapped.count
-    //       """#
-    //     )
-    //     return
-    //   }
-    // #endif
+        dump = ""
+        customDump(\(x: Double, y: Double).x, to: &dump)
+        XCTAssertNoDifference(
+          dump,
+          #"""
+          \(x: Double, y: Double).x
+          """#
+        )
+
+        dump = ""
+        customDump(\Item.$isInStock, to: &dump)
+        XCTAssertNoDifference(
+          dump,
+          #"""
+          \Item.$isInStock
+          """#
+        )
+
+        dump = ""
+        customDump(\Wrapped<String>.count, to: &dump)
+        XCTAssertNoDifference(
+          dump,
+          #"""
+          \Wrapped<String>.subscript(dynamicMember: <unknown>)
+          """#
+        )
+        return
+      }
+    #endif
     #if os(iOS) || os(macOS) || os(tvOS) || os(watchOS)
       // Run twice to exercise cached lookup
       for _ in 1...2 {