MINOR - test case summary list optimization (#26004)

* fix(testCase-Summary): summary field optimization

* fix(testCase-Summary): remove collectors

(cherry picked from commit e387503)

Author: TeddyCr

openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/TestSuiteRepository.java

@@ -238,6 +238,8 @@ private Map<UUID, List<EntityReference>> batchFetchTestCases(List<TestSuite> tes
   public void clearFields(TestSuite entity, EntityUtil.Fields fields) {
     entity.setPipelines(fields.contains("pipelines") ? entity.getPipelines() : null);
     entity.setSummary(fields.contains("summary") ? entity.getSummary() : null);
+    entity.setTestCaseResultSummary(
+        fields.contains("summary") ? entity.getTestCaseResultSummary() : null);
     entity.withTests(fields.contains(UPDATE_FIELDS) ? entity.getTests() : null);
   }
 
@@ -423,25 +425,23 @@ private void fetchAndSetTestCaseResultSummary(
     List<UUID> suiteIds = testSuites.stream().map(TestSuite::getId).toList();
     Map<UUID, List<ResultSummary>> testCaseResultSummaryMap = batchGetResultSummary(suiteIds);
 
-    Set<String> allTestCaseFQNs =
-        testCaseResultSummaryMap.values().stream()
-            .flatMap(List::stream)
-            .map(ResultSummary::getTestCaseName)
-            .collect(Collectors.toSet());
-    Map<String, String> entityLinkMap = batchResolveEntityLinks(allTestCaseFQNs);
-
-    Map<UUID, TestSummary> testSummaryMap =
-        testCaseResultSummaryMap.entrySet().stream()
-            .collect(
-                Collectors.toMap(
-                    Map.Entry::getKey, entry -> getTestSummary(entry.getValue(), entityLinkMap)));
-
-    setFieldFromMap(
-        true, testSuites, testCaseResultSummaryMap, TestSuite::setTestCaseResultSummary);
+    Map<UUID, TestSummary> testSummaryMap = new HashMap<>();
+    testCaseResultSummaryMap.forEach(
+        (id, results) -> testSummaryMap.put(id, computeSimpleSummary(results)));
 
     setFieldFromMap(true, testSuites, testSummaryMap, TestSuite::setSummary);
   }
 
+  private TestSummary computeSimpleSummary(List<ResultSummary> results) {
+    Map<String, Integer> statusCounts = new HashMap<>();
+    for (ResultSummary r : results) {
+      statusCounts.merge(r.getStatus().toString(), 1, Integer::sum);
+    }
+    TestSummary summary = createTestSummary(statusCounts);
+    summary.setTotal(results.size());
+    return summary;
+  }
+
   protected void fetchAndSetIngestionPipelines(List<TestSuite> entities, EntityUtil.Fields fields) {
     if (!fields.contains("pipelines") || entities == null || entities.isEmpty()) {
       return;
@@ -472,21 +472,6 @@ private Map<UUID, List<ResultSummary>> batchGetResultSummary(List<UUID> testSuit
     return repository.listResultSummariesForTestSuites(testSuiteIds);
   }
 
-  private Map<String, String> batchResolveEntityLinks(Set<String> testCaseFQNs) {
-    if (testCaseFQNs.isEmpty()) {
-      return Map.of();
-    }
-    List<TestCase> testCases =
-        Entity.getEntityByNames(TEST_CASE, new ArrayList<>(testCaseFQNs), "", ALL);
-    Map<String, String> entityLinkMap = new HashMap<>();
-    for (TestCase tc : testCases) {
-      MessageParser.EntityLink entityLink = MessageParser.EntityLink.parse(tc.getEntityLink());
-      String linkString = entityLink.getFieldName() == null ? "table" : entityLink.getLinkString();
-      entityLinkMap.put(tc.getFullyQualifiedName(), linkString);
-    }
-    return entityLinkMap;
-  }
-
   private TestSummary getTestSummary(
       List<ResultSummary> testCaseResults, Map<String, String> entityLinkMap) {
     record ProcessedTestCaseResults(String entityLink, String status) {}

openmetadata-service/src/test/java/org/openmetadata/service/resources/dqtests/TestSuiteResourceTest.java

@@ -1155,22 +1155,18 @@ void test_testSuiteReindexConsistency(TestInfo test)
       verifyTestCases(testSuiteBeforeReindex.getTests(), testSuiteAfterReindex.getTests());
     }
 
-    // testCaseResultSummary is computed during reindex from DB (not stored per-result).
-    // Before reindex, the ES document has no summary because updateTestSuiteSummary()
-    // no longer runs on each result POST. After reindex, the summary is populated from DB.
-    assertFalse(
-        testSuiteAfterReindex.getTestCaseResultSummary().isEmpty(),
-        "After reindex, testCaseResultSummary should be populated from DB");
+    // The batch /search/list path computes summary (total/success/failed counts) directly
+    // from result statuses without populating testCaseResultSummary (which is expensive).
+    assertNotNull(
+        testSuiteAfterReindex.getSummary(), "After reindex, summary should be populated from DB");
     assertEquals(
         1,
-        testSuiteAfterReindex.getTestCaseResultSummary().size(),
-        "Should have exactly one test case result summary entry");
+        testSuiteAfterReindex.getSummary().getTotal(),
+        "Should have exactly one test case in summary total");
     assertEquals(
-        testCase.getFullyQualifiedName(),
-        testSuiteAfterReindex.getTestCaseResultSummary().get(0).getTestCaseName());
-    assertEquals(
-        TestCaseStatus.Success,
-        testSuiteAfterReindex.getTestCaseResultSummary().get(0).getStatus());
+        1,
+        testSuiteAfterReindex.getSummary().getSuccess(),
+        "The single test case result should be Success");
   }
 
   private void postTriggerSearchIndexingApp(Map<String, String> authHeaders) throws IOException {