Skip to content

Conversation

@dacharyc
Copy link
Collaborator

@dacharyc dacharyc commented Nov 6, 2025

This PR builds on #27 and adds the following:

  • Renames the existing test_movie_routes.py file to tests_movie_schemas.py, since those tests are really just testing the Pydantic models
  • Adds a new test_movie_routes.py file with mocks to unit test the mocks. These tests identified a bug we need to resolve, and we'll need to update the tests after we do.
  • Adds a new integration test directory with integration tests against a running FastAPI server instance
  • Adds a README with info about the tests
  • Updates the GitHub workflow to run the unit tests and integration tests as separate steps, and report on them discretely in the summary

Copy link
Collaborator

@shuangela shuangela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm % one nit.

@dacharyc dacharyc force-pushed the add-python-integration-tests branch from 9674022 to 833321f Compare November 7, 2025 00:11
@dacharyc dacharyc merged commit 7e40f17 into mongodb:development Nov 7, 2025
4 checks passed
@dacharyc dacharyc deleted the add-python-integration-tests branch November 7, 2025 00:14
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.

2 participants