Skip to content

Standardize Rust test validation - #734

Open
mehmetozguldev wants to merge 4 commits into
mainfrom
rust-test-standardization
Open

Standardize Rust test validation#734
mehmetozguldev wants to merge 4 commits into
mainfrom
rust-test-standardization

Conversation

@mehmetozguldev

Copy link
Copy Markdown
Member

Rust testing was spread across modules and the shared Rust gate did not run Clippy or tests, so local, CI, and release validation could drift.

  • Standardizes the Rust gate around formatting, all-target checks, Clippy, and workspace tests
  • Documents where unit and integration tests belong
  • Moves oversized installer and Docker suites into dedicated test modules
  • Adds focused debugger, project, and remote coverage

How to test:

  1. Run bun check:rust.
  2. Confirm all workspace checks and tests pass.

Screenshots

Not applicable.

@mehmetozguldev
mehmetozguldev marked this pull request as ready for review August 2, 2026 15:39
@mehmetozguldev mehmetozguldev self-assigned this Aug 2, 2026
@mehmetozguldev mehmetozguldev added the rust Pull requests that update rust code label Aug 2, 2026
Comment thread crates/github/src/api.rs Outdated
@mehmetozguldev
mehmetozguldev force-pushed the rust-test-standardization branch 4 times, most recently from 3237a34 to ec40945 Compare August 5, 2026 00:45
Run formatting, all-target checks, Clippy, and workspace tests through the shared Rust gate used by CI and release preflight.

Document the placement convention, extract oversized unit test modules, and add coverage for debugger, project, and remote behavior.
Set the shared threshold for existing command APIs and remove redundant per-function allowances so Clippy behavior stays consistent across the workspace.
Raise the centralized command argument threshold for the expanded remote
terminal API. Use the idiomatic issue-state pattern so the shared Clippy
gate passes without function-local suppressions.
@mehmetozguldev
mehmetozguldev force-pushed the rust-test-standardization branch from ec40945 to 7421e20 Compare August 10, 2026 15:07
Keep the menu module out of Linux builds where the platform-specific menu
configuration does not reference it. This preserves the current terminal
session import while keeping the shared Clippy gate warning-free.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants