JohnBasrai / JohnBasrai/fire-control

🧩 Issue #6: Add unit test for multiple independent fire commands in sequence

Open
#6 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

Add a unit test to `controller.rs` that verifies support for **multiple fire commands in sequence** — e.g., schedule `Fire(1.0)`, then after it completes, schedule `Fire(2.0)`. Both should trigger independently.

### Goals

- ✅ Ensure two separate fire events result in `"firing now!"` twice
- ✅ Confirm that commands are not overwritten or canceled unintentionally
- ✅ Use atomic counter or similar mechanism to assert correct behavior
- ✅ Keep test duration short (e.g., < 3s) using `sleep(0.05)` style timing

---

| Field | Value |
|-------------|-------------|
| Status | Ready |
| Type | Test |
| Area | Controller |
| Priority | Medium |
| Branch | TBD |
| Blocks | |
| Blocked By | |
| Related To | #1 |
| 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.