Improve the CI process entirely

.github/workflows/ci.yml

@@ -13,16 +13,22 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  ci:
+  macos:
     runs-on: macos-14
     timeout-minutes: 30
-    env:
-      SCHEME: OneWay
+    steps:
+    - uses: actions/checkout@v4
+    - name: Select Xcode
+      run: sudo xcode-select -s /Applications/Xcode_15.2.app
+    - name: Build
+      run: make build-all
+    - name: Test
+      run: make test-swift
 
+  linux:
+    runs-on: ubuntu-latest
+    timeout-minutes: 30
     steps:
-    - uses: actions/checkout@v3
-    - uses: maxim-lobanov/setup-xcode@v1.6.0
-      with:
-        xcode-version: '15.2'
-    - name: Build and test
-      run: make test-all
+    - uses: actions/checkout@v4
+    - name: Test
+      run: make test-swift

Makefile

@@ -5,24 +5,28 @@ PLATFORM_VISIONOS = visionOS Simulator,name=Apple Vision Pro
 PLATFORM_WATCHOS = watchOS Simulator,name=Apple Watch Series 9 (45mm)
 CONFIG = debug
 
-default: test-all
+default: test-swift
 
-test-all:
-	CONFIG=debug make test
-	CONFIG=release make test
+build-all:
+	CONFIG=debug make build
+	CONFIG=release make build
 
-test:
+build:
 	for platform in \
 		"$(PLATFORM_IOS)" \
 		"$(PLATFORM_MACOS)" \
 		"$(PLATFORM_TVOS)" \
 		"$(PLATFORM_VISIONOS)" \
 		"$(PLATFORM_WATCHOS)"; \
 	do \
-		xcodebuild clean build test \
+		xcodebuild build \
 			-scheme OneWay \
 			-configuration $(CONFIG) \
 			-destination platform="$$platform" || exit 1; \
 	done;
 
-.PHONY: test-all test
+test-swift:
+	swift test --parallel -c debug
+	swift test --parallel -c release
+
+.PHONY: build-all build test-swift

Sources/OneWay/ViewStore.swift

@@ -5,6 +5,7 @@
 //  Copyright (c) 2022-2024 SeungYeop Yeom ( https://github.com/DevYeom ).
 //
 
+#if !os(Linux)
 #if canImport(Combine)
 import Combine
 #endif
@@ -97,3 +98,4 @@ where R.Action: Sendable, R.State: Sendable & Equatable {
 #if canImport(Combine)
 extension ViewStore: ObservableObject { }
 #endif
+#endif

Tests/OneWayTests/StoreTests.swift

@@ -6,7 +6,9 @@
 //
 
 import Clocks
+#if canImport(Combine)
 import Combine
+#endif
 import OneWay
 import XCTest
 
@@ -94,6 +96,7 @@ final class StoreTests: XCTestCase {
         await expect { await sut.state.text == "Success" }
     }
 
+    #if canImport(Combine)
     func test_bind() async {
         var result: Set<String> = []
 
@@ -114,6 +117,7 @@ final class StoreTests: XCTestCase {
 
         XCTAssertEqual(result, ["", "first", "1", "second", "2"])
     }
+    #endif
 
     func test_removeDuplicates() async {
         await sut.send(.response("First"))
@@ -251,8 +255,10 @@ final class StoreTests: XCTestCase {
     }
 }
 
+#if canImport(Combine)
 private let textPublisher = PassthroughSubject<String, Never>()
 private let numberPublisher = PassthroughSubject<Int, Never>()
+#endif
 
 @available(macOS 13.0, iOS 16.0, tvOS 16.0, watchOS 9.0, *)
 private var _clock = TestClock()
@@ -358,6 +364,7 @@ private struct TestReducer: Reducer {
         }
     }
 
+#if canImport(Combine)
     func bind() -> AnyEffect<Action> {
         return .merge(
             .sequence { send in
@@ -372,4 +379,5 @@ private struct TestReducer: Reducer {
             }
         )
     }
+#endif
 }

Tests/OneWayTests/TestHelper/Publisher+Async.swift

@@ -5,6 +5,7 @@
 //  Copyright (c) 2022-2024 SeungYeop Yeom ( https://github.com/DevYeom ).
 //
 
+#if canImport(Combine)
 @preconcurrency import Combine
 
 extension Publisher where Failure == Never {
@@ -21,3 +22,4 @@ extension Publisher where Failure == Never {
         }
     }
 }
+#endif

Tests/OneWayTests/TestHelper/XCTestCase+Expect.swift

@@ -50,3 +50,9 @@ extension XCTestCase {
         }
     }
 }
+
+#if canImport(Darwin)
+#else
+let NSEC_PER_SEC: UInt64 = 1_000_000_000
+let NSEC_PER_MSEC: UInt64 = 1_000_000
+#endif

Tests/OneWayTests/ViewStoreTests.swift

@@ -8,6 +8,7 @@
 import OneWay
 import XCTest
 
+#if !os(Linux)
 @MainActor
 final class ViewStoreTests: XCTestCase {
     private var sut: ViewStore<TestReducer>!
@@ -244,3 +245,4 @@ private actor Result {
         values.append(value)
     }
 }
+#endif