Skip to content

Conversation

@YuriiMotov
Copy link
Member

This PR moves the backend tests from backend/app/tests to backend/tests.

Motivation:

  • It's the conventional location for Python tests
  • Currently, test files are included into the coverage report that makes it harder to analyze

Related discussion: #1555

@YuriiMotov YuriiMotov requested a review from alejsdev September 8, 2025 14:47
@YuriiMotov YuriiMotov changed the title 🚚 Move backend tests outside the app dirrectory 🚚 Move backend tests outside the app directory Sep 8, 2025
Copy link
Member

@alejsdev alejsdev left a comment

Choose a reason for hiding this comment

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

Just a minor thing

Copy link
Member

@tiangolo tiangolo left a comment

Choose a reason for hiding this comment

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

Nice, thank you @YuriiMotov! 🚀

And thanks @alejsdev for the review! 🍰

@tiangolo tiangolo merged commit 0b473f5 into fastapi:master Sep 20, 2025
16 checks passed
jpizquierdo pushed a commit to jpizquierdo/full-stack-fastapi-template that referenced this pull request Sep 24, 2025
Co-authored-by: Alejandra <90076947+alejsdev@users.noreply.github.com>
rajdavee pushed a commit to OrganyzAI/python-backend that referenced this pull request Dec 30, 2025
Co-authored-by: Alejandra <90076947+alejsdev@users.noreply.github.com>
azzi2023 pushed a commit to azzi2023/organyz-python-backend that referenced this pull request Jan 9, 2026
Co-authored-by: Alejandra <90076947+alejsdev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants