Skip to content

test: assert hook called with expected message#454

Merged
werk24 merged 1 commit into
mainfrom
codex/modify-test_read_drawing_with_hooks-to-use-mock
Aug 14, 2025
Merged

test: assert hook called with expected message#454
werk24 merged 1 commit into
mainfrom
codex/modify-test_read_drawing_with_hooks-to-use-mock

Conversation

@jmtts
Copy link
Copy Markdown
Collaborator

@jmtts jmtts commented Aug 14, 2025

Summary

  • use AsyncMock to capture hook call in test_read_drawing_with_hooks
  • assert the hook was invoked once with TechreadMessageType.ASK

Testing

  • pytest tests/test_client.py::test_read_drawing_with_hooks -q (fails: InvalidLicenseException: The provided license is invalid or has expired.)

https://chatgpt.com/codex/tasks/task_e_689dbcb31adc83329ded47f07f878ac4

@werk24 werk24 merged commit c182114 into main Aug 14, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants