gitacc-switcher 1.2.0__tar.gz → 1.2.1__tar.gz

{gitacc_switcher-1.2.0/gitacc_switcher.egg-info → gitacc_switcher-1.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitacc-switcher
-Version: 1.2.0
+Version: 1.2.1
 Summary: Git Account Switcher - Manage multiple Git SSH accounts easily
 Home-page: https://github.com/ktechhub/gitacc_switcher
 Author: ktechhub

{gitacc_switcher-1.2.0 → gitacc_switcher-1.2.1}/gitacc_switcher/__init__.py

@@ -1,4 +1,4 @@
 """Git Account Switcher - Manage multiple Git SSH accounts easily."""
 
-__version__ = "1.2.0"
+__version__ = "1.2.1"
 __author__ = "ktechhub"

{gitacc_switcher-1.2.0 → gitacc_switcher-1.2.1}/gitacc_switcher/config_manager.py

@@ -359,22 +359,37 @@ class ConfigManager:
             line = lines[i]
             if re.match(rf"^\s*\[{re.escape(account_name)}\]\s*$", line):
                 in_target_section = True
+                field_written = False
                 new_lines.append(line)
                 i += 1
                 while i < len(lines) and in_target_section:
                     if re.match(r"^\s*\[", lines[i]) and not re.match(
                         rf"^\s*\[{re.escape(account_name)}\]\s*$", lines[i]
                     ):
+                        if not field_written:
+                            # Field never appeared in this section (e.g. an
+                            # account written before this field existed) —
+                            # add it rather than silently leaving it unset.
+                            new_lines.append(f"\t{field} = {new_value}\n")
+                            field_written = True
                         in_target_section = False
                         new_lines.append(lines[i])
                         i += 1
                         break
                     elif re.match(rf"^\s*{re.escape(field)}\s*=", lines[i]):
                         new_lines.append(f"\t{field} = {new_value}\n")
+                        field_written = True
                         i += 1
                     else:
                         new_lines.append(lines[i])
                         i += 1
+                else:
+                    # Loop ended because we reached end of file, not a break —
+                    # the target section is the last one, so append here too.
+                    if in_target_section and not field_written:
+                        new_lines.append(f"\t{field} = {new_value}\n")
+                        field_written = True
+                    in_target_section = False
             else:
                 new_lines.append(line)
                 i += 1

{gitacc_switcher-1.2.0 → gitacc_switcher-1.2.1/gitacc_switcher.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitacc-switcher
-Version: 1.2.0
+Version: 1.2.1
 Summary: Git Account Switcher - Manage multiple Git SSH accounts easily
 Home-page: https://github.com/ktechhub/gitacc_switcher
 Author: ktechhub

{gitacc_switcher-1.2.0 → gitacc_switcher-1.2.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "gitacc-switcher"
-version = "1.2.0"
+version = "1.2.1"
 description = "Git Account Switcher - Manage multiple Git SSH accounts easily"
 readme = "README.md"
 requires-python = ">=3.7"

{gitacc_switcher-1.2.0 → gitacc_switcher-1.2.1}/setup.py

@@ -9,7 +9,7 @@ long_description = readme_file.read_text() if readme_file.exists() else ""
 
 setup(
     name="gitacc-switcher",
-    version="1.2.0",
+    version="1.2.1",
     description="Git Account Switcher - Manage multiple Git SSH accounts easily",
     long_description=long_description,
     long_description_content_type="text/markdown",

{gitacc_switcher-1.2.0 → gitacc_switcher-1.2.1}/tests/test_config_manager.py

@@ -164,6 +164,27 @@ class TestUpdateAccountField:
     def test_returns_false_for_missing_account(self, config_with_accounts):
         assert config_with_accounts.update_account_field("nobody", "name", "X") is False
 
+    def test_appends_field_missing_from_section(self, config_with_accounts):
+        # SAMPLE_GITACC accounts have no "sign" field — this reproduces a pre-1.2.0
+        # account whose section was written before that field existed.
+        assert "sign" not in config_with_accounts.get_account("work")
+        assert config_with_accounts.update_account_field("work", "sign", "true") is True
+        assert config_with_accounts.get_account("work")["sign"] == "true"
+
+    def test_appending_missing_field_preserves_other_fields(self, config_with_accounts):
+        config_with_accounts.update_account_field("work", "sign", "true")
+        account = config_with_accounts.get_account("work")
+        assert account["name"] == "Work User"
+        assert account["email"] == "work@company.com"
+        assert account["private_key"] == "/home/user/.ssh/id_rsa_work"
+        assert account["public_key"] == "/home/user/.ssh/id_rsa_work.pub"
+
+    def test_appending_missing_field_does_not_affect_other_account(
+        self, config_with_accounts
+    ):
+        config_with_accounts.update_account_field("work", "sign", "true")
+        assert "sign" not in config_with_accounts.get_account("personal")
+
 
 class TestListAccountNames:
     def test_empty(self, config):