jackylee-ch opened a new pull request, #688: URL: https://github.com/apache/paimon-rust/pull/688
Four `PartialUpdateConfig` branches and eight `BlobViewStruct::deserialize` branches have no test, while the siblings right next to them are covered. `validated_sequence_groups` tests the primary-key conflict and the duplicate-protected-field case but not "field does not exist in the table schema"; `validated_aggregate_functions` tests the per-field unknown function but not the `fields.default-aggregate-function` one, and never reaches either empty-field-name guard (`fields..list-agg-delimiter`). `BlobViewStruct` has three tests for eight error branches. **Tests only, no production change.** A `payload()` helper assembles the BlobViewStruct wire format from parts so each header field can be corrupted independently — a negative identifier length cannot be produced through `serialize()`. Each new test was confirmed to fail with its guard disabled and to pass once restored. `blob_descriptor.rs` still has four uncovered branches; happy to fold them in if you would rather have both files done at once. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
