Skip to content

Update CanvasSpec.js - #1

Open
antoniotp2 wants to merge 29 commits into
mainfrom
antoniotp2-patch-1
Open

Update CanvasSpec.js#1
antoniotp2 wants to merge 29 commits into
mainfrom
antoniotp2-patch-1

Conversation

@antoniotp2

Copy link
Copy Markdown
Owner

Adds test coverage for Canvas _containsPoint, including
points inside polygons, outside polygons, and polygon holes.
The tests were added to the existing CanvasSpec.js file and
adjusted to match the current test style and structure.

jonkoops and others added 29 commits December 25, 2025 18:48
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Co-authored-by: Simon Legner <Simon.Legner@gmail.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Co-authored-by: Simon Legner <Simon.Legner@gmail.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
…eaflet#10053)

Co-authored-by: Levon B <levon.barsegjan@defsecintel.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Co-authored-by: Jon Koops <jonkoops@gmail.com>
Signed-off-by: Iván Sánchez Ortega <ivan@sanchezortega.es>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Add tests for point inside polygon, outside polygon and polygon holes using Canvas renderer.
Fix indentation in Canvas_containsPoint tests
Rename describe block, fix indentation and follow Leaflet test conventions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants