Tests cleanup; cargo clippy fixes, minor doc updates

[HJLebbink]

Minor issues found while debugging an issue

[Copilot]

Pull Request Overview

This PR cleans up various tests, fixes Clippy warnings, and refines minor documentation and error messages across the codebase.

• Remove obsolete test_object_remove.rs and consolidate object-put tests with a shared helper and additional edge-case scenarios
• Update skip logic in list-objects tests, refine bucket/object-name validation messages, and enhance delete_bucket behavior to log remaining objects
• Apply small doc/format tweaks in builders, client methods, response error formatting, and introduce UUID-based random name generators

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tests/test_object_remove.rs	Removed outdated test file
tests/test_object_put.rs	Extracted helper, added specialized put tests
tests/test_list_objects.rs	Replaced println skips with panic and added skip macros
src/s3/utils.rs	Improved bucket/object name validation and error messages
src/s3/client/delete_bucket.rs	Enhanced delete_bucket to purge/log non-empty buckets
common/src/utils.rs	Switched random-name generators to UUID-based formats

Comments suppressed due to low confidence (1)

src/s3/utils.rs:307

• The variable name bucket_name_n_bytes is misleading in check_object_name; it should reflect object_name, e.g., object_name_len.

    let bucket_name_n_bytes = object_name.len();