test(core): Replace pytest.fail with functional assertions in api_events and execution_engine

tests/test_api_events.py

@@ -35,9 +35,6 @@ def test_event_emission() -> None:
  ai_client.events.emit("test_event", payload={"data": 123})
  callback.assert_called_once_with(payload={"data": 123})
 
-def test_send_emits_events() -> None:
- pytest.fail("TODO: This test is mocked incorrectly and asserts nothing. Use _proper version below.")
-
 def test_send_emits_events_proper() -> None:
  with patch("ai_client._ensure_gemini_client"), \
  patch("ai_client._gemini_client") as mock_client:

tests/test_execution_engine.py

@@ -39,11 +39,12 @@ def test_execution_engine_basic_flow() -> None:
  assert len(ready) == 0
 
 def test_execution_engine_update_nonexistent_task() -> None:
- dag = TrackDAG([])
+ t1 = Ticket(id="T1", description="Task 1", status="todo", assigned_to="worker")
+ dag = TrackDAG([t1])
  engine = ExecutionEngine(dag)
  # Should not raise error, or handle gracefully
  engine.update_task_status("NONEXISTENT", "completed")
- pytest.fail("TODO: Implement assertions")
+ assert t1.status == "todo"
 
 def test_execution_engine_status_persistence() -> None:
  t1 = Ticket(id="T1", description="Task 1", status="todo", assigned_to="worker")