Expand "invalid hash" test to assert normal StopIteration

EliahKagan · EliahKagan · commit 822846a95deb · 2023-12-22T02:16:28.000-05:00
Returning an explicit value from a generator function causes that value to be bound to the `value` attribute of the StopIteration exception. This is available as the result of "yield from" when it is used as an expression; or by explicitly catching StopIteration, binding the StopIteration exception to a variable, and accessing the attribute. This feature of generators is rarely used. The `return iter([])` statement in Submodule.iter_items uses this feature, causing the resulting StopIteration exception object to have a `value` attribute that refers to a separate second iterator that also yields no values. From context, this behavior is clearly not the goal; a bare return statement should be used here (which has the same effect except for the `value` attribute of the StopIteration exception). The code had used a bare return prior to 82b131c (gitpython-developers#1282), when `return` was changed to `return iter([])`. That was part of a change that added numerous type annotations. It looks like it was either a mistake, or possibly an attempt to work around an old bug in a static type checker. This commit extends the test_iter_items_from_invalid_hash test to assert that the `value` attribute of the StopIteration is its usual default value of None. This commit only extends the test; it does not fix the bug.
diff --git a/test/test_submodule.py b/test/test_submodule.py
@@ -698,6 +698,7 @@ def test_iter_items_from_invalid_hash(self):
         it = Submodule.iter_items(self.rorepo, "xyz")
         with self.assertRaises(StopIteration) as ctx:
             next(it)
+        self.assertIsNone(ctx.exception.value)
 
     @with_rw_repo(k_no_subm_tag, bare=False)
     def test_first_submodule(self, rwrepo):
