Reduce the amount of prints in tests

Author: Simon-Rey

tests/test_rules/test_chamberlin_courant.py

@@ -11,10 +11,9 @@ class TestChamberlinCourant(TestCase):
     def test_cc_on_random_instance(self):
         for _ in range(50):
             profile = get_random_profile(50, 100)
-            print(f"Computing CC on randomly generated instance: {profile}")
             max_size = random.randint(1, len(profile.alternatives))
             res = chamberlin_courant(profile, max_size, resoluteness=True)
-            self.assertLessEqual(len(res), max_size)
+            self.assertLessEqual(len(res), max_size, f"Failure with CC on: {profile}, k={max_size}")
 
     def test_cc_on_trivial_instances(self):
         # Empty profile

tests/test_rules/test_on_abcvoting_instances.py

@@ -128,10 +128,6 @@ def process_yaml_file(yaml_file_path: str):
     profile_raw = parse_abcvoting_yaml(yaml_file_path)
 
     for profile in [profile_raw, profile_raw.as_multiprofile()]:
-        print(
-            f"Testing on {os.path.basename(yaml_file_path)}: {len(profile.alternatives)} alternatives and {profile.num_ballots()} voters"
-        )
-
         expected_result = read_abcvoting_expected_result(yaml_file_path, profile)
 
         for rule_id, rules in RULE_MAPPING.items():
@@ -148,7 +144,7 @@ def process_yaml_file(yaml_file_path: str):
                         selection.selected, profile
                     )
                     # print("\t", "R", selection_repr, potential_results_repr)
-                    assert selection_repr in potential_results_repr
+                    assert selection_repr in potential_results_repr, f"Failure on {os.path.basename(yaml_file_path)} (m={len(profile.alternatives)}, n={profile.num_ballots()}) with resolute {rule.__name__}: {selection_repr} not in {potential_results_repr}"
                 except NotImplementedError:
                     pass
 
@@ -160,7 +156,7 @@ def process_yaml_file(yaml_file_path: str):
                         [s.selected for s in selections], profile
                     )
                     # print("\t", "IR", selections_repr, potential_results_repr)
-                    assert selections_repr == potential_results_repr
+                    assert selections_repr == potential_results_repr, f"Failure on {os.path.basename(yaml_file_path)} (m={len(profile.alternatives)}, n={profile.num_ballots()}) with resolute {rule.__name__}: {selections_repr} != {potential_results_repr}"
                 except NotImplementedError:
                     pass
 

tests/test_rules/test_phragmen.py

@@ -15,10 +15,9 @@ class TestPhragmen(TestCase):
     def test_phragmen_on_random_instance(self):
         for _ in range(50):
             profile = get_random_profile(50, 100)
-            print(f"Computing Phragmén on randomly generated instance: {profile}")
             max_size = random.randint(1, len(profile.alternatives))
             res = sequential_phragmen(profile, max_size, resoluteness=True)
-            self.assertLessEqual(len(res), max_size)
+            self.assertLessEqual(len(res), max_size, f"Failure with Phragmén on: {profile}, k={max_size}")
 
     def test_phragmen_on_trivial_instances(self):
         # Empty profile

tests/test_rules/test_tax_rules.py

@@ -31,11 +31,10 @@ class TestMES(TestCase):
     def test_tax_rules_on_random_instance(self):
         for _ in range(50):
             profile = get_random_profile(50, 100)
-            print(f"Computing Tax MES on randomly generated instance: {profile}")
             max_size = random.randint(1, len(profile.alternatives))
             for rule in tax_rules(max_size):
                 res = rule(profile, max_size, resoluteness=True)
-                self.assertLessEqual(len(res), max_size)
+                self.assertLessEqual(len(res), max_size, f"Failure with Tax MES on: {profile}, k={max_size}")
 
     def test_tax_rules_on_trivial_instances(self):
         for rule in tax_rules(0):

tests/test_rules/test_thiele_sequential.py

@@ -25,12 +25,9 @@ def test_seq_pav_on_random_instance(self):
                 PAVScoreHervouin2025,
             ]:
                 profile = get_random_profile(20, 50)
-                print(
-                    f"Computing Sequential PAV with {score.__name__} on randomly generated instance: {profile}"
-                )
                 max_size = random.randint(1, len(profile.alternatives))
                 res = sequential_thiele(profile, max_size, score, resoluteness=True)
-                self.assertLessEqual(len(res), max_size)
+                self.assertLessEqual(len(res), max_size, f"Failure with Sequential PAV[{score.__name__}] on: {profile}, k={max_size}")
 
     def test_seq_pav_on_trivial_instances(self):
         for score in [