Skip to content

fix(tests): unblock kanban diagnostics + compression boundary - #28466

Closed
felix-windsor wants to merge 1 commit into
NousResearch:mainfrom
felix-windsor:felix/fix-main-tests-2026-05-19
Closed

fix(tests): unblock kanban diagnostics + compression boundary#28466
felix-windsor wants to merge 1 commit into
NousResearch:mainfrom
felix-windsor:felix/fix-main-tests-2026-05-19

Conversation

@felix-windsor

@felix-windsor felix-windsor commented May 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix kanban dashboard diagnostics severity filter to be an exact match (warning vs error).
  • Avoid treating non-bool _last_compress_aborted sentinels as a hard abort (MagicMock truthiness) so compression boundary tests exercise the intended path.

Tests

  • ./scripts/run_tests.sh tests/plugins/test_kanban_dashboard_plugin.py::test_diagnostics_endpoint_severity_filter tests/run_agent/test_compression_boundary_hook.py
  • ./.venv/bin/ruff check agent/conversation_compression.py plugins/kanban/dashboard/plugin_api.py

Notes: There is already an open PR for the separate /compress force= test stub drift (#28303); this PR stays scoped to the kanban + compression-boundary failures.

@alt-glitch alt-glitch added type/test Test coverage or test infrastructure comp/plugins Plugin system and bundled plugins comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint P3 Low — cosmetic, nice to have labels May 19, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Superseded by #28465 (merged) which fixes the same kanban diagnostics severity filter and compression boundary test failures. Both PRs address the same stale tests from the same recent merges.

@felix-windsor

Copy link
Copy Markdown
Contributor Author

Closing as superseded by #28465 (merged).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have type/test Test coverage or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants