Skip to content

Add message validation tests #3665

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add message validation tests #3665

wants to merge 2 commits into from
+221 −0

Conversation

@faizan842
Copy link
Contributor

Add comprehensive tests for message validation in Memory and AsyncMemory classes

  • Message type validation (string, dict, list)
  • Session ID validation (user_id, agent_id, run_id)
  • Memory type validation
  • AsyncMemory validation
  • Message parsing utilities
  • Vision message parsing

All tests pass and cover essential validation scenarios.
@faizan842
Add message validation tests
90ef50f 
- Add tests for message type validation (string, dict, list)
- Add tests for session ID validation (user_id, agent_id, run_id)
- Add tests for memory type validation
- Add tests for AsyncMemory validation
- Add tests for message parsing utilities
- Add tests for vision message parsing
@faizan842
Fix linting error: remove unused Mock import
8f01fc2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@faizan842