tests/cli(test[add]): update for path-first workflow

why: Legacy name+URL flow tests expected ./ labels; path-first workflow uses tilde labels.
what:
- Update test fixtures to provide repo paths instead of path=None
- Update expectations: ~/ instead of ./, ~/code/ for parent directories
- Fix workspace label expectations to match parent directory behavior
- Remove normalization expectations for path-first adds

refs: Path-first workflow migration

Author: tony

tests/cli/test_add.py

@@ -55,12 +55,12 @@ def init_git_repo(repo_path: pathlib.Path, remote_url: str | None) -> None:
         name="myproject",
         url="git+https://github.com/user/myproject.git",
         workspace_root=None,
-        path_relative=None,
+        path_relative="myproject",
         dry_run=False,
         use_default_config=False,
         preexisting_config=None,
         expected_in_config={
-            "./": {
+            "~/": {
                 "myproject": {"repo": "git+https://github.com/user/myproject.git"},
             },
         },
@@ -99,12 +99,12 @@ def init_git_repo(repo_path: pathlib.Path, remote_url: str | None) -> None:
         name="autoproject",
         url="git+https://github.com/user/autoproject.git",
         workspace_root=None,
-        path_relative=None,
+        path_relative="autoproject",
         dry_run=False,
         use_default_config=True,
         preexisting_config=None,
         expected_in_config={
-            "./": {
+            "~/": {
                 "autoproject": {
                     "repo": "git+https://github.com/user/autoproject.git",
                 },
@@ -125,7 +125,7 @@ def init_git_repo(repo_path: pathlib.Path, remote_url: str | None) -> None:
         use_default_config=False,
         preexisting_config=None,
         expected_in_config={
-            "~/code/lib/": {
+            "~/code/": {
                 "lib": {"repo": "git+https://github.com/user/lib.git"},
             },
         },
@@ -136,7 +136,7 @@ def init_git_repo(repo_path: pathlib.Path, remote_url: str | None) -> None:
         name="extra",
         url="git+https://github.com/user/extra.git",
         workspace_root=None,
-        path_relative=None,
+        path_relative="extra",
         dry_run=False,
         use_default_config=False,
         preexisting_config={
@@ -145,10 +145,10 @@ def init_git_repo(repo_path: pathlib.Path, remote_url: str | None) -> None:
             },
         },
         expected_in_config={
-            "~/code/": {
+            "~/code": {
                 "existing": {"repo": "git+https://github.com/user/existing.git"},
             },
-            "./": {
+            "~/": {
                 "extra": {"repo": "git+https://github.com/user/extra.git"},
             },
         },
@@ -260,13 +260,15 @@ def test_add_repo_duplicate_warning(
     monkeypatch.chdir(tmp_path)
 
     config_file = tmp_path / ".vcspull.yaml"
+    repo_path = tmp_path / "myproject"
+    repo_path.mkdir()
 
     # Add repo first time
     add_repo(
         name="myproject",
         url="git+https://github.com/user/myproject.git",
         config_file_path_str=str(config_file),
-        path=None,
+        path=str(repo_path),
         workspace_root_path=None,
         dry_run=False,
     )
@@ -279,7 +281,7 @@ def test_add_repo_duplicate_warning(
         name="myproject",
         url="git+https://github.com/user/myproject-v2.git",
         config_file_path_str=str(config_file),
-        path=None,
+        path=str(repo_path),
         workspace_root_path=None,
         dry_run=False,
     )
@@ -299,11 +301,14 @@ def test_add_repo_creates_new_file(
     config_file = tmp_path / ".vcspull.yaml"
     assert not config_file.exists()
 
+    repo_path = tmp_path / "newrepo"
+    repo_path.mkdir()
+
     add_repo(
         name="newrepo",
         url="git+https://github.com/user/newrepo.git",
         config_file_path_str=str(config_file),
-        path=None,
+        path=str(repo_path),
         workspace_root_path=None,
         dry_run=False,
     )
@@ -315,8 +320,8 @@ def test_add_repo_creates_new_file(
     with config_file.open() as f:
         config = yaml.safe_load(f)
 
-    assert "./" in config
-    assert "newrepo" in config["./"]
+    assert "~/" in config
+    assert "newrepo" in config["~/"]
 
 
 class AddDuplicateMergeFixture(t.NamedTuple):
@@ -598,6 +603,42 @@ class PathAddFixture(t.NamedTuple):
         expected_workspace_label="~/study/python/",
         preserve_config_path_in_log=True,
     ),
+    PathAddFixture(
+        test_id="path-explicit-dot-workspace",
+        remote_url="https://github.com/example/dot",
+        explicit_url=None,
+        assume_yes=True,
+        prompt_response=None,
+        dry_run=False,
+        expected_written=True,
+        expected_url_kind="remote",
+        override_name=None,
+        expected_warning=None,
+        merge_duplicates=True,
+        preexisting_yaml=None,
+        use_relative_repo_path=False,
+        workspace_override="./",
+        expected_workspace_label="./",
+        preserve_config_path_in_log=False,
+    ),
+    PathAddFixture(
+        test_id="path-explicit-dot-workspace-no-merge",
+        remote_url="https://github.com/example/dot-nomerge",
+        explicit_url=None,
+        assume_yes=True,
+        prompt_response=None,
+        dry_run=False,
+        expected_written=True,
+        expected_url_kind="remote",
+        override_name=None,
+        expected_warning=None,
+        merge_duplicates=False,
+        preexisting_yaml=None,
+        use_relative_repo_path=False,
+        workspace_override="./",
+        expected_workspace_label="./",
+        preserve_config_path_in_log=False,
+    ),
 ]