Add unit tests for nav2, person follow, and slam hooks

[openminddev]

This pull request adds comprehensive unit tests for the nav2_hook and person_follow_hook modules. The new tests cover a wide range of scenarios for both starting and stopping navigation and person-following features, including success cases, error handling, custom parameters, and correct use of timeouts and dependencies.

Unit tests for nav2_hook:

• Added tests for start_nav2_hook covering successful starts (with default and custom parameters), HTTP error responses (with and without JSON), client connection errors, and timeout configuration. Also verifies integration with the ElevenLabsTTSProvider for status messaging.
• Added tests for stop_nav2_hook covering successful stops (with default and custom parameters), HTTP error responses, client connection errors, and timeout usage.

Unit tests for person_follow_hook:

• Added tests for start_person_follow_hook including first-attempt and retry successes, handling of not-tracked status, custom base URLs, client and connection errors, default constant usage, and timeout configuration. Also verifies correct messaging via ElevenLabsTTSProvider.
• Added tests for stop_person_follow_hook covering successful stops (default and custom URLs), HTTP and client errors, and timeout configuration.

[Copilot]

Pull request overview

This pull request adds comprehensive unit test coverage for three hook modules: nav2_hook, person_follow_hook, and slam_hook. The tests validate both successful operations and error handling scenarios for starting and stopping navigation, person-following, and SLAM functionalities.

Changes:

• Added 234 lines of tests for nav2_hook covering start/stop operations with various success and error scenarios
• Added 374 lines of tests for person_follow_hook covering enrollment, tracking status, retries, and error conditions
• Added 309 lines of tests for slam_hook covering start/stop operations including map saving functionality

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File	Description
tests/hooks/test_nav2_hook.py	Tests for nav2 start/stop operations including HTTP errors, client errors, custom parameters, and timeout configurations
tests/hooks/test_person_follow_hook.py	Tests for person follow enrollment/tracking with retry logic, status polling, connection errors, and timeout handling
tests/hooks/test_slam_hook.py	Tests for SLAM start/stop operations including map saving, HTTP/client errors, and timeout configurations

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!