(gravitino) branch branch-1.1 updated: [Cherry-pick to branch-1.1] [#10006] fix(test): Replace brittle version count assertion with semantic checks in TestPolicyMetaService (#10082) (#10086)

[yuqi4733]

This is an automated email from the ASF dual-hosted git repository.

yuqi4733 pushed a commit to branch branch-1.1
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-1.1 by this push:
     new ef07bcc628 [Cherry-pick to branch-1.1] [#10006] fix(test): Replace 
brittle version count assertion with semantic checks in TestPolicyMetaService 
(#10082) (#10086)
ef07bcc628 is described below

commit ef07bcc6280a9a252e827d3ff417ae20a8c26519
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Sat Feb 28 11:45:33 2026 +0800

    [Cherry-pick to branch-1.1] [#10006] fix(test): Replace brittle version 
count assertion with semantic checks in TestPolicyMetaService (#10082) (#10086)
    
    **Cherry-pick Information:**
    - Original commit: 6524fddf2ac229adf21ef333c989e0bdc488cd0d
    - Target branch: `branch-1.1`
    - Status: ✅ Clean cherry-pick (no conflicts)
    
    Co-authored-by: mchades <liminghu...@datastrato.com>
    Co-authored-by: Copilot <223556219+copi...@users.noreply.github.com>
---
 .../storage/relational/service/TestPolicyMetaService.java        | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git 
a/core/src/test/java/org/apache/gravitino/storage/relational/service/TestPolicyMetaService.java
 
b/core/src/test/java/org/apache/gravitino/storage/relational/service/TestPolicyMetaService.java
index 4e0739e287..5dd6056e41 100644
--- 
a/core/src/test/java/org/apache/gravitino/storage/relational/service/TestPolicyMetaService.java
+++ 
b/core/src/test/java/org/apache/gravitino/storage/relational/service/TestPolicyMetaService.java
@@ -208,16 +208,19 @@ public class TestPolicyMetaService extends 
TestJDBCBackend {
     }
     assertFalse(legacyRecordExistsInDB(policy.id(), Entity.EntityType.POLICY));
     assertEquals(0, listPolicyVersions(policy.id()).size());
-    assertEquals(3, listPolicyVersions(anotherPolicy.id()).size());
+    Map<Integer, Long> anotherPolicyVersionsAfterHardDelete =
+        listPolicyVersions(anotherPolicy.id());
+    assertTrue(anotherPolicyVersionsAfterHardDelete.containsKey(3));
+    assertEquals(0L, anotherPolicyVersionsAfterHardDelete.get(3));
 
     // soft delete for old version policy
     for (Entity.EntityType entityType : Entity.EntityType.values()) {
       backend.deleteOldVersionData(entityType, 1);
     }
     Map<Integer, Long> versionDeletedMap2 = 
listPolicyVersions(anotherPolicy.id());
-    assertEquals(3, versionDeletedMap2.size());
+    assertTrue(versionDeletedMap2.containsKey(3));
+    assertEquals(0L, versionDeletedMap2.get(3));
     assertEquals(1, versionDeletedMap2.values().stream().filter(value -> value 
== 0L).count());
-    assertEquals(2, versionDeletedMap2.values().stream().filter(value -> value 
!= 0L).count());
   }
 
   @TestTemplate