🧪 test: add error test for GET /api/search no parameters#9
Conversation
- Created __tests__/search.test.js - Added mock for passport to bypass auth checks in test mode - Added a test for missing `name` and `id` parameters to expect 400 response and correct error message - Added a test for empty string values for `name` and `id` parameters to verify 400 response Co-authored-by: deitaur <113350206+deitaur@users.noreply.github.com>
|
👋 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. |
1 participant
🎯 What: Added missing testing for the
GET /api/searchendpoint whennameandidare missing or empty.📊 Coverage: The scenarios covered are: completely missing parameters and empty string parameters. The tests also verify the exact error message that should be returned.
✨ Result: Test coverage for this endpoint's validation logic has been improved and regression-preventing assertions have been added.
PR created automatically by Jules for task 5722200544000238980 started by @deitaur