JeonDaehong opened a new pull request, #8296:
URL: https://github.com/apache/gravitino/pull/8296

   **What changes were proposed in this pull request?**
   In `TestModelMetaService.java`, the tests testInsertAndUpdateModelComment 
and testInsertAndUpdateModelProperties were incorrectly setting the name and 
comment fields. This PR fixes the assignments so that name and comment are 
updated correctly according to the test's intent.
   
   **Why are the changes needed?**
   Previously, the tests would either overwrite the name with a comment value 
or keep the comment unchanged, causing the tests to not properly verify 
updates. Fixing the assignments ensures that comment updates and property 
updates are correctly tested.
   
   Fixes: #8282
   
   **Does this PR introduce any user-facing change?**
   No. This only fixes test behavior.
   
   **How was this patch tested?**
   - Corrected field assignments in the affected tests.
   - Verified that `testInsertAndUpdateModelComment` correctly updates the 
comment without changing the name.
   - Verified that `testInsertAndUpdateModelProperties` correctly maintains the 
name and updates properties.
   - Ran all existing tests to confirm no regressions.
   
   


-- 
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]

Reply via email to