Skip to content
This repository was archived by the owner on Jan 27, 2026. It is now read-only.

New testers for Tagging Mission#158

Open
Hector-c0de wants to merge 2 commits intomainfrom
new_tagging_mission_testers
Open

New testers for Tagging Mission#158
Hector-c0de wants to merge 2 commits intomainfrom
new_tagging_mission_testers

Conversation

@Hector-c0de
Copy link
Contributor

Files added

tagging_mission_tester.py:

New tester for tagging mission with clean ROS integration

tagging_mission_tester_2.py:

Updated original tester (tagging_mission_test.py) in case tagging_mission_tester.py does not work

Key changes:

  • Replaced unittest framework with continuous mission simulation
  • Simplified ROS mocking using clean module-level mocking instead of per-test setup
  • Added time-based progression with realistic 20+ second mission timeline
  • Integrated torpedo firing with manual triggers at t=12s and t=16s for guaranteed testing
  • Real scoring validation that tests complete scoring system with all bonuses
  • Comprehensive object simulation using progressive detection (board to animals to openings to bars)

Removed Complexity:

  • Individual test methods replaced with single integrated test run
  • Complex mock assertions replaced with simple functional validation
  • Static test data replaced with dynamic time-based simulation

Test Coverage

Both testers validate:

  • Mission phase progression (6 phases)
  • Object detection and configuration
  • Torpedo firing simulation (full + partial hits)
  • Scoring system (225 points with bonuses)
  • Board configuration detection
  • Far distance bonus calculation

…nd tagging_mission_tester_2.py with comprehensive ROS mocking for environments without ROS.
@Hector-c0de Hector-c0de added this to the Summer-Delivarables milestone Jun 24, 2025
@Hector-c0de Hector-c0de requested a review from Cruiz102 June 24, 2025 22:40
@Hector-c0de Hector-c0de self-assigned this Jun 24, 2025
@Hector-c0de Hector-c0de added testing CV Computer vision tasks labels Jun 24, 2025
@Hector-c0de Hector-c0de marked this pull request as ready for review June 25, 2025 21:45
@Cruiz102 Cruiz102 requested a review from javier-rivera8 June 29, 2025 17:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CV Computer vision tasks testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant