thib8956/new-shapes
1
0
Fork 0
Code Issues 7 Pull Requests 7 Actions Wiki Activity

test: add contains(Point) hit-testing tests for all shapes #21

Merged
thib8956 merged 1 commits from issue-11/contains-point-tests into master 2026-03-27 23:28:59 +00:00
Conversation 0 Commits 1 Files Changed 1 +387
thib8956 commented 2026-03-27 15:37:56 +00:00
Owner
Copy Link

Summary

  • Add 36 unit tests covering bounds-based hit-testing (getBounds().contains(Point)) for all five shape types: SRectangle, SCircle, STriangle, SText, SCollection.
  • Tests cover: point inside, outside, on boundary (inclusive top-left, exclusive bottom-right per java.awt.Rectangle semantics), and point after translate.
  • SCollection tests verify union bounds, gap between children, and single-child bounds equality.

Closes #11

## Summary - Add 36 unit tests covering bounds-based hit-testing (`getBounds().contains(Point)`) for all five shape types: SRectangle, SCircle, STriangle, SText, SCollection. - Tests cover: point inside, outside, on boundary (inclusive top-left, exclusive bottom-right per `java.awt.Rectangle` semantics), and point after translate. - SCollection tests verify union bounds, gap between children, and single-child bounds equality. Closes #11
thib8956 added 1 commit 2026-03-27 23:25:32 +00:00
test: add contains(Point) hit-testing tests for all shapes
All checks were successful
CI / build-and-test (pull_request) Successful in 16s
Details
740572bfb7 
Add 36 tests covering bounds-based hit-testing (getBounds().contains(Point))
for SRectangle, SCircle, STriangle, SText, and SCollection:
- Point inside, outside, on boundary (inclusive top-left, exclusive bottom-right)
- Point after translate to verify position update
- SCollection union bounds, gap between children, single-child bounds

Closes #11
thib8956 force-pushed issue-11/contains-point-tests from b87deecf40 to 740572bfb7 2026-03-27 23:25:32 +00:00 Compare
thib8956 merged commit bf01bae17f into master 2026-03-27 23:28:59 +00:00
thib8956 deleted branch issue-11/contains-point-tests 2026-03-27 23:28:59 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
thib8956
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: thib8956/new-shapes#21
Delete Branch "issue-11/contains-point-tests"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?