The test directory contains several instances where .value is added to HTTPStatus unnecessarily. One of the instance is:-
https://github.com/anitab-org/mentorship-backend/blob/df8e4a4a3c167f01750ded0c4a89fc5a3ab0b32d/tests/task_comments/test_api_delete_task_comment.py#L38
Here, .value can be removed as both enum and int representations of status codes are equal when asserted.
You can check the following document for test cases:
.value are removed from the tests directory.2 hours
I would like to work on this issue
@devkapilbansal since this is maintenance and not a new feature I think it can be assigned.
@Ishaandham19 assigning you. Happy coding.
Since this is my first commit, I want to get some feedback on the process I will take to solve this issue
1) Fork -> clone -> branch -> "change code" - > push -> pull request
2) For the "change code": I would be going through all the .py files in the "tests" directory and changing instances of .value added to HTTPStatus in the assertEqual method
Since this is my first commit, I want to get some feedback on the process I will take to solve this issue
- Fork -> clone -> branch -> "change code" - > push -> pull request
- For the "change code": I would be going through all the .py files in the "tests" directory and changing instances of .value added to HTTPStatus in the assertEqual method
Yes somewhat like that. Although you missed a few steps. You can go through the Readme for the setup process.
If you do face any problems feel free to ask here or on Zulip. : )
This is my first commit, I read the issue but somewhat I understood. So can someone help me out what is the issue.
Thank you