Fix unimplemented endpoints (#101)

tgrapperon · Jul 25, 2024 · 90c775f · 90c775f
1 parent 406ed2d
commit 90c775f
Show file tree

Hide file tree

Showing 18 changed files with 509 additions and 429 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -16,39 +16,39 @@ concurrency:
 jobs:
   library-swift-latest:
     name: Library (swift-latest)
-    runs-on: macos-13
+    runs-on: macos-14
     strategy:
       matrix:
         config: ['debug', 'release']
     steps:
       - uses: actions/checkout@v3
-      - name: Select Xcode 14.3
-        run: sudo xcode-select -s /Applications/Xcode_14.3.app
+      - name: Select Xcode 15.4
+        run: sudo xcode-select -s /Applications/Xcode_15.4.app
       - name: Run ${{ matrix.config }} tests
         run: CONFIG=${{ matrix.config }} make test-library
       - name: Build platforms ${{ matrix.config }}
         run: CONFIG=${{ matrix.config }} make build-all-platforms
 
   library-evolution:
     name: Library (evolution)
-    runs-on: macos-13
+    runs-on: macos-14
     steps:
       - uses: actions/checkout@v3
-      - name: Select Xcode 14.3
-        run: sudo xcode-select -s /Applications/Xcode_14.3.app
+      - name: Select Xcode 15.4
+        run: sudo xcode-select -s /Applications/Xcode_15.4.app
       - name: Build for library evolution
         run: make build-for-library-evolution
 
-  library-swift-5-7:
-    name: Library (swift 5.7)
-    runs-on: macos-12
+  library-swift-5-9:
+    name: Library (swift 5.9)
+    runs-on: macos-14
     strategy:
       matrix:
         config: ['debug', 'release']
     steps:
       - uses: actions/checkout@v3
-      - name: Select Xcode 14.2
-        run: sudo xcode-select -s /Applications/Xcode_14.2.app
+      - name: Select Xcode 15.4
+        run: sudo xcode-select -s /Applications/Xcode_15.4.app
       - name: Run ${{ matrix.config }} tests
         run: CONFIG=${{ matrix.config }} make test-library
       - name: Build platforms ${{ matrix.config }}
@@ -67,7 +67,7 @@ jobs:
   static-stdlib:
     strategy:
       matrix:
-        image: ['swift:5.7-focal', 'swift:5.8-focal']
+        image: ['swift:5.9-focal']
     runs-on: ubuntu-latest
     container:
       image: ${{ matrix.image }}

diff --git a/DependenciesAdditions.xcworkspace/xcshareddata/swiftpm/Package.resolved b/DependenciesAdditions.xcworkspace/xcshareddata/swiftpm/Package.resolved
@@ -5,8 +5,8 @@
       "kind" : "remoteSourceControl",
       "location" : "https://github.com/pointfreeco/combine-schedulers",
       "state" : {
-        "revision" : "9dc9cbe4bc45c65164fa653a563d8d8db61b09bb",
-        "version" : "1.0.0"
+        "revision" : "9fa31f4403da54855f1e2aeaeff478f4f0e40b13",
+        "version" : "1.0.2"
       }
     },
     {
@@ -23,8 +23,8 @@
       "kind" : "remoteSourceControl",
       "location" : "https://github.com/pointfreeco/swift-clocks",
       "state" : {
-        "revision" : "d1fd837326aa719bee979bdde1f53cd5797443eb",
-        "version" : "1.0.0"
+        "revision" : "3581e280bf0d90c3fb9236fb23e75a5d8c46b533",
+        "version" : "1.0.4"
       }
     },
     {
@@ -50,8 +50,8 @@
       "kind" : "remoteSourceControl",
       "location" : "https://github.com/pointfreeco/swift-dependencies",
       "state" : {
-        "revision" : "350e1e119babe8525f9bd155b76640a5de270184",
-        "version" : "1.3.0"
+        "revision" : "cc26d06125dbc913c6d9e8a905a5db0b994509e0",
+        "version" : "1.3.5"
       }
     },
     {
@@ -75,7 +75,7 @@
     {
       "identity" : "swift-syntax",
       "kind" : "remoteSourceControl",
-      "location" : "https://github.com/apple/swift-syntax",
+      "location" : "https://github.com/swiftlang/swift-syntax",
       "state" : {
         "revision" : "303e5c5c36d6a558407d364878df131c3546fad8",
         "version" : "510.0.2"
@@ -95,8 +95,8 @@
       "kind" : "remoteSourceControl",
       "location" : "https://github.com/pointfreeco/xctest-dynamic-overlay",
       "state" : {
-        "revision" : "6f30bdba373bbd7fbfe241dddd732651f2fbd1e2",
-        "version" : "1.1.2"
+        "revision" : "357ca1e5dd31f613a1d43320870ebc219386a495",
+        "version" : "1.2.2"
       }
     }
   ],

diff --git a/Makefile b/Makefile
@@ -1,5 +1,5 @@
 CONFIG = debug
-PLATFORM_IOS = iOS Simulator,name=iPhone 11 Pro Max
+PLATFORM_IOS = iOS Simulator,name=iPhone 15 Pro Max
 PLATFORM_MACOS = macOS
 PLATFORM_MAC_CATALYST = macOS,variant=Mac Catalyst
 PLATFORM_TVOS = tvOS Simulator,name=Apple TV
@@ -47,7 +47,7 @@ test-linux:
 		--rm \
 		-v "$(PWD):$(PWD)" \
 		-w "$(PWD)" \
-		swift:5.7-focal \
+		swift:5.9-focal \
 		bash -c 'apt-get update && apt-get -y install make && make test-swift'
 
 build-for-static-stdlib:
@@ -64,12 +64,12 @@ build-for-static-stdlib-docker:
 	@docker run \
 		-v "$(PWD):$(PWD)" \
 		-w "$(PWD)" \
-		swift:5.8-focal \
+		swift:5.9-focal \
 		bash -c "swift build -c debug --static-swift-stdlib"
 	@docker run \
 		-v "$(PWD):$(PWD)" \
 		-w "$(PWD)" \
-		swift:5.8-focal \
+		swift:5.9-focal \
 		bash -c "swift build -c release --static-swift-stdlib"
 
 format:

diff --git a/Package.swift b/Package.swift
@@ -1,4 +1,4 @@
-// swift-tools-version: 5.7
+// swift-tools-version: 5.9
 // The swift-tools-version declares the minimum version of Swift required to build this package.
 
 import PackageDescription
@@ -43,9 +43,9 @@ let package = Package(
     .library(name: "_SwiftUIDependency", targets: ["_SwiftUIDependency"]),
   ],
   dependencies: [
-    .package(url: "https://github.com/pointfreeco/swift-dependencies", from: "1.3.0"),
+    .package(url: "https://github.com/pointfreeco/swift-dependencies", from: "1.3.5"),
     .package(url: "https://github.com/apple/swift-docc-plugin", from: "1.0.0"),
-    .package(url: "https://github.com/pointfreeco/xctest-dynamic-overlay", from: "1.0.0"),
+    .package(url: "https://github.com/pointfreeco/xctest-dynamic-overlay", from: "1.2.2"),
   ],
   targets: [
 

diff --git a/Sources/AccessibilityDependency/AccessibilityDependency_iOS.swift b/Sources/AccessibilityDependency/AccessibilityDependency_iOS.swift
@@ -401,47 +401,47 @@
         .init(
           _implementation: Implementation(
             buttonShapesEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.buttonShapesEnabled)"#),
+              #"@Dependency(\.accessibility.buttonShapesEnabled)"#, placeholder: false),
             hearingDevicePairedEar: .unimplemented(
               #"@Dependency(\.accessibility.hearingDevicePairedEar)"#, placeholder: []),
             isAssistiveTouchRunning: .unimplemented(
-              #"@Dependency(\.accessibility.isAssistiveTouchRunning)"#),
+              #"@Dependency(\.accessibility.isAssistiveTouchRunning)"#, placeholder: false),
             isBoldTextEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isBoldTextEnabled)"#),
+              #"@Dependency(\.accessibility.isBoldTextEnabled)"#, placeholder: false),
             isClosedCaptioningEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isClosedCaptioningEnabled)"#),
+              #"@Dependency(\.accessibility.isClosedCaptioningEnabled)"#, placeholder: false),
             isDarkerSystemColorsEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isDarkerSystemColorsEnabled)"#),
+              #"@Dependency(\.accessibility.isDarkerSystemColorsEnabled)"#, placeholder: false),
             isGrayscaleEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isGrayscaleEnabled)"#),
+              #"@Dependency(\.accessibility.isGrayscaleEnabled)"#, placeholder: false),
             isGuidedAccessEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isGuidedAccessEnabled)"#),
