Skip to content

style: rustfmt the reference-edge tests - #2

Merged
prom3theu5 merged 1 commit into
mainfrom
fix/fmt
Jun 1, 2026
Merged

style: rustfmt the reference-edge tests#2
prom3theu5 merged 1 commit into
mainfrom
fix/fmt

Conversation

@prom3theu5

Copy link
Copy Markdown
Member

Fixes the CI failure on the merge of #1: the cargo fmt --check gate flagged the new tests in tests/unit.rs, tests/cli.rs, and tests/ladybug_smoke.rs, which were written with long single-line statements rather than rustfmt's wrapped form.

Formatting only — no behaviour change. Verified locally: cargo fmt --check, cargo clippy --all-targets, and cargo test (75 tests) all pass.

The new tests in unit.rs/cli.rs/ladybug_smoke.rs were not rustfmt-formatted,
failing the CI `cargo fmt --check` gate. Formatting only; no behaviour change.
@qodo-code-review

Copy link
Copy Markdown

Review Summary by Qodo

Apply rustfmt formatting to reference-edge tests

🧪 Tests

Grey Divider

Walkthroughs

Description
• Apply rustfmt formatting to test files
• Reformat long single-line statements across multiple lines
• Fix CI cargo fmt --check gate failures
• No behavior changes, formatting only
Diagram
flowchart LR
  A["Test files<br/>unit.rs, cli.rs,<br/>ladybug_smoke.rs"] -- "Apply rustfmt<br/>formatting" --> B["Formatted code<br/>with wrapped<br/>statements"]
  B -- "Pass CI gate" --> C["cargo fmt --check<br/>succeeds"]

Loading

Grey Divider

File Changes

1. tests/cli.rs Formatting +12/-2

Reformat pack command and assertions

• Reformat pack command arguments from single line to multi-line format
• Wrap assert! macro call with status.contains("\"referenceEdges\"") across multiple lines
• Improve readability of long function call chains

tests/cli.rs


2. tests/ladybug_smoke.rs Formatting +10/-8

Reformat store operations and assertions

• Reformat store.upsert_symbol() calls to wrap across multiple lines
• Reformat store.symbol_references() call with method chaining
• Wrap assert_eq! macro to improve readability

tests/ladybug_smoke.rs


3. tests/unit.rs Formatting +72/-11

Reformat file operations and index calls

• Reformat std::fs::write() calls with long string arguments across multiple lines
• Reformat index_repo() function calls with multiple parameters
• Wrap related.iter().any() closure across multiple lines
• Reformat assert! macro calls for better readability
• Remove trailing blank line at end of file

tests/unit.rs


Grey Divider

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented Jun 1, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@prom3theu5
prom3theu5 merged commit 104b2e6 into main Jun 1, 2026
1 check passed
@prom3theu5
prom3theu5 deleted the fix/fmt branch June 1, 2026 20:27
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