Skip to content

Conversation

@niklashigi
Copy link
Member

@niklashigi niklashigi commented Dec 18, 2025

Note

Standardizes test mocks to use status_code, simplifies request body capture and respx lifecycle, and adds a POST body serialization test.

  • Test utilities (tests/conftest.py):
    • Standardize mocked response key to status_code and set default headers for JSON bodies.
    • Simplify request body capture to parse JSON from request.content for non-GET requests.
    • Manage respx via an autouse reset_respx fixture; MockContext.clear() now only clears routes and captured requests.
  • Tests:
    • Migrate all mocks to status_code and remove redundant isinstance assertions in error tests.
    • Add test_should_correctly_serialize_post_request_body validating POST body serialization.
    • Minor wording fix in test_job_board docstring.

Written by Cursor Bugbot for commit c71f6c7. This will update automatically on new commits. Configure here.

@niklashigi niklashigi merged commit 4899279 into main Dec 18, 2025
2 checks passed
@niklashigi niklashigi deleted the test/core/clean-up-structure branch December 18, 2025 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant