🧪 [test] Add unit tests for classifier prompt generation#9
Conversation
Implemented unit tests for `src/prompts/classifier.py` to verify: - System prompt structure and sections. - Inclusion of agent-specific keywords. - Proper injection of classification examples. - User query formatting logic. The tests use a robust mocking strategy to allow execution in environments where external dependencies like `pydantic` or `langchain` are not installed.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
2 participants
This PR addresses the testing gap for the classifier prompt generation logic.
🎯 What: Added comprehensive unit tests for
src/prompts/classifier.py.📊 Coverage:
build_system_prompt(): Verified structure, keywords injection, and examples rendering.pack_classification_query(): Verified wrapping of user input.✨ Result: Improved reliability and test coverage for the core intent routing prompt logic. The tests are isolated and handle missing dependencies through scoped mocking.
PR created automatically by Jules for task 7434396178463211647 started by @ishaanxgupta