Skip to content

test: push Cli line coverage to 95%#9

Merged
Theauxm merged 1 commit intomainfrom
test/exclude-program-from-coverage
May 5, 2026
Merged

test: push Cli line coverage to 95%#9
Theauxm merged 1 commit intomainfrom
test/exclude-program-from-coverage

Conversation

@Theauxm
Copy link
Copy Markdown
Member

@Theauxm Theauxm commented May 5, 2026

Summary

  • Mark the top-level `Program.cs` entry point with `[ExcludeFromCodeCoverage]` via a partial class declaration. The CLI's Main method is never invoked from unit tests (tests target `GenerateCommand.Handle` directly), so it sat as a permanent 0% drag on the total.
  • Add three more OpenAPI parser tests covering: numeric-suffix fallback when operationId collides and no path params can disambiguate, string-enum component schema → ApiEnum, and the cache-hit path when two operations reference the same component schema.

Local line coverage moves from 94.2% to 95.1%.

Test plan

  • `dotnet test` — passing (3 new)
  • `dotnet csharpier check .` — clean

… numeric-collision / string-enum-component / cached-ref paths
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Theauxm Theauxm merged commit a3303c1 into main May 5, 2026
2 checks passed
@Theauxm Theauxm deleted the test/exclude-program-from-coverage branch May 5, 2026 21:29
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.

1 participant