Skip to content

[T-004] Add 82 new tests (38→120 total, +216%)#3

Open
SuperInstance wants to merge 1 commit intomainfrom
greenhorn/T-004-greenhorn
Open

[T-004] Add 82 new tests (38→120 total, +216%)#3
SuperInstance wants to merge 1 commit intomainfrom
greenhorn/T-004-greenhorn

Conversation

@SuperInstance
Copy link
Copy Markdown
Owner

@SuperInstance SuperInstance commented Apr 12, 2026

Summary

  • Added 3 new test files: allocator (10), rigging (7), connector (7)
  • Enhanced 4 existing test files: vm (+22), coordinator (+15), handshake (+10), scavenger (+11)
  • 120 tests total, all passing

New Coverage

  • pkg/allocator: Budget lifecycle, AutoFromProfile, LoadConfig (YAML, env, missing, invalid)
  • pkg/rigging: AutoSelect (10 sub-cases), Deploy, CanCompute, Execute, Park
  • pkg/connector: New, Connect, Post, ReportStatus, FetchFenceBoard
  • Enhanced VM: SUB, MUL, DIV, MOD, AND, OR, NOT, NEG, ADDI, SUBI, MOV, jumps, loops
  • Enhanced coordinator: not-found, vessel stats, history isolation, fleet status

Testing

  • go test ./... → 120/120 passed

Staging: Open in Devin

- New test files: allocator_test.go, rigging_test.go, connector_test.go
- Enhanced: coordinator_test.go, flux/vm_test.go, handshake_test.go, scavenger_test.go
- New tests: 82 (total: 120 tests, all passing)
- Coverage: Budget, AutoFromProfile, LoadConfig, AutoSelect, Deploy, CanCompute,
  Execute, Park, Connector HTTP, all VM opcodes (SUB, MUL, DIV, MOD, AND, OR,
  NOT, NEG, ADDI, SUBI, MOV, JZ, JMP, LOOP, NOP), coordinator edge cases,
  scavenger edge cases, handshake JSON roundtrips
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 bugs or issues to report.

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