Skip to content

Commit 30d5d3a

Browse files
cdeustclaude
andcommitted
fix(tests): update exploding_decay signature to match Phase B memories= param
The Phase B consolidation-scoped cache (80ccc2f) added an optional memories= parameter to run_decay_cycle. The regression test's fake was pinned to the old 2-arg signature, so CI failed with TypeError 'takes 2 positional arguments but 3 were given'. The fix was already in the working copy but was not staged into 80ccc2f (only mcp_server/ was added). Committing now to unblock CI on v3.11.0. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 86e6647 commit 30d5d3a

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

tests_py/handlers/test_consolidate_telemetry.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -119,14 +119,12 @@ async def test_consolidate_stage_exception_does_not_crash_siblings(
119119
"""If one stage raises, other stages still run and an error field surfaces."""
120120
from mcp_server.handlers.consolidation import decay as decay_mod
121121

122-
def exploding_decay(store, settings):
122+
def exploding_decay(store, settings, memories=None):
123123
raise RuntimeError("boom in decay")
124124

125125
monkeypatch.setattr(decay_mod, "run_decay_cycle", exploding_decay)
126126
# Also patch the re-export imported by consolidate.py at module load time
127-
monkeypatch.setattr(
128-
consolidate_handler, "run_decay_cycle", exploding_decay
129-
)
127+
monkeypatch.setattr(consolidate_handler, "run_decay_cycle", exploding_decay)
130128

131129
result = await consolidate_handler.handler()
132130

0 commit comments

Comments
 (0)