Skip to content

test(cli): cover noun-first object parser#137

Draft
overtrue wants to merge 1 commit intomainfrom
codex/test-gap-detection-202604171936
Draft

test(cli): cover noun-first object parser#137
overtrue wants to merge 1 commit intomainfrom
codex/test-gap-detection-202604171936

Conversation

@overtrue
Copy link
Copy Markdown
Contributor

Related issue

Automation run: rustfs/cli test gap detection.

Background

The noun-first rc object ... command group was added to make object workflows easier to discover, but the parser coverage on origin/main only exercised the bucket CORS path. That left several object subcommand delegations untested, so regressions in option wiring could silently break expected object commands.

Root cause

The object command group delegates to existing command argument structs, but there were no parser tests proving that the noun-first routes preserve important flags and positional arguments for copy, move, show, head, find, and tree.

Solution

Added focused parser tests for rc object copy, rc object move, rc object show, rc object head, rc object find, and rc object tree. The tests assert that representative positional arguments and options land in the expected delegated command variants. I also added a minimal make pre-commit target that runs the repository's documented formatting, clippy, and test checks so the automation-required validation command is available.

Validation

  • cargo test -p rustfs-cli cli_accepts_object_
  • make pre-commit

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