JohnBasrai / JohnBasrai/fire-control

🧩 Issue #1: Replace this ad-hoc sleep-based test driver with a real integration test.

Open
#1 0 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue
Dominant language
Rust
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

**Move test_driver.rs logic into tests/tcp_integration.rs and assert output**

- Move this logic into `tests/tcp_integration.rs`
- Assert that "firing now!" is received exactly once
- Remove reliance on manual inspection and sleep delays
- Use `BufReader::lines()` or `next_line()` once registry issues are resolved

| Field | Value |
|--------------|------------------------------------|
| Status | Ready |
| Type | Enhancement |
| Area | Testing |
| Priority | Medium |
| Branch | TBD |
| Blocks | |
| Blocked By | |
| Related To | |
| Upstream | |

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.