pygame-ce icon indicating copy to clipboard operation
pygame-ce copied to clipboard

Test function for pygame.draw.arc (resolves #2636)

Open Houda222 opened this issue 1 year ago • 2 comments
trafficstars

This pull request resolves issue #2636

I added a test function test_arc__correct_drawing that replaced the todo_test_arc function in the DrawArcMixin class. It ensures that pygame.draw.arc works correctly, by checking whether the drawn arc matches the ellipse defined by the bounding rectangle. The comparison is done by measuring the difference between the ellipse's points and the arc's point and making sure it's less than a certain threshold. The test is done on different bounding rectangles.

Houda222 avatar Dec 28 '23 13:12 Houda222

The circleci fail on this PR is not this PRs fault, don't worry. It's being resolved in another PR

ankith26 avatar Dec 28 '23 18:12 ankith26

Hi @Houda222, do you think you will make any of the changes asked for? Or should we pass this PR over to someone else to finish off?

MyreMylar avatar May 31 '24 10:05 MyreMylar