babumahesh opened a new pull request, #10617:
URL: https://github.com/apache/gravitino/pull/10617
### What changes were proposed in this pull request?
Adds a unit test to verify that SchemaUpdatesRequest properly validates null
updates and throws an appropriate IllegalArgumentException with a descriptive
error message.
### Why are the changes needed?
The SchemaUpdatesRequest.validate() method includes a precondition check to
ensure the updates field is not null. However, this validation logic was not
covered by unit tests. This test ensures the precondition is correctly
enforced and provides the expected error message when violated.
### How was this patch tested?
New unit test testSchemaUpdatesRequestWithUpdatesNull() in
TestSchemaOperations verifies:
- Passing null to SchemaUpdatesRequest constructor
- Calling validate() throws IllegalArgumentException
All existing unit tests and integration tests pass.
Fix: #8348
--
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]
