refactor: switch admin create_queue from protobuf to binary encoding
Merged
VI
8mo ago+24+24+01234567890123456789-15-15-01234567890123456789
Summary
- Update
FibpAdminClienttest helper from hand-encoded protobuf to binary wire format - Replace
writeVarintwithwriteU16/writeU32helpers for binary encoding encodeCreateQueueRequestnow producesqueue_len:u16 + queue + on_enqueue_len:u16 + on_failure_len:u16 + visibility_timeout_ms:u32
Test plan
-
./gradlew compileJava compileTestJavapasses (verified locally) - Integration tests pass against a FIBP server with binary admin encoding (PR #153)
- No protobuf encoding remains in test infrastructure
Summary by cubic
Switch the test FibpAdminClient CreateQueue request from hand-rolled protobuf to the binary wire format. Aligns admin tests with the server’s binary encoding, removes protobuf from test code, and adds UTF-8 length validation to prevent u16 overflow.
-
Refactors
- Encode CreateQueue as: queue_len:u16 + queue:utf8 + on_enqueue_len:u16 + on_enqueue:utf8 + on_failure_len:u16 + on_failure:utf8 + visibility_timeout_ms:u32.
- Replace varint writer with
writeU16andwriteU32.
-
Bug Fixes
- Validate queue name UTF-8 length (<= 65535); throw IOException if exceeded.