Skip to content

Cover relevance search and Rails indexing; fix relevance scores - #156

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

Baseline87.67% default-branch coverage. Enable the three previously skipped relevance tests against real MongoDB; fix the uncovered bugs they expose: reduction returned arrays instead of numeric counts and sorting put least relevant results first. Restore numeric relevance2,1 and persisted result IDs. Map-reduce remains deprecated upstream; no aggregation rewrite here.

Test configuration, ANSI/silent output, and the actual Railtie registration/index rake task, including indexing persisted documents. Add railties as a development-only dependency with a compatible range so each existing Mongoid matrix entry resolves a supported Rails release. Eager loading is mocked only at the host-application boundary.

65 passing examples, clean lint,100% CI line coverage(154/154) with no source exclusions or narrowed files.

Accepted CI coverage: 100.00% (154/154 relevant lines), Coveralls build 82425285.

Prepared by an AI on behalf of @dblock.

@github-actions

github-actions Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

@dblock
dblock force-pushed the dblock/coverage branch 6 times, most recently from 3561df3 to 081096d Compare October 18, 2026 14:03
Restore real map-reduce regressions and fix their scalar score reduction and descending ranking. Cover configuration, console output, and Rails rake registration/execution with real persisted documents. Library line coverage rises from87.67% to99.35% without exclusions; Rails integration uses an explicit development dependency.

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