+              #"@Dependency(\.accessibility.isGuidedAccessEnabled)"#, placeholder: false),
             isInvertColorsEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isInvertColorsEnabled)"#),
+              #"@Dependency(\.accessibility.isInvertColorsEnabled)"#, placeholder: false),
             isMonoAudioEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isMonoAudioEnabled)"#),
+              #"@Dependency(\.accessibility.isMonoAudioEnabled)"#, placeholder: false),
             isOnOffSwitchLabelsEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isOnOffSwitchLabelsEnabled)"#),
+              #"@Dependency(\.accessibility.isOnOffSwitchLabelsEnabled)"#, placeholder: false),
             isReduceMotionEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isReduceMotionEnabled)"#),
+              #"@Dependency(\.accessibility.isReduceMotionEnabled)"#, placeholder: false),
             isReduceTransparencyEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isReduceTransparencyEnabled)"#),
+              #"@Dependency(\.accessibility.isReduceTransparencyEnabled)"#, placeholder: false),
             isShakeToUndoEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isShakeToUndoEnabled)"#),
+              #"@Dependency(\.accessibility.isShakeToUndoEnabled)"#, placeholder: false),
             isSpeakScreenEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isSpeakScreenEnabled)"#),
+              #"@Dependency(\.accessibility.isSpeakScreenEnabled)"#, placeholder: false),
             isSpeakSelectionEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isSpeakSelectionEnabled)"#),
