Skip to content

test: cover OpenAPI parser collision and empty-ref edge cases#13

Merged
Theauxm merged 1 commit intomainfrom
test/raise-coverage-above-95
May 6, 2026
Merged

test: cover OpenAPI parser collision and empty-ref edge cases#13
Theauxm merged 1 commit intomainfrom
test/raise-coverage-above-95

Conversation

@Theauxm
Copy link
Copy Markdown
Member

@Theauxm Theauxm commented May 6, 2026

Summary

  • Adds a fixture and four targeted parser tests for OpenAPI edges that were not previously hit:
    • Synthesized operation name colliding with a component schema (verb-prefix path)
    • Operation-id collision where path-param disambiguation runs out and falls back to a numeric suffix
    • Empty component type referenced directly and inside a list (rewriting to object / List<object>)
    • Inline-promoted object name collision producing a numeric suffix via EnsureUniqueName
  • Lifts local Cobertura line coverage from 95.99% to 96.39% (97.30% excluding generated regex files).

Test plan

  • dotnet test (293 passed, 0 failed)
  • dotnet build zero new warnings
  • dotnet csharpier format . clean
  • Cobertura report shows the targeted lines now covered (233, 692, 637/647, 708-713)

…tion fallback, empty-type ref rewriting, and inline-promoted name suffix
@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Theauxm Theauxm merged commit 1d0b236 into main May 6, 2026
2 checks passed
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