Delete correct todo when filtering. (#1696)

(cherry picked from commit 1c19319aed7ab85e10e07de570f6160e46140d2b)

Author: mbrandonw

Examples/Todos/Todos/Todos.swift

@@ -50,7 +50,10 @@ struct Todos: ReducerProtocol {
         return .none
 
       case let .delete(indexSet):
-        state.todos.remove(atOffsets: indexSet)
+        let filteredTodos = state.filteredTodos
+        for index in indexSet {
+          state.todos.remove(id: filteredTodos[index].id)
+        }
         return .none
 
       case let .editModeChanged(editMode):

Examples/Todos/TodosTests/TodosTests.swift

@@ -196,6 +196,41 @@ final class TodosTests: XCTestCase {
     }
   }
 
+  func testDeleteWhileFiltered() async {
+    let state = Todos.State(
+      filter: .completed,
+      todos: [
+        Todo.State(
+          description: "",
+          id: UUID(uuidString: "00000000-0000-0000-0000-000000000000")!,
+          isComplete: false
+        ),
+        Todo.State(
+          description: "",
+          id: UUID(uuidString: "00000000-0000-0000-0000-000000000001")!,
+          isComplete: false
+        ),
+        Todo.State(
+          description: "",
+          id: UUID(uuidString: "00000000-0000-0000-0000-000000000002")!,
+          isComplete: true
+        ),
+      ]
+    )
+
+    let store = TestStore(
+      initialState: state,
+      reducer: Todos()
+    )
+
+    await store.send(.delete([0])) {
+      $0.todos = [
+        $0.todos[0],
+        $0.todos[1],
+      ]
+    }
+  }
+
   func testEditModeMoving() async {
     let state = Todos.State(
       todos: [