-
Notifications
You must be signed in to change notification settings - Fork 162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add/fix unit test(s) to check HTTP status codes for invalid lookups #691
Comments
Hi I would love to work on this issue :) |
Hey @sharktankful, I missed your comment somehow. Feel free to take it! |
And let me know if you need any help. |
This issue has been inactive for 345 hours (14.38 days) and will be unassigned after 63 more hours (2.63 days). If you have questions, please leave a comment, message @codethesaurus or @geekygirlsarah on Twitter, or email [email protected].If you are still working on this issue, that's fine. Please comment here to tell the bot to give you more time. |
This issue has been inactive for 417 hours (17.38 days) and is past the limit of 408 hours (17.00 days) so is being unassigned.You’ve just been unassigned from this ticket due to inactivity – but feel free to pick it back up (or a new one!) in the future! Thank you for your interest in contributing to this project. |
Description
Bug #690 resulted from a invalid
language_name
/friendly_language_name
mixup that would result in 500 server errors. The unit test that seem to check for invalid language/structures didn't catch this.This issue should add new unit test(s) or fix the existing unit test(s) to make this more robust.
Requirements
Additional Notes
This need to be tested locally to ensure they work:
python manage.py test
should run all unit testspython manage.py validatelanginfofiles
should run tests on the language filespython manage.py validatemetainfofile
should run tests on the meta info fileWhile they'll be run when a pull request is made, making sure they run and pass locally first will speed things up.
The text was updated successfully, but these errors were encountered: