[sdairs]

Part of #1173 (4 of 6: tests/client_test.rs). Stacked on #1178.

Summary

This splits tests/client_test.rs (7.2k lines, 231 wiremock tests) into tests/client/, which is still one client_test binary. Behaviour and test names are unchanged. Each test moves to the module named after the src/client/ file that defines the Client method it calls.

File	Lines	Contents
main.rs	52	mod declarations and the shared wiremock helpers (ok_json, ok_empty, created_json, setup)
client.rs	417	19 tests: construction, bearer/basic auth, base URL, error handling, malformed responses, HTTP status codes, and the one cross-domain error test
activity.rs	89	4 tests
api_keys.rs	243	8 tests
backups.rs	607	19 tests: backups, snapshots, backup configuration, backup buckets
byoc.rs	465	12 tests
clickpipes.rs	634	21 tests: pipes, schema discovery, CDC scaling, reverse private endpoints, service context
clickstack.rs	1196	37 tests: alerts, saved searches, dashboards, sources, webhooks, roles
invitations.rs	98	4 tests
members.rs	109	4 tests
organizations.rs	661	22 tests: organizations, quotas, usage cost, Prometheus, credit balances, notifications
postgres.rs	648	20 tests: services, config, logs, Prometheus, backups
query_api_endpoints.rs	243	5 tests
saved_queries.rs	286	6 tests
services.rs	859	30 tests: services, scaling, endpoints, upgrade window, settings, profiles, replica-aware routing
udfs.rs	509	16 tests
whoami.rs	130	4 tests

There is no rbac.rs, because no client test calls an organization_role* method. The ClickStack role tests go in clickstack.rs. tests/run_query_test.rs is unchanged.

Every section divider survives once. A section whose tests now land in several modules keeps its divider in the module that gets its first test.

Commits

1. Pure git mv of tests/client_test.rs → tests/client/main.rs (100%), plus a [[test]] name = "client_test" target so the binary keeps its name. Cargo does not infer a second client target.
2. Moves only. A script checked that all 235 original items (tests and helpers, with their attributes and doc comments) appear exactly once and byte-identical across the new files. Everything else is use/mod lines, dividers and blank lines. Review it with git show --color-moved=dimmed-zebra.
3. Classifier. Each tests/client/*.rs path maps to no live Cloud suite, like tests/client_test.rs, which stays mapped so diffs that name it remain classified. The fixture in scripts/tests/ is updated. The install classifier only inventories crates/clickhousectl.
4. .git-blame-ignore-revs entry for commit 2. This SHA must be updated if the stack is rebased before merge.
5. Docs. crates/clickhouse-cloud-api/AGENTS.md describes the tests/client/ layout. The README needs no change because it names only the unchanged --test client_test target.

Visibility widenings

None. The helpers stay private in main.rs, and the modules reach them with use crate::{...}.

Tests

• Counts match before and after: 916 listed tests (887 passed, 29 ignored) across clickhouse-cloud-api and clickhouse-openapi-analyzer. client_test lists the same 231 names. Names match once module paths are stripped.
• check-openapi-drift.py --dry-run output is identical before and after.
• fmt, both clippy configurations, cargo clippy -p clickhousectl, the --no-default-features check, cargo check --workspace --all-features, the API and analyzer tests (spec_coverage_test and model_facade_test untouched) and the Python classifier tests all pass.

Merge with a merge commit, not a squash. The .git-blame-ignore-revs entry names this PR's move commit. A squash merge would leave it pointing at a commit that isn't on main, and git would silently ignore it.

🤖 Generated with Claude Code

---

Note

Low Risk
Test-only file moves and Cargo/docs metadata; no changes to the public API client implementation.

Overview
Reorganizes the offline wiremock client_test suite from one ~7k-line tests/client_test.rs into tests/client/, still a single binary named client_test. Cargo.toml adds an explicit [[test]] entry pointing at tests/client/main.rs so the target name stays stable when Cargo no longer auto-discovers a lone file.

main.rs declares one module per src/client/ domain (plus client.rs for auth, construction, and shared error handling) and keeps the shared helpers (setup, ok_json, ok_empty, created_json). Existing tests are moved into matching files without behavior or name changes; AGENTS.md documents where to add new client tests. .git-blame-ignore-revs records the bulk move commit for blame hygiene.

^{Reviewed by Cursor Bugbot for commit f8cc2a5. Bugbot is set up for automated code reviews on this repo. Configure here.}