Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contributes to #46
What did you do?
Refactored tests:
Now all of the tests are under the 'testing' directory. They are further categorized into 'unit' and 'integration' tests.
Removed duplicate test:
'test_post_headers.py' was actually doing a subset of what 'test_mark.py' was doing - so I've deleted 'test_post_headers.py'
Fixed imports:
I've added 'init.py' files to the project directory, as well as the test directories. This allowed me to import stuff 'the python way' (or rather, 'the fastapi way'), removing the need to add filepaths to the system path variable for imports.
Why did you do it?
Appending filepaths to system path variables is not the ideal way to handle imports. Further, organizing all tests under a single directory helps keep imports clean, and can also be beneficial in CI/CD (we can just call 'pytest' in the GitHub Action that compiles the code, and it will run all the tests itself).
How have you tested it?
Ran pytest on the project directory - all tests are running and succeeding
Were docs updated if needed?
Type of change
Checklist: