Skip to content
This repository was archived by the owner on Mar 18, 2026. It is now read-only.

Backend cleanup#8

Merged
pheonix8 merged 5 commits into
mainfrom
dev/test-clean-up
May 18, 2025
Merged

Backend cleanup#8
pheonix8 merged 5 commits into
mainfrom
dev/test-clean-up

Conversation

@pheonix8
Copy link
Copy Markdown
Collaborator

  • updated tests:

    • clean test structure
    • remove unneccesary test (auth error)
    • add tests for 500 error No DB
    • added root endpoint test
  • updated endpoints:

    • removed /api now handled via root_path (dockerfile)
    • standardised enpoints to /plans
    • added try except to plan endpoints
  • updated database:

    • added sqlalchemie sessionmaker
    • cleaned get_session function
  • updated linting:

    • removed F401 from linting exceptions
    • # noqa: F401 can be used to exclude lines like in create_tables.py

@pheonix8 pheonix8 requested a review from Matti3939 May 17, 2025 16:08
@pheonix8 pheonix8 merged commit 0865aa6 into main May 18, 2025
2 checks passed
@pheonix8 pheonix8 deleted the dev/test-clean-up branch May 18, 2025 16:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants