fix: address PR comments

Author: zepfred

tests/test_solution_manager.py

@@ -147,13 +147,19 @@ def assert_score_analysis_summary(score_analysis: ScoreAnalysis):
     assert "3: constraint (Maximize Value) has 3 matches:" in summary
     assert "1: justified with" in summary
 
+    summary_str = str(score_analysis)
+    assert summary == summary_str
+
     match = score_analysis.constraint_analyses[0]
     match_summary = match.summary
     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
     assert "1: justified with" in match_summary
 
+    match_summary_str = str(match)
+    assert match_summary == match_summary_str
+
 
 def assert_solution_manager(solution_manager: SolutionManager[Solution]):
     problem: Solution = Solution([Entity('A', 1), Entity('B', 1), Entity('C', 1)], [1, 2, 3])
@@ -203,7 +209,6 @@ def test_score_manager_diff():
 
     diff_operation = score_analysis - second_score_analysis
     assert diff_operation.score.score == -1
-    print(diff, diff_operation)
 
     constraint_analyses = score_analysis.constraint_analyses
     assert len(constraint_analyses) == 1

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

@@ -479,7 +479,7 @@ def __init__(self, delegate: '_JavaConstraintAnalysis[Score_]'):
         delegate.constraintRef()
 
     def __str__(self):
-        return self.summarize
+        return self.summary
 
     @property
     def constraint_ref(self) -> ConstraintRef:
@@ -611,16 +611,16 @@ def constraint_analysis(self, *args) -> ConstraintAnalysis:
 
         Parameters
         ----------
-        constraint_package : str
-        constraint_name : str
-        constraint_ref : ConstraintRef
+        *args: *tuple[str, str] | *tuple[ConstraintRef]
+            Either two strings or a single ConstraintRef can be passed as positional arguments.
+            If two strings are passed, they are taken to be the constraint package and constraint name, respectively.
+            If a ConstraintRef is passed, it is used to perform the lookup.
 
         Returns
         -------
         ConstraintAnalysis
             None if no constraint matches of such constraint are present
         """
-        print(args)
         if len(args) == 1:
             return ConstraintAnalysis(self._delegate.getConstraintAnalysis(args[0]._to_java()))
         else: