Skip to content

Cover ordering helpers and transaction failure contracts - #93

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
Contributor

Raise line coverage95.93% to99.64%(561/563). Cover generated move methods/setters/shorthands with actual persisted order, membership and endpoint queries, destroying embedded children and roots, nested real sessions, exhausted low-level operation failures and thread-state cleanup, invalid scope arrays, and standalone version loading. Server failures are injected only at the transaction boundary; real integration suite uses an isolated MongoDB8 replica set onlocalhost27041.

387 examples and repository rake pass. No production changes, source exclusions or narrowed tracked files. Two honest fallback lines remain in the handler/scope code.

Accepted CI coverage: 99.64% (561/563 relevant lines), Coveralls build 82425390. (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.

@dblock
dblock force-pushed the dblock/coverage branch 3 times, most recently from db15c69 to d197a8b Compare October 18, 2026 13:28
…ndaries

Exercise persisted generated moves, membership/endpoints, embedded-child and root deletion, nested real transactions, exhausted server retry wrapping/thread cleanup, invalid scope fields, and standalone version loading. Real replica-set suite387 examples passes; coverage rises95.93% to99.64% (561/563), with no production changes or exclusions.

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