Skip to content

Add a unit test for nearly overlapping polygons#329

Merged
mcara merged 1 commit into
spacetelescope:mainfrom
mcara:add-unit-test-for-malformed-poly
Jun 18, 2026
Merged

Add a unit test for nearly overlapping polygons#329
mcara merged 1 commit into
spacetelescope:mainfrom
mcara:add-unit-test-for-malformed-poly

Conversation

@mcara

@mcara mcara commented Jun 17, 2026

Copy link
Copy Markdown
Member

Add a unit test that is known to fail with the current code when trying to intersect almost perfectly overlaping polygons resulting in a MalformedPolygonError. The test is marked with XFAIL. Improvements to the code should result in this test not XFAIL-ing.

@mcara mcara self-assigned this Jun 17, 2026
@mcara mcara requested a review from a team as a code owner June 17, 2026 06:12
@mcara mcara added the testing Anything related to testing label Jun 17, 2026
@mcara mcara requested a review from zacharyburnett June 17, 2026 06:13
@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (bb9a3c5) to head (9b98db0).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #329   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           10        10           
=========================================
  Hits            10        10           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mcara mcara merged commit 5055a98 into spacetelescope:main Jun 18, 2026
23 checks passed
@mcara mcara changed the title Add a unit test for nearly overlaping polygons Add a unit test for nearly overlapping polygons Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Anything related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant