Skip to content

test: cover EtlPipelineProgress value semantics (fix coverage gate) - #279

Merged
Chris-Wolfgang merged 1 commit into
vNextfrom
test/etlpipelineprogress-coverage
Jul 20, 2026
Merged

test: cover EtlPipelineProgress value semantics (fix coverage gate)#279
Chris-Wolfgang merged 1 commit into
vNextfrom
test/etlpipelineprogress-coverage

Conversation

@Chris-Wolfgang

Copy link
Copy Markdown
Owner

Fixes the Stage 1 coverage-gate failure on #278: EtlPipelineProgress was at 83% because the positional record's synthesized value-semantics members were untested. Adds one focused test exercising equality (Equals/==/!=/GetHashCode), Deconstruct, with, ToString, and the property accessors. Coverage now 100% locally; 277 tests pass. Merge into vNext → #278 re-runs green → then vNext→main → re-cut v0.16.0.

The Stage 1 coverage gate failed on #278 (vNext->main) at EtlPipelineProgress
83% — the positional record's synthesized members (Equals/GetHashCode/==/!=/
Deconstruct/ToString/with) were never exercised. Add a focused value-semantics
test; EtlPipelineProgress coverage is now 100% locally. 277 tests pass.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 20, 2026 13:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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