Partial signature message semantic validation

[diegomrsantos]

Issue Addressed

#161

Proposed Changes

Add in a new module the partial signature message semantic validation logic and tests with additional helper functions to enhance test coverage and clarity.

[Copilot]

Pull Request Overview

This PR refactors the partial signature message validation logic by moving its implementation into a new module and updates the tests with additional helper functions to enhance test coverage and clarity.

• Moved the partial signature message validation function into a new module.
• Removed the inlined implementation of validate_partial_signature_message.
• Added test helpers for creating message IDs and asserting validation errors.

Comments suppressed due to low confidence (3)

anchor/message_validator/src/lib.rs:7

• Ensure that the new validate_partial_signature_message implementation in the partial_signature module is adequately covered by tests, particularly for edge-case partial signature validations.

use crate::partial_signature::validate_partial_signature_message;

anchor/message_validator/src/lib.rs:272

• [nitpick] Ensure that create_message_id_for_test is actively used in tests for partial signature validation, so its behavior and output are verified in various scenarios.

pub fn create_message_id_for_test(role: Role) -> MessageId {

anchor/message_validator/src/lib.rs:282

• [nitpick] Consider renaming the parameter 'expected_error' to 'error_predicate' in the assert_validation_error helper to better reflect that it represents a predicate function for evaluating errors.

pub fn assert_validation_error<T, F>(...

[jking-aus]

apologies on the delay -- lgtm great work diego