+              #"@Dependency(\.accessibility.isSpeakSelectionEnabled)"#, placeholder: false),
             isSwitchControlRunning: .unimplemented(
-              #"@Dependency(\.accessibility.isSwitchControlRunning)"#),
+              #"@Dependency(\.accessibility.isSwitchControlRunning)"#, placeholder: false),
             isVideoAutoplayEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isVideoAutoplayEnabled)"#),
+              #"@Dependency(\.accessibility.isVideoAutoplayEnabled)"#, placeholder: false),
             isVoiceOverRunning: .unimplemented(
-              #"@Dependency(\.accessibility.isVoiceOverRunning)"#),
+              #"@Dependency(\.accessibility.isVoiceOverRunning)"#, placeholder: false),
             prefersCrossFadeTransitions: .unimplemented(
-              #"@Dependency(\.accessibility.prefersCrossFadeTransitions)"#),
+              #"@Dependency(\.accessibility.prefersCrossFadeTransitions)"#, placeholder: false),
             shouldDifferentiateWithoutColor: .unimplemented(
-              #"@Dependency(\.accessibility.shouldDifferentiateWithoutColor)"#),
+              #"@Dependency(\.accessibility.shouldDifferentiateWithoutColor)"#, placeholder: false),
             configureForGuidedAccess: .unimplemented(
               #"@Dependency(\.accessibility.configureForGuidedAccess)"#),
             convertToScreenCoordinatesUsingUIBezierPath: .unimplemented(
@@ -457,58 +457,61 @@
               #"@Dependency(\.accessibility.guidedAccessRestrictionState)"#,
               placeholder: { _ in .deny }),
             post: .unimplemented(
-              #"@Dependency(\.accessibility.post)"#),
+              #"@Dependency(\.accessibility.post)"#, placeholder: { _, _ in () }),
             registerGestureConflictWithZoom: .unimplemented(
               #"@Dependency(\.accessibility.registerGestureConflictWithZoom)"#),
             requestGuidedAccessSession: .unimplemented(
-              #"@Dependency(\.accessibility.requestGuidedAccessSession)"#),
+              #"@Dependency(\.accessibility.requestGuidedAccessSession)"#,
+              placeholder: { _ in false }
+            ),
             zoomFocusChanged: .unimplemented(
-              #"@Dependency(\.accessibility.zoomFocusChanged)"#)
+              #"@Dependency(\.accessibility.zoomFocusChanged)"#
+            )
           )
         )
       #elseif os(tvOS)
         .init(
           _implementation: Implementation(
             buttonShapesEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.buttonShapesEnabled)"#),
+              #"@Dependency(\.accessibility.buttonShapesEnabled)"#, placeholder: false),
             isAssistiveTouchRunning: .unimplemented(
-              #"@Dependency(\.accessibility.isAssistiveTouchRunning)"#),
+              #"@Dependency(\.accessibility.isAssistiveTouchRunning)"#, placeholder: false),
             isBoldTextEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isBoldTextEnabled)"#),
+              #"@Dependency(\.accessibility.isBoldTextEnabled)"#, placeholder: false),
             isClosedCaptioningEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isClosedCaptioningEnabled)"#),
+              #"@Dependency(\.accessibility.isClosedCaptioningEnabled)"#, placeholder: false),
             isDarkerSystemColorsEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isDarkerSystemColorsEnabled)"#),
+              #"@Dependency(\.accessibility.isDarkerSystemColorsEnabled)"#, placeholder: false),
             isGrayscaleEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isGrayscaleEnabled)"#),
+              #"@Dependency(\.accessibility.isGrayscaleEnabled)"#, placeholder: false),
             isGuidedAccessEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isGuidedAccessEnabled)"#),
+              #"@Dependency(\.accessibility.isGuidedAccessEnabled)"#, placeholder: false),
             isInvertColorsEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isInvertColorsEnabled)"#),
