Skip to content

Add second mock battery - #7

Closed
dymk wants to merge 3 commits into
mainfrom
dymk/two-mock-batteries
Closed

Add second mock battery#7
dymk wants to merge 3 commits into
mainfrom
dymk/two-mock-batteries

Conversation

@dymk

@dymk dymk commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • register a 3S mock fuel gauge as battery 0
  • register a distinct 2S mock fuel gauge as battery 1
  • keep both mock state machines updated independently

Validation

  • cargo fmt --check
  • cargo build --locked
  • cargo clippy --locked -- -D warnings

Stack dependency for the battery-ID E2E in dymk/odp-platform-qemu-arm-virt.

Register a distinct 2S fuel gauge as battery 1 so integration tests can verify that callers select a battery by ID.

Assisted-by: GitHub Copilot:gpt-5.6-sol

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: a3539bc8-3a1f-44b3-ba4a-c171a7822595
dymk added 2 commits July 29, 2026 15:59
Include each registered battery ID in update and recovery errors so failures from the two mock gauges are distinguishable.

Assisted-by: GitHub Copilot:gpt-5.6-sol

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: a3539bc8-3a1f-44b3-ba4a-c171a7822595
Apply the platform-common rustfmt layout required by its standalone CI formatting job.

Assisted-by: GitHub Copilot:gpt-5.6-sol

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: a3539bc8-3a1f-44b3-ba4a-c171a7822595
@dymk

dymk commented Jul 29, 2026

Copy link
Copy Markdown
Owner Author

Merged upstream as OpenDevicePartnership#37.

@dymk dymk closed this Jul 29, 2026
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