Skip to content

Cover cursor boundary contracts and floating-point round trips - #53

Merged
dblock merged 1 commit into
mongoid:masterfrom
dblock:dblock/coverage
Oct 18, 2026
Merged

dblock merged 1 commit into
mongoid:masterfrom
dblock:dblock/coverage

Conversation

@dblock

@dblock dblock commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator

Add real float cursor round trips, unsupported serialization type errors, base-class subclass contracts, inclusive descending tiebreaker selectors, and an isolated version entry-point test.220 examples pass; changed specs pass lint. No production changes or source exclusions.

Default-branch baseline97.52%; fresh final local report is in the validation log and CI will report the tracked library result. Existing full-project lint has unrelated Gemfile spacing/order offenses; changed specs are clean.

Accepted CI coverage: 99.65% (284/285 relevant lines), Coveralls build 82425286. (Historical report before version-test simplification; not the revised head.)

Prepared by an AI on behalf of @dblock.

Update: version tests now assert the already-loaded public constant directly. Removed constant hiding/replacement, version-file reloading, and manual cleanup used only to count assignments under coverage. Any coverage decrease is intentional; earlier percentages above predate this simplification. Genuine compatibility and integration tests remain unchanged.

@github-actions

github-actions Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

Exercise public cursor float round trips, invalid serialization types, base-class subclass contracts, inclusive descending tiebreakers, and isolated standalone version loading. Real suite increases measured line coverage from97.52% to near100 without production changes or exclusions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock merged commit 12d1dff into mongoid:master Oct 18, 2026
16 checks passed
@dblock
dblock deleted the dblock/coverage branch October 18, 2026 13:47
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