test(cargo_provider): improve test coverage

Author: bearomorphism

tests/providers/test_cargo_provider.py

@@ -301,3 +301,150 @@ def test_cargo_provider_with_lock(
     provider.set_version("42.1")
     assert file.read_text() == dedent(toml_expected)
     assert lock_file.read_text() == dedent(lock_expected)
+
+
+def test_cargo_provider_workspace_member_without_version_key(
+    config: BaseConfig,
+    chdir: Path,
+):
+    """Test workspace member that has no version key at all (should not crash)."""
+    workspace_toml = """\
+[workspace]
+members = ["member_without_version"]
+
+[workspace.package]
+version = "0.1.0"
+"""
+
+    # Create a member that has no version key at all
+    member_content = """\
+[package]
+name = "member_without_version"
+# No version key - this should trigger NonExistentKey exception
+"""
+
+    lock_content = """\
+[[package]]
+name = "member_without_version"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "123abc"
+"""
+
+    expected_workspace_toml = """\
+[workspace]
+members = ["member_without_version"]
+
+[workspace.package]
+version = "42.1"
+"""
+
+    expected_lock_content = """\
+[[package]]
+name = "member_without_version"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "123abc"
+"""
+
+    # Create the workspace file
+    filename = CargoProvider.filename
+    file = chdir / filename
+    file.write_text(dedent(workspace_toml))
+
+    # Create the member directory and file
+    os.mkdir(chdir / "member_without_version")
+    member_file = chdir / "member_without_version" / "Cargo.toml"
+    member_file.write_text(dedent(member_content))
+
+    # Create the lock file
+    lock_filename = CargoProvider.lock_filename
+    lock_file = chdir / lock_filename
+    lock_file.write_text(dedent(lock_content))
+
+    config.settings["version_provider"] = "cargo"
+
+    provider = get_provider(config)
+    assert isinstance(provider, CargoProvider)
+    assert provider.get_version() == "0.1.0"
+
+    # This should not crash even though the member has no version key
+    provider.set_version("42.1")
+    assert file.read_text() == dedent(expected_workspace_toml)
+    # The lock file should remain unchanged since the member doesn't inherit workspace version
+    assert lock_file.read_text() == dedent(expected_lock_content)
+
+
+def test_cargo_provider_workspace_member_without_workspace_key(
+    config: BaseConfig,
+    chdir: Path,
+):
+    """Test workspace member that has version key but no workspace subkey."""
+    workspace_toml = """\
+[workspace]
+members = ["member_without_workspace"]
+
+[workspace.package]
+version = "0.1.0"
+"""
+
+    # Create a member that has version as a table but no workspace subkey
+    # This should trigger NonExistentKey when trying to access version["workspace"]
+    member_content = """\
+[package]
+name = "member_without_workspace"
+
+[package.version]
+# Has version table but no workspace key - should trigger NonExistentKey
+"""
+
+    lock_content = """\
+[[package]]
+name = "member_without_workspace"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "123abc"
+"""
+
+    expected_workspace_toml = """\
+[workspace]
+members = ["member_without_workspace"]
+
+[workspace.package]
+version = "42.1"
+"""
+
+    expected_lock_content = """\
+[[package]]
+name = "member_without_workspace"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "123abc"
+"""
+
+    # Create the workspace file
+    filename = CargoProvider.filename
+    file = chdir / filename
+    file.write_text(dedent(workspace_toml))
+
+    # Create the member directory and file
+    os.mkdir(chdir / "member_without_workspace")
+    member_file = chdir / "member_without_workspace" / "Cargo.toml"
+    member_file.write_text(dedent(member_content))
+
+    # Create the lock file
+    lock_filename = CargoProvider.lock_filename
+    lock_file = chdir / lock_filename
+    lock_file.write_text(dedent(lock_content))
+
+    config.settings["version_provider"] = "cargo"
+
+    provider = get_provider(config)
+    assert isinstance(provider, CargoProvider)
+    assert provider.get_version() == "0.1.0"
+
+    # This should not crash even though the member has no version.workspace key
+    provider.set_version("42.1")
+    assert file.read_text() == dedent(expected_workspace_toml)
+    # The lock file should remain unchanged since the member doesn't inherit workspace version
+    assert lock_file.read_text() == dedent(expected_lock_content)