Skip to content

Add comprehensive test suite (64 tests) and fix CI#2

Open
SuperInstance wants to merge 2 commits intomainfrom
superz/add-tests
Open

Add comprehensive test suite (64 tests) and fix CI#2
SuperInstance wants to merge 2 commits intomainfrom
superz/add-tests

Conversation

@SuperInstance
Copy link
Copy Markdown
Owner

@SuperInstance SuperInstance commented Apr 12, 2026

Summary

  • 64 tests covering both Python modules
  • CI fix: removed || true that was masking test failures

Tests Added

Module Tests What is covered
agent_bridge.py 29 GitHubBridge (init, read/write file, list files, bottles read/leave, clone, create vessel, open issue, get commits, discover agents), FluxAgentRuntime (init), KeeperAgentBridge (init, boot register success/failure, pack_baton quality gate)
i2i_agent_bridge.py 35 I2IAgentBridge (init, API get/post/put, read/write file, list dir), I2I protocol (all 20 valid message types, invalid rejection, oracle1 routing, envelope format validation), task execution (taskboard scan, issue/taskboard/bottle tasks), repo analysis (missing files, README quality, auto-improve), fleet improvements (README/CAPABILITY.toml generation), boot sequence, diary logging, status reporting, energy/confidence mechanics

CI Fix

The existing CI had || true appended to the pytest command, meaning all test failures were silently swallowed and CI always passed (green). Removed the || true so test failures now properly fail CI.

All 64 tests pass in <1 second.


Staging: Open in Devin

Super Z added 2 commits April 12, 2026 18:32
- Tests for agent_bridge.py: GitHubBridge (init, read/write file, list files, bottles,
  clone, create vessel, open issue, get commits, discover agents), FluxAgentRuntime (init),
  KeeperAgentBridge (init, boot, pack_baton with quality gate)
- Tests for i2i_agent_bridge.py: I2IAgentBridge (init, API layer, I2I protocol with
  all 20 message types, envelope format, routing to for-fleet vs for-oracle1),
  task execution (taskboard/issue/bottle), repo analysis, fleet improvements,
  diary logging, status reporting, boot sequence, energy/confidence mechanics
- CI fix: remove '|| true' that masked test failures
Copy link
Copy Markdown

@beta-devin-ai-integration beta-devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 6 additional findings.

Staging: Open in Devin

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant