Skip to content

test(admin): cover site replication add body#263

Merged
overtrue merged 1 commit into
mainfrom
automation/test-gap-detection-20260719
Jul 19, 2026
Merged

test(admin): cover site replication add body#263
overtrue merged 1 commit into
mainfrom
automation/test-gap-detection-20260719

Conversation

@overtrue

Copy link
Copy Markdown
Contributor

Summary

  • add a focused AdminClient regression test for site-replication/add
  • verify the PUT route and serialized peer site body fields

Evidence

Recent main added admin site replication commands and AdminClient::site_replication_add, while existing CLI coverage only checked dispatch and success output.

Validation

  • cargo test -p rc-s3 test_site_replication_add_puts_peer_sites_body
  • cargo fmt --all --check
  • git diff --check
  • make pre-commit

@overtrue
overtrue marked this pull request as ready for review July 19, 2026 14:24
Copilot AI review requested due to automatic review settings July 19, 2026 14:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@overtrue
overtrue merged commit 20e284f into main Jul 19, 2026
16 checks passed
@overtrue
overtrue deleted the automation/test-gap-detection-20260719 branch July 19, 2026 15:55
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