Skip to content

test: add end-to-end Q7 map_content.refresh coverage#6

Draft
arduano wants to merge 1 commit intomainfrom
leo/pr785-followup-q7-mapcontent-e2e
Draft

test: add end-to-end Q7 map_content.refresh coverage#6
arduano wants to merge 1 commit intomainfrom
leo/pr785-followup-q7-mapcontent-e2e

Conversation

@arduano
Copy link
Copy Markdown
Owner

@arduano arduano commented Mar 31, 2026

Openclaw:

Summary

  • replace the Q7 map_content.refresh() test that mocked B01MapParser.parse
  • use a real encrypted MAP_RESPONSE payload built from the checked-in SCMap fixture
  • assert the full parse/render/cache path by checking PNG output, parsed map_data, and room-name extraction

Why

PR Python-roborock#785 deferred concern: existing test didn't exercise the real Q7 map-content decode/parse/render path.

Validation

  • . .venv/bin/activate && pytest -q tests/devices/traits/b01/q7/test_map_content.py tests/map/test_b01_map_parser.py
  • ruff check tests/devices/traits/b01/q7/test_map_content.py

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