+              #"@Dependency(\.accessibility.isInvertColorsEnabled)"#, placeholder: false),
             isMonoAudioEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isMonoAudioEnabled)"#),
+              #"@Dependency(\.accessibility.isMonoAudioEnabled)"#, placeholder: false),
             isOnOffSwitchLabelsEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isOnOffSwitchLabelsEnabled)"#),
+              #"@Dependency(\.accessibility.isOnOffSwitchLabelsEnabled)"#, placeholder: false),
             isReduceMotionEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isReduceMotionEnabled)"#),
+              #"@Dependency(\.accessibility.isReduceMotionEnabled)"#, placeholder: false),
             isReduceTransparencyEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isReduceTransparencyEnabled)"#),
+              #"@Dependency(\.accessibility.isReduceTransparencyEnabled)"#, placeholder: false),
             isShakeToUndoEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isShakeToUndoEnabled)"#),
+              #"@Dependency(\.accessibility.isShakeToUndoEnabled)"#, placeholder: false),
             isSpeakScreenEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isSpeakScreenEnabled)"#),
+              #"@Dependency(\.accessibility.isSpeakScreenEnabled)"#, placeholder: false),
             isSpeakSelectionEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isSpeakSelectionEnabled)"#),
+              #"@Dependency(\.accessibility.isSpeakSelectionEnabled)"#, placeholder: false),
             isSwitchControlRunning: .unimplemented(
-              #"@Dependency(\.accessibility.isSwitchControlRunning)"#),
+              #"@Dependency(\.accessibility.isSwitchControlRunning)"#, placeholder: false),
             isVideoAutoplayEnabled: .unimplemented(
-              #"@Dependency(\.accessibility.isVideoAutoplayEnabled)"#),
+              #"@Dependency(\.accessibility.isVideoAutoplayEnabled)"#, placeholder: false),
             isVoiceOverRunning: .unimplemented(
-              #"@Dependency(\.accessibility.isVoiceOverRunning)"#),
+              #"@Dependency(\.accessibility.isVoiceOverRunning)"#, placeholder: false),
             prefersCrossFadeTransitions: .unimplemented(
-              #"@Dependency(\.accessibility.prefersCrossFadeTransitions)"#),
+              #"@Dependency(\.accessibility.prefersCrossFadeTransitions)"#, placeholder: false),
             shouldDifferentiateWithoutColor: .unimplemented(
-              #"@Dependency(\.accessibility.shouldDifferentiateWithoutColor)"#),
+              #"@Dependency(\.accessibility.shouldDifferentiateWithoutColor)"#, placeholder: false),
             convertToScreenCoordinatesUsingUIBezierPath: .unimplemented(
               #"@Dependency(\.accessibility.convertToScreenCoordinatesUsingUIBezierPath)"#,
               placeholder: { _, _ in UIBezierPath() }),
@@ -522,13 +525,14 @@
               #"@Dependency(\.accessibility.guidedAccessRestrictionState)"#,
               placeholder: { _ in .deny }),
             post: .unimplemented(
-              #"@Dependency(\.accessibility.post)"#),
+              #"@Dependency(\.accessibility.post)"#, placeholder: { _, _ in }),
             registerGestureConflictWithZoom: .unimplemented(
-              #"@Dependency(\.accessibility.registerGestureConflictWithZoom)"#),
+              #"@Dependency(\.accessibility.registerGestureConflictWithZoom)"#, placeholder: {}),
             requestGuidedAccessSession: .unimplemented(
-              #"@Dependency(\.accessibility.requestGuidedAccessSession)"#),
+              #"@Dependency(\.accessibility.requestGuidedAccessSession)"#,
+              placeholder: { _ in false }),
             zoomFocusChanged: .unimplemented(
-              #"@Dependency(\.accessibility.zoomFocusChanged)"#)
+              #"@Dependency(\.accessibility.zoomFocusChanged)"#, placeholder: { _, _, _ in })
           )
         )
       #endif