Skip to content

Test infrastructure for api service, and Test Database config in docker compose#7

Merged
gmgrahamgm merged 11 commits into
mainfrom
karl-api_test_infrastructure
Feb 16, 2026
Merged

Test infrastructure for api service, and Test Database config in docker compose#7
gmgrahamgm merged 11 commits into
mainfrom
karl-api_test_infrastructure

Conversation

@KarlFarrar

Copy link
Copy Markdown
Collaborator

This was done really quickly by me so it is very rough sort of still getting used to using python again plz don't judge :)

All documentation for how to use is in README.

Also I didn't write a requirements.txt, but on top of fastapi, sqlalchemy, etc. You'll need to make sure you have pip installed pytest.
I'll make sure to add a requirements.txt to the api services eventually

@gmgrahamgm

gmgrahamgm commented Feb 15, 2026

Copy link
Copy Markdown
Collaborator

The code itself looks good, however, before we merge, let's get rid of the pycache stuff;

  • Delete all __pycache__/ files, commit the changes
  • Add __pycache__/ and *.pyc to .gitignore, commit changes.

Other than that though, this is good to go.

@KarlFarrar

Copy link
Copy Markdown
Collaborator Author

Merged in. Someone do a final check on this PR and we can merge to main. We can use this set up for now, but if we determine we would rather use postman I can look into updating it.

@gmgrahamgm gmgrahamgm merged commit a7efc6c into main Feb 16, 2026
4 checks passed
@gmgrahamgm gmgrahamgm deleted the karl-api_test_infrastructure branch March 17, 2026 16:32
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