Skip to content

[Dart] Improve test coverage - #514

Merged
mpkorstanje merged 3 commits into
cucumber:mainfrom
Fried-man:dart-coverage-enforcement
Aug 31, 2026
Merged

[Dart] Improve test coverage#514
mpkorstanje merged 3 commits into
cucumber:mainfrom
Fried-man:dart-coverage-enforcement

Conversation

@Fried-man

Copy link
Copy Markdown
Contributor

🤔 What's changed?

The dart solution now has full line coverage via unit test by:

  • excluding generated files from coverage reporting
  • adding more tests

Also, the dart CI now enforces full line coverage.

⚡️ What's your motivation?

Outside of best practices, the low line coverage was brought up in this issue.

🏷️ What kind of change is this?

  • 🏦 Refactoring/debt/DX (improvement to code design, tooling, etc. without changing behaviour)

♻️ Anything particular you want feedback on?

N/A

📋 Checklist:

  • I agree to respect and uphold the Cucumber Community Code of Conduct
  • I've changed the behaviour of the code
    • I have added/updated tests to cover my changes.
  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
  • Users should know about my change
    • I have added an entry to the "Unreleased" section of the CHANGELOG, linking to this pull request.

This text was originally generated from a template, then edited by hand. You can modify the template here.

@mpkorstanje
mpkorstanje self-requested a review August 31, 2026 15:11
@mpkorstanje mpkorstanje changed the title [Dart] Full test coverage & CI enforcement [Dart] Improve test coverage Aug 31, 2026
@mpkorstanje
mpkorstanje merged commit b00924f into cucumber:main Aug 31, 2026
55 checks passed
@Fried-man
Fried-man deleted the dart-coverage-enforcement branch August 31, 2026 22:04
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.

2 participants