fix: address PR comments

Author: zepfred

tests/test_solution_manager.py

@@ -141,7 +141,7 @@ def assert_score_analysis(problem: Solution, score_analysis: ScoreAnalysis):
 
 
 def assert_score_analysis_summary(score_analysis: ScoreAnalysis):
-    summary = score_analysis.summary
+    summary = score_analysis.summarize
     assert "Explanation of score (3):" in summary
     assert "Constraint matches:" in summary
     assert "3: constraint (Maximize Value) has 3 matches:" in summary
@@ -151,7 +151,7 @@ def assert_score_analysis_summary(score_analysis: ScoreAnalysis):
     assert summary == summary_str
 
     match = score_analysis.constraint_analyses[0]
-    match_summary = match.summary
+    match_summary = match.summarize
     assert "Explanation of score (3):" in match_summary
     assert "Constraint matches:" in match_summary
     assert "3: constraint (Maximize Value) has 3 matches:" in match_summary
@@ -260,6 +260,7 @@ def test_score_manager_constraint_ref():
         'getJustificationList',  # deprecated
         'getJustification',  # built-in constructor and properties with @dataclass
         'getScore',  # built-in constructor and properties with @dataclass
+        'getIndictedObjectList',  # built-in constructor and properties with @dataclass
     },
     'ConstraintMatchTotal': {
         'getConstraintRef',  # built-in constructor and properties with @dataclass

timefold-solver-python-core/src/main/python/score/_score_analysis.py

@@ -118,10 +118,6 @@ class ConstraintMatch(Generic[Score_]):
     def identification_string(self) -> str:
         return self.constraint_ref.constraint_id
 
-    @property
-    def indicted_object_list(self):
-        return self.indicted_objects
-
     def __hash__(self) -> int:
         combined_hash = hash(self.constraint_ref)
         combined_hash ^= _safe_hash(self.justification)
@@ -466,7 +462,7 @@ class ConstraintAnalysis(Generic[Score_]):
          but still non-zero constraint weight; non-empty if constraint has matches.
          This is a list to simplify access to individual elements,
          but it contains no duplicates just like `set` wouldn't.
-    summary : str
+    summarize : str
         Returns a diagnostic text
         that explains part of the score quality through the ConstraintAnalysis API.
     match_count : int
@@ -479,7 +475,7 @@ def __init__(self, delegate: '_JavaConstraintAnalysis[Score_]'):
         delegate.constraintRef()
 
     def __str__(self):
-        return self.summary
+        return self.summarize
 
     @property
     def constraint_ref(self) -> ConstraintRef:
@@ -512,7 +508,7 @@ def score(self) -> Score_:
         return to_python_score(self._delegate.score())
 
     @property
-    def summary(self) -> str:
+    def summarize(self) -> str:
         return self._delegate.summarize()
 
 
@@ -546,7 +542,7 @@ class ScoreAnalysis:
     constraint_analyses : list[ConstraintAnalysis]
         Individual ConstraintAnalysis instances that make up this ScoreAnalysis.
 
-    summary : str
+    summarize : str
         Returns a diagnostic text that explains the solution through the `ConstraintAnalysis` API to identify which
         Constraints cause that score quality.
         The string is built fresh every time the method is called.
@@ -571,7 +567,7 @@ def __init__(self, delegate: '_JavaScoreAnalysis'):
         self._delegate = delegate
 
     def __str__(self):
-        return self.summary
+        return self.summarize
 
     def __sub__(self, other):
         return self.diff(other)
@@ -627,7 +623,7 @@ def constraint_analysis(self, *args) -> ConstraintAnalysis:
             return ConstraintAnalysis(self._delegate.getConstraintAnalysis(args[0], args[1]))
 
     @property
-    def summary(self) -> str:
+    def summarize(self) -> str:
         return self._delegate.summarize()
 
     @property