Skip to content

test: add firmware servo speed coverage#75

Open
Pro777 wants to merge 1 commit into
m5stack:mainfrom
Spitfire-Cowboy:rowan/upstream-servo-test-coverage
Open

test: add firmware servo speed coverage#75
Pro777 wants to merge 1 commit into
m5stack:mainfrom
Spitfire-Cowboy:rowan/upstream-servo-test-coverage

Conversation

@Pro777
Copy link
Copy Markdown

@Pro777 Pro777 commented May 19, 2026

Summary

  • extract the servo speed-to-spring profile math into a small pure helper
  • add a host-side CMake/CTest harness for speed clamping and spring profile coverage
  • document the local firmware test command in firmware/README.md

Why

This adds firmware-side coverage for servo motion tuning logic without introducing repo-wide tooling or touching the Flutter/server surfaces.

Test

  • cmake -S tests -B build-host-tests
  • cmake --build build-host-tests
  • ctest --test-dir build-host-tests --output-on-failure

@Pro777
Copy link
Copy Markdown
Author

Pro777 commented May 19, 2026

日本語要約:

  • サーボ速度からスプリング特性を求める計算を純粋なヘルパー関数として切り出しました。
  • ホスト側で実行できる CMake/CTest のテストを追加し、速度クランプとスプリング特性のカバレッジを強化しました。
  • firmware/README.md にローカルでのテスト実行手順を追記しました。

このPRは、Flutter やサーバー側には触れず、レビューしやすい最小スコープでファームウェアのテスト性を改善するものです.

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