Repository navigation
better rect tests using literal values - #3190
Conversation
bilhox
changed the title
|
Hi antoine, since you're new to contributing, let me give you one little advice on how you can improve your PRs next time. You don't have to mention the link of the issue in the title, you can mention it in your message directly. With this, you can opt for a more descriptive title for your PR. |
yunline
added
tests
|
I think there is a little quiproquo, when I talked about tests for literal values, I meant for rect attributes, not these ones. |
|
Hello @bilhox, I think I don't really understand what should be done. Could you explain more specifically, maybe ? |
What I meant is we should have a test similar to |
…attributes of a rect, in rect_tests.py
ankith26
left a comment
ankith26
left a comment
There was a problem hiding this comment.
Sorry for taking this long to get to reviewing your PR. I agree, literal tests do improve the quality of the tests.
Thanks for contributing to pygame-ce! 🎉
Fix the following functions in rect_test to use literal values for attribute checks : test_inflate__larger, test_inflate__smaller, test_inflate_ip__larger and test_inflate_ip__smaller.