Skip to content

fix(tests): rustfmt layout for corpus_cli_dispatch (rust-ci fmt red on #418) - #425

Merged
hyperpolymath merged 1 commit into
mainfrom
arena/4cfe4690-echidna
Oct 9, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
arena/4cfe4690-echidna

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

Follow-up to #418 (merged). Its `Rust CI` run on `main` (7fd369d) failed at Cargo fmt; Cargo check passed, clippy was skipped.

This PR changes only `tests/corpus_cli_dispatch.rs` layout to match rustfmt:

  • `SUPPORTED` becomes a whitespace-separated string constant (rustfmt packs literal arrays into mixed rows)
  • one `assert!` whose arguments exceed `fn_call_width` is split vertically

No behaviour change.

Notes

  • Job logs were unavailable from the sandbox (blob download EOF), so the fmt diagnosis comes from the per-step result (`Cargo fmt` = failure, `Cargo check` = success) plus a manual check against rustfmt rules. rustfmt itself could not be run here (crates.io and static.rust-lang.org are not reachable).
  • Clippy and the test job have not run on this code yet; they are gated behind fmt in the same job, so CI is the next check.

rust-ci's `cargo fmt --check` failed on #418 (main 7fd369d). Two layout
spots did not match rustfmt output:

- the SUPPORTED array: rustfmt packs literal arrays into mixed rows;
  use a single whitespace-separated string constant instead, iterated
  with split_whitespace();
- an assert! whose argument list exceeds fn_call_width (60): split it
  vertically.

No behaviour change.

Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 330f19e1-1c26-4e17-af86-08d4ac7df224

📥 Commits

Reviewing files that changed from the base of the PR and between 7fd369d and c5a1e4c.


📒 Files selected for processing (1)
  • tests/corpus_cli_dispatch.rs


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hyperpolymath
hyperpolymath merged commit f6ce4c9 into main Oct 9, 2026
68 of 73 checks passed
@hyperpolymath
hyperpolymath deleted the arena/4cfe4690-echidna branch October 9, 2026 19:33
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.

2 participants