[test] Fix broken tests with a compile-only CI guard #114
Description
Enhancement Description
Fix test compilation failures and add a compilation-only CI gate to prevent future regressions.
Background
Tests currently fail to compile due to undefined symbols in cmd/devnet-builder/main_test.go. This blocks CI, prevents safe refactoring, and undermines confidence in test coverage.
Scope
Fix undefined symbol references in main_test.go
- Files: cmd/devnet-builder/main_test.go
Add compilation-only CI guard
-
Files: .github/workflows/* (or existing CI workflow)
-
Add step: go test ./... -run='^$'
Non-Goals
- Adding broad new test coverage beyond compilation fixes
- Refactoring production behavior as part of this issue
- Reworking CI pipelines unrelated to Go tests
Risks and Open Questions
- CI runners may differ from local environments; ensure the guard runs in all relevant workflows
- If repo has multiple Go modules, ensure command targets the correct module(s)
- Avoid slowing CI by duplicating heavy test runs
Validation Plan
Unit and Integration Checks
- go test ./... -run='^$' passes
- go test ./... passes where full suite is runnable
End-to-End Checks
- CI run confirms compilation-only guard executes and blocks regressions
- Introduce a controlled compile error (locally) to verify guard effectiveness
Evidence Required in Issue Updates
- go test compilation-only output snippet
- CI log excerpt showing the new compilation-only step
Acceptance Criteria
- go test ./... -run='^$' exits with code 0
- No undefined symbols remain in test files
- CI fails if test compilation breaks again
Deliverables
- PR fixing main_test.go compilation
- PR adding CI compilation-only step (may be combined if repo prefers)