test(git): Update test_repo_git_obtain_initial_commit_repo

Author: tony

tests/test_git.py

@@ -44,7 +44,31 @@ def gitconfig_default(monkeypatch: pytest.MonkeyPatch, user_path: pathlib.Path):
     monkeypatch.setenv("HOME", str(user_path))
 
 
-def test_repo_git_obtain_initial_commit_repo(tmp_path: pathlib.Path):
+@pytest.mark.parametrize(
+    # Postpone evaluation of options so fixture variables can interpolate
+    "constructor,lazy_constructor_options",
+    [
+        [
+            GitRepo,
+            lambda bare_repo_dir, tmp_path, **kwargs: {
+                "url": f"file://{bare_repo_dir}",
+                "repo_dir": tmp_path / "obtaining a bare repo",
+            },
+        ],
+        [
+            create_repo_from_pip_url,
+            lambda bare_repo_dir, tmp_path, **kwargs: {
+                "pip_url": f"git+file://{bare_repo_dir}",
+                "repo_dir": tmp_path / "obtaining a bare repo",
+            },
+        ],
+    ],
+)
+def test_repo_git_obtain_initial_commit_repo(
+    tmp_path: pathlib.Path,
+    constructor,
+    lazy_constructor_options,
+):
     """initial commit repos return 'initial'.
 
     note: this behaviors differently from git(1)'s use of the word "bare".
@@ -55,13 +79,7 @@ def test_repo_git_obtain_initial_commit_repo(tmp_path: pathlib.Path):
     run(["git", "init", repo_name], cwd=tmp_path)
 
     bare_repo_dir = tmp_path / repo_name
-
-    git_repo = create_repo_from_pip_url(
-        **{
-            "pip_url": f"git+file://{bare_repo_dir}",
-            "repo_dir": tmp_path / "obtaining a bare repo",
-        }
-    )
+    git_repo: GitRepo = constructor(**lazy_constructor_options(**locals()))
 
     git_repo.obtain()
     assert git_repo.get_revision() == "initial"