Skip to content

Commit d909716

Browse files
T5 Teamt5-copybara
T5 Team
authored andcommitted
Fix unit tests for comparing Dictionaries.
Replaces `assertContainsSubset` in tests with a functional version that checks equality of asserted values. Previously, only keys were compared. PiperOrigin-RevId: 571983744
1 parent dd1cede commit d909716

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

t5/evaluation/test_utils.py

+8
Original file line numberDiff line numberDiff line change
@@ -26,3 +26,11 @@ def assertDictClose(self, a, b, delta=None, places=None):
2626
self.assertAlmostEqual(a[k], b[k], delta=delta, places=places)
2727
except AssertionError as e:
2828
raise AssertionError(str(e) + " for key '%s'" % k)
29+
30+
def assertDictContainsSubset(self, expected_subset, actual_set):
31+
self.assertContainsSubset(expected_subset.keys(), actual_set.keys())
32+
for k in expected_subset:
33+
try:
34+
self.assertEqual(expected_subset[k], actual_set[k])
35+
except AssertionError as e:
36+
raise AssertionError(str(e) + " for key '%s'" % k) from None

0 commit comments

Comments
 (0)