Remove redundant tests.

Author: beacoder

tests/test_agent.py

@@ -1528,7 +1528,8 @@ def test_compact_empty_summary_returns_false(self):
         session = RecordingSession()
 
         def empty_chat_sync(
-            messages, system=None, temperature=None, max_tokens=None, reasoning_effort=None
+            messages, system=None, temperature=None, max_tokens=None, reasoning_effort=None,
+            cancel_check=None,
         ):
             return Message(role="assistant", content=""), Usage()
 
@@ -1810,32 +1811,6 @@ def test_sync_tools_execute_in_call_order(self):
         self.assertEqual(by_id["2"], "result of Bash")
         self.assertEqual(by_id["3"], "result of Grep")
 
-    def test_every_sync_call_runs_one_by_one(self):
-        """Sync rounds are sequential by default: EVERY call still
-        executes (nothing is dropped), peak concurrency stays 1, the
-        round takes ~the sum of the durations, and results still
-        arrive in original order."""
-        session = ParallelToolSession(duration=0.15)
-        session.tools_enabled = False
-        calls = [
-            ToolCall(id=str(i), name="Read", arguments='{"file_path": "/tmp/x.py"}')
-            for i in range(1, 4)
-        ]
-        session.client.script = [("", calls), "done"]
-        loop = AgentLoop(session, messages=[Message(role="user", content="go")])
-        start = time.monotonic()
-        result = loop.run()
-        elapsed = time.monotonic() - start
-        self.assertEqual(result, "done")
-        self.assertEqual(session.executed_count, 3)
-        self.assertEqual(session.max_active, 1)
-        # ~3 x 0.15s serialized, not a single 0.15s batch
-        self.assertGreaterEqual(elapsed, 0.4)
-        self.assertEqual(
-            [m.tool_call_id for m in loop.messages if m.role == "tool"],
-            ["1", "2", "3"],
-        )
-
     def test_cancel_during_delivery_discards_partial_round(self):
         """Ctrl-C landing while results are being DELIVERED (after all
         tools already ran) must stop the delivery loop: the tools'

tests/test_invariants.py

@@ -368,14 +368,6 @@ class TestFSMStructuralInvariants(unittest.TestCase):
     entry, so the machine can never route out of a terminal state;
     the driver stops at the first terminal state."""
 
-    def test_terminal_states_have_no_outgoing_transitions(self):
-        """DONE/ERRS/ABRT are sinks: the table has no entry for them,
-        so the machine can never route out of a terminal state.  (They
-        may be transition TARGETS — WAIT routes to ERRS on API error —
-        but never sources.)"""
-        for state in AgentLoop.TERMINAL:
-            self.assertNotIn(state, AgentLoop.TRANSITIONS)
-
     def test_every_non_terminal_state_has_transitions_with_true_default(self):
         for state in set(AgentLoop.HANDLERS) - AgentLoop.TERMINAL:
             table = AgentLoop.TRANSITIONS.get(state)

tests/test_tools_misc.py

@@ -160,10 +160,6 @@ def test_invalid_questions_returns_error(self):
         result = Question().run({"questions": "nope"}, ToolContext())
         self.assertEqual(result, "Error: questions must be an array")
 
-    def test_missing_questions_returns_error(self):
-        result = Question().run({}, ToolContext())
-        self.assertEqual(result, "Error: questions must be an array")
-
     def test_ask_questions_exception_contained(self):
         def boom(qs):
             raise RuntimeError("ui broke")
@@ -219,16 +215,6 @@ def test_skill_found_reads_file(self):
             out = Skill().run({"skill": "my-skill"}, ToolContext(sess))
             self.assertEqual(out, "[Skill: my-skill]\n\ninstructions")
 
-    def test_skill_found_without_session_args(self):
-        with tempfile.TemporaryDirectory() as d:
-            p = os.path.join(d, "skill.md")
-            with open(p, "w") as f:
-                f.write("x")
-            sess = FakeSession()
-            sess.find_skill = lambda name: p
-            out = Skill().run({}, ToolContext(sess))
-            self.assertEqual(out, "[Skill: ]\n\nx")
-
 
 class TestTodoWriteTool(unittest.TestCase):
     """TodoWrite updates the session and returns the JSON payload."""