Skip to content

Add test coverage for SKPath.ConvertConicToQuads#3999

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/add-test-coverage-convert-conic-to-quads
Draft

Add test coverage for SKPath.ConvertConicToQuads#3999
Copilot wants to merge 2 commits into
mainfrom
copilot/add-test-coverage-convert-conic-to-quads

Conversation

Copilot AI commented May 15, 2026

Copy link
Copy Markdown
Contributor

Description of Change

SKPath.ConvertConicToQuads has three public overloads with zero test coverage. Adds tests verifying:

  • Point array length matches formula 2 * (1 << pow2) + 1
  • Quad count return value from the out overload
  • Start/end points are preserved in output
  • ArgumentNullException on null pts array

Bugs Fixed

None.

API Changes

None.

Behavioral Changes

None.

Required skia PR

None.

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Merged related skia PRs
  • Changes adhere to coding standard
  • Updated documentation

Agent-Logs-Url: https://github.com/mono/SkiaSharp/sessions/02291817-9d32-4bd2-810e-2c85b3a5ef36

Co-authored-by: mattleibow <1096616+mattleibow@users.noreply.github.com>
Copilot AI changed the title [WIP] Add test coverage for SKPath.ConvertConicToQuads Add test coverage for SKPath.ConvertConicToQuads May 16, 2026
Copilot AI requested a review from mattleibow May 16, 2026 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[fix-finder] Add test coverage for SKPath.ConvertConicToQuads

2 participants