Skip to content

Move ruby_indexer tests to separate file#705

Open
alexcrocha wants to merge 1 commit intomainfrom
03-26-move_ruby_indexer_tests_to_separate_file
Open

Move ruby_indexer tests to separate file#705
alexcrocha wants to merge 1 commit intomainfrom
03-26-move_ruby_indexer_tests_to_separate_file

Conversation

@alexcrocha
Copy link
Copy Markdown
Contributor

@alexcrocha alexcrocha commented Mar 26, 2026

Part of #649

This PR moves the tests from ruby_indexer.rs into ruby_indexer_tests.rs, separating test code from the indexer implementation as the file was getting too large at over 6k lines.

This is the base for follow-up PRs in this stack which will group the relocated tests into submodules.

Extract all tests from ruby_indexer.rs into ruby_indexer_tests.rs
using #[path]. Nest name_dependent_tests as a submodule with
use super::*.
Copy link
Copy Markdown
Contributor Author

alexcrocha commented Mar 26, 2026

@alexcrocha alexcrocha marked this pull request as ready for review March 26, 2026 23:56
@alexcrocha alexcrocha requested a review from a team as a code owner March 26, 2026 23:56
@alexcrocha alexcrocha self-assigned this Mar 26, 2026
Copy link
Copy Markdown
Contributor

@soutaro soutaro left a comment

Choose a reason for hiding this comment

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

👍

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.

3 participants