Close submodule fixture handles before testing moves on Windows

The Python package CI job test (windows, 3.13) failed both test_move_normal_destination cases with WinError 32 during os.renames(): the newly cloned checkout was still held open. The intermediate-symlink rejection cases and leaf-link compatibility cases passed; the latter already close a module repository while snapshotting state.

Close the fixture module repository before yielding it, reusing Repo.close() and its Windows handle cleanup. This lets the normal-move tests exercise destination validation without depending on garbage-collection timing. Production behavior is unchanged.

Validation: all 30 focused move cases pass locally, along with Ruff lint and formatting and git diff --check. Windows verification will come from the next CI run.

Author: codex

test/test_submodule.py

@@ -60,6 +60,8 @@ def movable_submodule(tmp_path):
         with _patch_git_config("protocol.file.allow", "always"):
             submodule = parent.create_submodule("logical-name", "module", source.working_tree_dir)
         parent.index.commit("Create submodule")
+        # Release clone handles before Windows moves the checkout.
+        submodule.module().close()
         yield submodule