tests/add_from_fs(style[code-quality]): Fix formatting and style issues

why: Ensure code passes linting and follows project formatting standards
what:
- Fix line length violations by splitting long strings
- Improve code readability with proper line breaks
- Use dict() constructor for cleaner code
- Remove unused loop variables with underscore prefix

Author: tony

tests/cli/test_add_from_fs.py

@@ -335,8 +335,14 @@ def test_skip_existing_repos(
 
         # Verify enhanced output for existing repos
         assert "Found 1 existing repositories in configuration:" in caplog.text
-        assert f"  - {repo_name} ({remote_url}) at {str(scan_dir)}/{repo_name} in {config_file}" in caplog.text
-        assert "All found repositories already exist in the configuration. Nothing to do." in caplog.text
+        assert (
+            f"  - {repo_name} ({remote_url}) at {scan_dir!s}/{repo_name} "
+            f"in {config_file}" in caplog.text
+        )
+        assert (
+            "All found repositories already exist in the configuration. Nothing to do."
+            in caplog.text
+        )
 
     def test_user_confirmation(
         self,
@@ -459,35 +465,31 @@ def test_detailed_existing_repos_output(
     ) -> None:
         """Test detailed output when multiple repositories already exist."""
         caplog.set_level("INFO")
-        
+
         # Create scan directory with multiple repos
         scan_dir = tmp_path / "existing_repos"
         scan_dir.mkdir()
-        
+
         # Create multiple repositories
         repos_data = []
-        for i, repo_name in enumerate(["repo1", "repo2", "repo3"]):
+        for _i, repo_name in enumerate(["repo1", "repo2", "repo3"]):
             remote_path = create_git_remote_repo()
             remote_url = f"file://{remote_path}"
             local_repo_path = scan_dir / repo_name
-            
+
             subprocess.run(
                 ["git", "clone", remote_url, str(local_repo_path)],
                 check=True,
                 capture_output=True,
                 env=git_commit_envvars,
             )
             repos_data.append((repo_name, remote_url))
-        
+
         # Pre-create config with all repos
         config_file = tmp_path / ".vcspull.yaml"
-        config_data = {
-            str(scan_dir) + "/": {
-                repo_name: remote_url for repo_name, remote_url in repos_data
-            }
-        }
+        config_data = {str(scan_dir) + "/": dict(repos_data)}
         save_config_yaml(config_file, config_data)
-        
+
         # Run add_from_filesystem
         add_from_filesystem(
             scan_dir_str=str(scan_dir),
@@ -496,17 +498,23 @@ def test_detailed_existing_repos_output(
             base_dir_key_arg=None,
             yes=True,
         )
-        
+
         # Verify detailed output
         assert "Found 3 existing repositories in configuration:" in caplog.text
-        
+
         # Check each repository is listed with correct details
         for repo_name, remote_url in repos_data:
-            expected_line = f"  - {repo_name} ({remote_url}) at {str(scan_dir)}/{repo_name} in {config_file}"
+            expected_line = (
+                f"  - {repo_name} ({remote_url}) at {scan_dir!s}/{repo_name} "
+                f"in {config_file}"
+            )
             assert expected_line in caplog.text
-        
+
         # Verify final message
-        assert "All found repositories already exist in the configuration. Nothing to do." in caplog.text
+        assert (
+            "All found repositories already exist in the configuration. Nothing to do."
+            in caplog.text
+        )
 
     def test_mixed_existing_and_new_repos(
         self,
@@ -517,52 +525,48 @@ def test_mixed_existing_and_new_repos(
     ) -> None:
         """Test output when some repos exist and some are new."""
         caplog.set_level("INFO")
-        
+
         # Create scan directory
         scan_dir = tmp_path / "mixed_repos"
         scan_dir.mkdir()
-        
+
         # Create repositories
         existing_repo_data = []
         new_repo_data = []
-        
+
         # Create two existing repos
-        for i, repo_name in enumerate(["existing1", "existing2"]):
+        for _i, repo_name in enumerate(["existing1", "existing2"]):
             remote_path = create_git_remote_repo()
             remote_url = f"file://{remote_path}"
             local_repo_path = scan_dir / repo_name
-            
+
             subprocess.run(
                 ["git", "clone", remote_url, str(local_repo_path)],
                 check=True,
                 capture_output=True,
                 env=git_commit_envvars,
             )
             existing_repo_data.append((repo_name, remote_url))
-        
+
         # Create two new repos
-        for i, repo_name in enumerate(["new1", "new2"]):
+        for _i, repo_name in enumerate(["new1", "new2"]):
             remote_path = create_git_remote_repo()
             remote_url = f"file://{remote_path}"
             local_repo_path = scan_dir / repo_name
-            
+
             subprocess.run(
                 ["git", "clone", remote_url, str(local_repo_path)],
                 check=True,
                 capture_output=True,
                 env=git_commit_envvars,
             )
             new_repo_data.append((repo_name, remote_url))
-        
+
         # Pre-create config with only existing repos
         config_file = tmp_path / ".vcspull.yaml"
-        config_data = {
-            str(scan_dir) + "/": {
-                repo_name: remote_url for repo_name, remote_url in existing_repo_data
-            }
-        }
+        config_data = {str(scan_dir) + "/": dict(existing_repo_data)}
         save_config_yaml(config_file, config_data)
-        
+
         # Run add_from_filesystem
         add_from_filesystem(
             scan_dir_str=str(scan_dir),
@@ -571,15 +575,18 @@ def test_mixed_existing_and_new_repos(
             base_dir_key_arg=None,
             yes=True,
         )
-        
+
         # Verify existing repos are listed
         assert "Found 2 existing repositories in configuration:" in caplog.text
         for repo_name, remote_url in existing_repo_data:
-            expected_line = f"  - {repo_name} ({remote_url}) at {str(scan_dir)}/{repo_name} in {config_file}"
+            expected_line = (
+                f"  - {repo_name} ({remote_url}) at {scan_dir!s}/{repo_name} "
+                f"in {config_file}"
+            )
             assert expected_line in caplog.text
-        
+
         # Verify new repos are added
         for repo_name, remote_url in new_repo_data:
             assert f"Adding '{repo_name}' ({remote_url})" in caplog.text
-        
+
         assert "Successfully updated" in caplog.text