Add more build and test actions to the CI

.github/workflows/ci.yaml

@@ -17,16 +17,19 @@ jobs:
         run: "sudo xcode-select -s /Applications/Xcode_$XCODE_VERSION.app"
       - name: Checkout
         uses: actions/checkout@v4
-      - name: Build project
+      - name: Lint Swift files
+        run: swift format lint Sources/ Tests/ Example/ --recursive
+      - name: Build sourcekit-bazel-bsp (debug)
         run: swift build
-      - name: Run tests
+      - name: Test sourcekit-bazel-bsp
         run: swift test
-      - name: Build example iOS app
+      - name: Build sourcekit-bazel-bsp (release)
+        run: swift build -c release
+      - name: Build example //HelloWorld iOS app
         run: |
           cd Example
-          bazel build //HelloWorld
-      # TODO: add test target to the example iOS app
-      # - name: Test example iOS app
-      #   run: |
-      #     cd Example
-      #     bazel test //HelloWorld/HelloWorldTests/...
+          bazelisk build //HelloWorld
+      - name: Test example //HelloWorld iOS app
+        run: |
+          cd Example
+          bazelisk test //HelloWorld:HelloWorldTests

Example/.gitignore

@@ -11,6 +11,7 @@
 /.settings
 /bazel.iml
 /bazel-*
+/custom-*
 /output/
 /production
 /.sass-cache

Example/HelloWorld/HelloWorldTests/TodoListManagerTests.swift

@@ -24,14 +24,21 @@ import XCTest
 
 class TodoListManagerTests: XCTestCase {
 
+    let fakeDefaults = UserDefaults(suiteName: "test")!
+
+    override func tearDown() {
+        super.tearDown()
+        fakeDefaults.removeObject(forKey: "todo-items")
+    }
+
     func testTodoListManagerInitialization() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         XCTAssertNotNil(manager)
         XCTAssertTrue(manager.todoItems.isEmpty)
     }
 
     func testAddTodoItem() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         let initialCount = manager.todoItems.count
 
         manager.addTodoItem(title: "New Task")
@@ -42,7 +49,7 @@ class TodoListManagerTests: XCTestCase {
     }
 
     func testToggleTodoItem() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         manager.addTodoItem(title: "Test Task")
 
         guard let item = manager.todoItems.first else {
@@ -57,7 +64,7 @@ class TodoListManagerTests: XCTestCase {
     }
 
     func testDeleteTodoItem() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         manager.addTodoItem(title: "Task to Delete")
 
         guard let item = manager.todoItems.first else {
@@ -73,16 +80,16 @@ class TodoListManagerTests: XCTestCase {
     }
 
     func testTodoListManagerPersistence() {
-        let manager1 = TodoListManager()
+        let manager1 = TodoListManager(userDefaults: fakeDefaults)
         manager1.addTodoItem(title: "Persistent Task")
 
-        let manager2 = TodoListManager()
+        let manager2 = TodoListManager(userDefaults: fakeDefaults)
         XCTAssertEqual(manager2.todoItems.count, 1)
         XCTAssertEqual(manager2.todoItems.first?.title, "Persistent Task")
     }
 
     func testMultipleTodoItems() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
 
         manager.addTodoItem(title: "Task 1")
         manager.addTodoItem(title: "Task 2")
@@ -95,7 +102,7 @@ class TodoListManagerTests: XCTestCase {
     }
 
     func testToggleNonExistentItem() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         let fakeItem = TodoItem(title: "Fake Item")
 
         // Should not crash
@@ -104,7 +111,7 @@ class TodoListManagerTests: XCTestCase {
     }
 
     func testDeleteNonExistentItem() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         let fakeItem = TodoItem(title: "Fake Item")
 
         // Should not crash
@@ -113,15 +120,15 @@ class TodoListManagerTests: XCTestCase {
     }
 
     func testEmptyTitleHandling() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         manager.addTodoItem(title: "")
 
         XCTAssertEqual(manager.todoItems.count, 1)
         XCTAssertEqual(manager.todoItems.first?.title, "")
     }
 
     func testVeryLongTitle() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
         let longTitle = String(repeating: "A", count: 1000)
         manager.addTodoItem(title: longTitle)
 
@@ -130,7 +137,7 @@ class TodoListManagerTests: XCTestCase {
     }
 
     func testTodoListManagerPerformance() {
-        let manager = TodoListManager()
+        let manager = TodoListManager(userDefaults: fakeDefaults)
 
         measure {
             for i in 0..<100 {

Example/HelloWorld/TodoModels/Sources/TodoListManager.swift

@@ -23,10 +23,11 @@ import SwiftUI
 public final class TodoListManager: ObservableObject {
     @Published public var todoItems: [TodoItem] = []
 
-    private let userDefaults = UserDefaults.standard
+    private let userDefaults: UserDefaults
     private let todoItemsKey = "todo-items"
 
-    public init() {
+    public init(userDefaults: UserDefaults = .standard) {
+        self.userDefaults = userDefaults
         loadTodoItems()
     }
 

Example/MODULE.bazel

@@ -4,10 +4,17 @@ bazel_dep(
     repo_name = "build_bazel_rules_swift",
 )
 
+# We need a commit that has not yet made into a release, so overriding it for now
 bazel_dep(
     name = "rules_apple",
-    version = "4.0.1",
     repo_name = "build_bazel_rules_apple",
 )
+archive_override(
+    module_name = "rules_apple",
+    sha256 = "0abe1a851d65d64a1692bdc95a0649cba4218f0007d36eda030eb6bdfd6b6937",
+    strip_prefix = "rules_apple-a9fcb1c3ae6382534d773d73c035035e9764eadb",
+    url = "https://github.com/bazelbuild/rules_apple/archive/a9fcb1c3ae6382534d773d73c035035e9764eadb.zip",
+)
+
 
 bazel_dep(name = "apple_support", version = "1.22.1", repo_name = "build_bazel_apple_support")

Sources/SourceKitBazelBSP/Server/MessageHandler/BSPServerMessageHandlerImpl.swift

@@ -218,7 +218,9 @@ final class BSPServerMessageHandlerImpl: @unchecked Sendable {
         var affectedTargets: Set<URI> = []
         for change in changes {
             let targetsForSrc = workspaceBuildTargetsHandler.srcToTargetsMap[change] ?? []
-            targetsForSrc.forEach { affectedTargets.insert($0) }
+            for target in targetsForSrc {
+                affectedTargets.insert(target)
+            }
         }
         let response = OnBuildTargetDidChangeNotification(
             changes: affectedTargets.map {