test: implement export missing job id test#3
Conversation
- Implement test `test_export_requires_job_id` in `backend/tests/test_api.py`. - Uncomment the `/api/export` route registration in `backend/app/main.py` so the test successfully returns 422 instead of 404. Co-authored-by: SarmaHighOnCode <218538054+SarmaHighOnCode@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. |
- Ran ruff check and ruff format to sort imports in `backend/app/main.py` which was causing a CI build failure. Co-authored-by: SarmaHighOnCode <218538054+SarmaHighOnCode@users.noreply.github.com>
1 participant
test_export_requires_job_idinbackend/tests/test_api.pyto confirm the /export endpoint returns a 422 error if the job_id parameter is missing./api/exportroute registration inbackend/app/main.py.PR created automatically by Jules for task 16794804536743140839 started by @SarmaHighOnCode