debian_analyzer/

debcargo.rs

//! debcargo.toml file manipulation

// TODO: Reuse the debcargo crate for more of this.

use debian_control::fields::MultiArch;
use std::collections::{HashMap, HashSet};
use std::path::{Path, PathBuf};
use toml_edit::{value, DocumentMut, Table};

pub use toml_edit;

/// The default maintainer for Rust packages.
pub const DEFAULT_MAINTAINER: &str =
    "Debian Rust Maintainers <pkg-rust-maintainers@alioth-lists.debian.net>";

/// The default section for Rust packages.
pub const DEFAULT_SECTION: &str = "rust";

/// The current standards version.
pub const CURRENT_STANDARDS_VERSION: &str = "4.5.1";

/// The default priority for Rust packages.
pub const DEFAULT_PRIORITY: debian_control::Priority = debian_control::Priority::Optional;

/// A wrapper around a debcargo.toml file.
pub struct DebcargoEditor {
    /// Path to the debcargo.toml file.
    debcargo_toml_path: Option<PathBuf>,

    /// The contents of the debcargo.toml file.
    pub debcargo: DocumentMut,

    /// The contents of the Cargo.toml file.
    pub cargo: Option<DocumentMut>,
}

impl From<DocumentMut> for DebcargoEditor {
    fn from(doc: DocumentMut) -> Self {
        Self {
            cargo: None,
            debcargo_toml_path: None,
            debcargo: doc,
        }
    }
}

impl Default for DebcargoEditor {
    fn default() -> Self {
        Self::new()
    }
}

impl DebcargoEditor {
    /// Create a new DebcargoEditor with no contents.
    pub fn new() -> Self {
        Self {
            debcargo_toml_path: None,
            debcargo: DocumentMut::new(),
            cargo: None,
        }
    }

    /// Return the name of the crate.
    fn crate_name(&self) -> Option<&str> {
        self.cargo
            .as_ref()
            .and_then(|c| c["package"]["name"].as_str())
    }

    /// Return the version of the crate.
    fn crate_version(&self) -> Option<semver::Version> {
        self.cargo
            .as_ref()
            .and_then(|c| c["package"]["version"].as_str())
            .map(|s| semver::Version::parse(s).unwrap())
    }

    /// Open a debcargo.toml file.
    pub fn open(path: &Path) -> Result<Self, std::io::Error> {
        let content = std::fs::read_to_string(path)?;
        Ok(Self {
            debcargo_toml_path: Some(path.to_path_buf()),
            cargo: None,
            debcargo: content.parse().unwrap(),
        })
    }

    /// Open a debcargo.toml file in a directory.
    pub fn from_directory(path: &std::path::Path) -> Result<Self, std::io::Error> {
        let debcargo_toml_path = path.join("debian/debcargo.toml");
        let debcargo_toml = std::fs::read_to_string(&debcargo_toml_path)?;
        let cargo_toml = std::fs::read_to_string(path.join("Cargo.toml"))?;
        Ok(Self {
            debcargo_toml_path: Some(debcargo_toml_path),
            debcargo: debcargo_toml.parse().unwrap(),
            cargo: Some(cargo_toml.parse().unwrap()),
        })
    }

    /// Commit changes to the debcargo.toml file.
    pub fn commit(&self) -> std::io::Result<bool> {
        let old_contents = std::fs::read_to_string(self.debcargo_toml_path.as_ref().unwrap())?;
        let new_contents = self.debcargo.to_string();
        if old_contents == new_contents {
            return Ok(false);
        }
        std::fs::write(
            self.debcargo_toml_path.as_ref().unwrap(),
            new_contents.as_bytes(),
        )?;
        Ok(true)
    }

    /// Return the source package
    pub fn source(&mut self) -> DebcargoSource<'_> {
        DebcargoSource { main: self }
    }

    fn semver_suffix(&self) -> bool {
        self.debcargo["source"]
            .get("semver_suffix")
            .and_then(|v| v.as_bool())
            .unwrap_or(false)
    }

    /// Return an iterator over the binaries in the package.
    pub fn binaries(&mut self) -> impl Iterator<Item = DebcargoBinary<'_>> {
        let semver_suffix = self.semver_suffix();

        let mut ret: HashMap<String, String> = HashMap::new();
        ret.insert(
            debcargo_binary_name(
                self.crate_name().unwrap(),
                &if semver_suffix {
                    semver_pair(&self.crate_version().unwrap())
                } else {
                    "".to_string()
                },
            ),
            "lib".to_string(),
        );

        if self.debcargo["bin"].as_bool().unwrap_or(!semver_suffix) {
            let bin_name = self.debcargo["bin_name"]
                .as_str()
                .unwrap_or_else(|| self.crate_name().unwrap());
            ret.insert(bin_name.to_owned(), "bin".to_string());
        }

        let global_summary = self.global_summary();
        let global_description = self.global_description();
        let crate_name = self.crate_name().unwrap().to_string();
        let crate_version = self.crate_version().unwrap();
        let features = self.features();

        self.debcargo
            .as_table_mut()
            .iter_mut()
            .filter_map(move |(key, item)| {
                let kind = ret.remove(&key.to_string())?;
                Some(DebcargoBinary::new(
                    kind,
                    key.to_string(),
                    item.as_table_mut().unwrap(),
                    global_summary.clone(),
                    global_description.clone(),
                    crate_name.clone(),
                    crate_version.clone(),
                    semver_suffix,
                    features.clone(),
                ))
            })
    }

    fn global_summary(&self) -> Option<String> {
        if let Some(summary) = self.debcargo.get("summary").and_then(|v| v.as_str()) {
            Some(format!("{} - Rust source code", summary))
        } else {
            self.cargo.as_ref().and_then(|c| {
                c["package"]
                    .get("description")
                    .and_then(|v| v.as_str())
                    .map(|s| s.split('\n').next().unwrap().to_string())
            })
        }
    }

    fn global_description(&self) -> Option<String> {
        self.debcargo
            .get("description")
            .and_then(|v| v.as_str())
            .map(|description| description.to_owned())
    }

    fn features(&self) -> Option<HashSet<String>> {
        self.cargo
            .as_ref()
            .and_then(|c| c["features"].as_table())
            .map(|t| t.iter().map(|(k, _)| k.to_string()).collect())
    }
}

/// The source package in a debcargo.toml file.
pub struct DebcargoSource<'a> {
    main: &'a mut DebcargoEditor,
}

impl DebcargoSource<'_> {
    /// Return the source section of the debcargo.toml file.
    pub fn toml_section_mut(&mut self) -> &mut Table {
        if !self.main.debcargo.contains_key("source") {
            self.main.debcargo["source"] = toml_edit::Item::Table(Table::new());
        }
        self.main.debcargo["source"].as_table_mut().unwrap()
    }

    /// Set the standards version.
    pub fn set_standards_version(&mut self, version: &str) -> &mut Self {
        self.toml_section_mut()["standards-version"] = value(version);
        self
    }

    /// Return the standards version.
    pub fn standards_version(&self) -> &str {
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("standards-version"))
            .and_then(|v| v.as_str())
            .unwrap_or(CURRENT_STANDARDS_VERSION)
    }

    /// Set the homepage.
    pub fn set_homepage(&mut self, homepage: &str) -> &mut Self {
        self.toml_section_mut()["homepage"] = value(homepage);
        self
    }

    /// Return the homepage.
    pub fn homepage(&self) -> Option<&str> {
        let default_homepage = self
            .main
            .cargo
            .as_ref()
            .and_then(|c| c.get("package"))
            .and_then(|x| x.get("homepage"))
            .and_then(|v| v.as_str());
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("homepage"))
            .and_then(|v| v.as_str())
            .or(default_homepage)
    }

    /// Set the VCS Git URL.
    pub fn set_vcs_git(&mut self, git: &str) -> &mut Self {
        self.toml_section_mut()["vcs_git"] = value(git);
        self
    }

    /// Return the VCS Git URL.
    pub fn vcs_git(&self) -> Option<String> {
        let default_git = self.main.crate_name().map(|c| {
            format!(
                "https://salsa.debian.org/rust-team/debcargo-conf.git [src/{}]",
                c.to_lowercase()
            )
        });

        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("vcs_git"))
            .and_then(|v| v.as_str())
            .map_or(default_git, |s| Some(s.to_string()))
    }

    /// Get the VCS browser URL.
    pub fn vcs_browser(&self) -> Option<String> {
        let default_vcs_browser = self.main.crate_name().map(|c| {
            format!(
                "https://salsa.debian.org/rust-team/debcargo-conf/tree/master/src/{}",
                c.to_lowercase()
            )
        });

        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("vcs_browser"))
            .and_then(|v| v.as_str())
            .map_or(default_vcs_browser, |s| Some(s.to_string()))
    }

    /// Set the VCS browser URL.
    pub fn set_vcs_browser(&mut self, browser: &str) -> &mut Self {
        self.toml_section_mut()["vcs_browser"] = value(browser);
        self
    }

    /// Get the section.
    pub fn section(&self) -> &str {
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("section"))
            .and_then(|v| v.as_str())
            .unwrap_or(DEFAULT_SECTION)
    }

    /// Set the section.
    pub fn set_section(&mut self, section: &str) -> &mut Self {
        self.toml_section_mut()["section"] = value(section);
        self
    }

    /// Get the name of the package.
    pub fn name(&self) -> Option<String> {
        let crate_name = self.main.crate_name()?;
        let semver_suffix = self.main.semver_suffix();
        if semver_suffix {
            let crate_version = self.main.crate_version()?;
            Some(format!(
                "rust-{}-{}",
                debnormalize(crate_name),
                semver_pair(&crate_version)
            ))
        } else {
            Some(format!("rust-{}", debnormalize(crate_name)))
        }
    }

    /// Get the priority.
    pub fn priority(&self) -> debian_control::Priority {
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("priority"))
            .and_then(|v| v.as_str())
            .and_then(|s| s.parse().ok())
            .unwrap_or(DEFAULT_PRIORITY)
    }

    /// Set the priority.
    pub fn set_priority(&mut self, priority: debian_control::Priority) -> &mut Self {
        self.toml_section_mut()["priority"] = value(priority.to_string());
        self
    }

    /// Get whether the package build requires root.
    pub fn rules_requires_root(&self) -> bool {
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("requires_root"))
            .and_then(|v| v.as_bool())
            .unwrap_or(false)
    }

    /// Set whether the package build requires root.
    pub fn set_rules_requires_root(&mut self, requires_root: bool) -> &mut Self {
        self.toml_section_mut()["requires_root"] = value(if requires_root { "yes" } else { "no" });
        self
    }

    /// Get the maintainer.
    pub fn maintainer(&self) -> &str {
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("maintainer"))
            .and_then(|v| v.as_str())
            .unwrap_or(DEFAULT_MAINTAINER)
    }

    /// Set the maintainer.
    pub fn set_maintainer(&mut self, maintainer: &str) -> &mut Self {
        self.toml_section_mut()["maintainer"] = value(maintainer);
        self
    }

    /// Get the uploaders.
    pub fn uploaders(&self) -> Option<Vec<String>> {
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("uploaders"))
            .and_then(|x| x.as_array())
            .map(|a| {
                a.iter()
                    .filter_map(|v| v.as_str())
                    .map(|s| s.to_string())
                    .collect()
            })
    }

    /// Set the uploaders.
    pub fn set_uploaders(&mut self, uploaders: Vec<String>) -> &mut Self {
        let mut array = toml_edit::Array::new();
        for u in uploaders {
            array.push(u);
        }
        self.toml_section_mut()["uploaders"] = value(array);
        self
    }

    /// Get the extra_lines field as a vector of strings.
    pub fn extra_lines(&self) -> Vec<String> {
        self.main
            .debcargo
            .get("source")
            .and_then(|s| s.get("extra_lines"))
            .and_then(|x| x.as_array())
            .map(|a| {
                a.iter()
                    .filter_map(|v| v.as_str())
                    .map(|s| s.to_string())
                    .collect()
            })
            .unwrap_or_default()
    }

    /// Set the extra_lines field.
    pub fn set_extra_lines(&mut self, lines: Vec<String>) -> &mut Self {
        let mut array = toml_edit::Array::new();
        for line in lines {
            array.push(line);
        }
        self.toml_section_mut()["extra_lines"] = value(array);
        self
    }

    /// Add a line to extra_lines if it doesn't already exist.
    pub fn add_extra_line(&mut self, line: String) -> &mut Self {
        let mut lines = self.extra_lines();
        if !lines.contains(&line) {
            lines.push(line);
            self.set_extra_lines(lines);
        }
        self
    }

    /// Remove a line from extra_lines.
    pub fn remove_extra_line(&mut self, line: &str) -> &mut Self {
        let lines = self.extra_lines();
        let filtered: Vec<String> = lines.into_iter().filter(|l| l != line).collect();
        self.set_extra_lines(filtered);
        self
    }

    /// Get a field value from extra_lines (for debian/control fields).
    /// Looks for lines in the format "Field: value" and returns the value.
    pub fn get_extra_field(&self, field_name: &str) -> Option<String> {
        let prefix = format!("{}:", field_name);
        self.extra_lines()
            .iter()
            .find(|line| line.starts_with(&prefix))
            .map(|line| line[prefix.len()..].trim().to_string())
    }

    /// Set a field in extra_lines (for debian/control fields).
    /// Updates existing field or adds new one if not present.
    pub fn set_extra_field(&mut self, field_name: &str, value: &str) -> &mut Self {
        let field_line = format!("{}: {}", field_name, value);
        let prefix = format!("{}:", field_name);

        let mut lines = self.extra_lines();
        let mut found = false;

        // Update existing field
        for line in &mut lines {
            if line.starts_with(&prefix) {
                *line = field_line.clone();
                found = true;
                break;
            }
        }

        // Add new field if not found
        if !found {
            lines.push(field_line);
        }

        self.set_extra_lines(lines);
        self
    }

    /// Remove a field from extra_lines.
    pub fn remove_extra_field(&mut self, field_name: &str) -> &mut Self {
        let prefix = format!("{}:", field_name);
        let lines = self.extra_lines();
        let filtered: Vec<String> = lines
            .into_iter()
            .filter(|line| !line.starts_with(&prefix))
            .collect();
        self.set_extra_lines(filtered);
        self
    }

    /// Set a VCS URL using the appropriate method.
    /// Uses native fields for Git and Browser, extra_lines for others.
    pub fn set_vcs_url(&mut self, vcs_type: &str, url: &str) -> &mut Self {
        match vcs_type.to_lowercase().as_str() {
            "git" => self.set_vcs_git(url),
            "browser" => self.set_vcs_browser(url),
            _ => self.set_extra_field(&format!("Vcs-{}", vcs_type), url),
        }
    }

    /// Get a VCS URL using the appropriate method.
    /// Uses native fields for Git and Browser, extra_lines for others.
    pub fn get_vcs_url(&self, vcs_type: &str) -> Option<String> {
        match vcs_type.to_lowercase().as_str() {
            "git" => self.vcs_git(),
            "browser" => self.vcs_browser(),
            _ => self.get_extra_field(&format!("Vcs-{}", vcs_type)),
        }
    }
}

#[allow(dead_code)]
/// A binary package in a debcargo.toml file.
pub struct DebcargoBinary<'a> {
    table: &'a mut Table,
    key: String,
    name: String,
    section: String,
    global_summary: Option<String>,
    global_description: Option<String>,
    crate_name: String,
    crate_version: semver::Version,
    semver_suffix: bool,
    features: Option<HashSet<String>>,
}

impl<'a> DebcargoBinary<'a> {
    fn new(
        key: String,
        name: String,
        table: &'a mut Table,
        global_summary: Option<String>,
        global_description: Option<String>,
        crate_name: String,
        crate_version: semver::Version,
        semver_suffix: bool,
        features: Option<HashSet<String>>,
    ) -> Self {
        Self {
            key: key.to_owned(),
            name,
            section: format!("packages.{}", key),
            table,
            global_summary,
            global_description,
            crate_name,
            crate_version,
            semver_suffix,
            features,
        }
    }

    /// Get the name of the binary package.
    pub fn name(&self) -> &str {
        &self.name
    }

    /// Get the architecture.
    pub fn architecture(&self) -> Option<&str> {
        Some("any")
    }

    /// Get the multi-architecture setting.
    pub fn multi_arch(&self) -> Option<MultiArch> {
        Some(MultiArch::Same)
    }

    /// Get the package section.
    pub fn section(&self) -> Option<&str> {
        self.table["section"].as_str()
    }

    /// Get the package summary.
    pub fn summary(&self) -> Option<&str> {
        if let Some(summary) = self.table.get("summary").and_then(|v| v.as_str()) {
            Some(summary)
        } else {
            self.global_summary.as_deref()
        }
    }

    /// Get the package long description.
    pub fn long_description(&self) -> Option<String> {
        if let Some(description) = self.table.get("description").and_then(|v| v.as_str()) {
            Some(description.to_string())
        } else if let Some(description) = self.global_description.as_ref() {
            Some(description.clone())
        } else {
            match self.key.as_str() {
                "lib" => Some(format!("Source code for Debianized Rust crate \"{}\"", self.crate_name)),
                "bin" => Some("This package contains the source for the Rust mio crate, packaged by debcargo for use with cargo and dh-cargo.".to_string()),
                _ => None,
            }
        }
    }

    /// Return the package description.
    pub fn description(&self) -> Option<String> {
        Some(crate::control::format_description(
            self.summary()?,
            self.long_description()?.lines().collect(),
        ))
    }

    /// Get the extra dependencies.
    pub fn depends(&self) -> Option<&str> {
        self.table["depends"].as_str()
    }

    /// Get the extra recommends.
    pub fn recommends(&self) -> Option<&str> {
        self.table["recommends"].as_str()
    }

    /// Get the extra suggests.
    pub fn suggests(&self) -> Option<&str> {
        self.table["suggests"].as_str()
    }

    #[allow(dead_code)]
    fn default_provides(&self) -> Option<String> {
        let mut ret = HashSet::new();
        let semver_suffix = self.semver_suffix;
        let semver = &self.crate_version;

        let mut suffixes = vec![];
        if !semver_suffix {
            suffixes.push("".to_string());
        }

        suffixes.push(format!("-{}", semver.major));
        suffixes.push(format!("-{}.{}", semver.major, semver.minor));
        suffixes.push(format!(
            "-{}.{}.{}",
            semver.major, semver.minor, semver.patch
        ));
        for ver_suffix in suffixes {
            let mut feature_suffixes = HashSet::new();
            feature_suffixes.insert("".to_string());
            feature_suffixes.insert("+default".to_string());
            feature_suffixes.extend(
                self.features
                    .as_ref()
                    .map(|k| k.iter().map(|k| format!("+{}", k)).collect::<HashSet<_>>())
                    .unwrap_or_default(),
            );
            for feature_suffix in feature_suffixes {
                ret.insert(debcargo_binary_name(
                    &self.crate_name,
                    &format!("{}{}", ver_suffix, &feature_suffix),
                ));
            }
        }
        ret.remove(self.name());
        if ret.is_empty() {
            None
        } else {
            Some(format!(
                "\n{}",
                &ret.iter()
                    .map(|s| format!("{} (= ${{binary:Version}})", s))
                    .collect::<Vec<_>>()
                    .join(",\n ")
            ))
        }
    }
}

fn debnormalize(s: &str) -> String {
    s.to_lowercase().replace('_', "-")
}

fn semver_pair(s: &semver::Version) -> String {
    format!("{}.{}", s.major, s.minor)
}

fn debcargo_binary_name(crate_name: &str, suffix: &str) -> String {
    format!("librust-{}{}-dev", debnormalize(crate_name), suffix)
}

/// Unmangle a debcargo version.
pub fn unmangle_debcargo_version(version: &str) -> String {
    version.replace("~", "-")
}

#[cfg(test)]
mod tests {
    #[test]
    fn test_debcargo_binary_name() {
        assert_eq!(super::debcargo_binary_name("foo", ""), "librust-foo-dev");
        assert_eq!(
            super::debcargo_binary_name("foo", "-1"),
            "librust-foo-1-dev"
        );
        assert_eq!(
            super::debcargo_binary_name("foo", "-1.2"),
            "librust-foo-1.2-dev"
        );
        assert_eq!(
            super::debcargo_binary_name("foo", "-1.2.3"),
            "librust-foo-1.2.3-dev"
        );
    }

    #[test]
    fn test_semver_pair() {
        assert_eq!(super::semver_pair(&"1.2.3".parse().unwrap()), "1.2");
        assert_eq!(super::semver_pair(&"1.2.6".parse().unwrap()), "1.2");
    }

    #[test]
    fn test_debnormalize() {
        assert_eq!(super::debnormalize("foo_bar"), "foo-bar");
        assert_eq!(super::debnormalize("foo"), "foo");
    }

    #[test]
    fn test_debcargo_editor() {
        let mut editor = super::DebcargoEditor::new();
        editor.debcargo["source"]["standards-version"] = toml_edit::value("4.5.1");
        editor.debcargo["source"]["homepage"] = toml_edit::value("https://example.com");
        editor.debcargo["source"]["vcs_git"] = toml_edit::value("https://example.com");
        editor.debcargo["source"]["vcs_browser"] = toml_edit::value("https://example.com");
        editor.debcargo["source"]["section"] = toml_edit::value("notrust");
        editor.debcargo["source"]["priority"] = toml_edit::value("optional");
        editor.debcargo["source"]["requires_root"] = toml_edit::value("no");
        editor.debcargo["source"]["maintainer"] =
            toml_edit::value("Jelmer Vernooij <jelmer@debian.org>");

        assert_eq!(editor.source().standards_version(), "4.5.1");
        assert_eq!(
            editor.source().vcs_git().as_deref(),
            Some("https://example.com")
        );
        assert_eq!(
            editor.source().vcs_browser().as_deref(),
            Some("https://example.com")
        );
        assert_eq!(editor.source().section(), "notrust");
        assert_eq!(editor.source().priority(), super::DEFAULT_PRIORITY);
        assert!(!editor.source().rules_requires_root());
        assert_eq!(
            editor.source().maintainer(),
            "Jelmer Vernooij <jelmer@debian.org>"
        );
        assert_eq!(editor.source().name(), None);
        assert_eq!(editor.source().uploaders(), None);
        assert_eq!(editor.source().homepage(), Some("https://example.com"));
    }

    #[test]
    fn test_extra_lines_manipulation() {
        let mut editor = super::DebcargoEditor::new();
        let mut source = editor.source();

        // Test initial state
        assert_eq!(source.extra_lines(), Vec::<String>::new());

        // Test set_extra_lines
        source.set_extra_lines(vec![
            "Vcs-Svn: https://svn.example.com/repo".to_string(),
            "X-Custom: value".to_string(),
        ]);
        assert_eq!(
            source.extra_lines(),
            vec![
                "Vcs-Svn: https://svn.example.com/repo".to_string(),
                "X-Custom: value".to_string(),
            ]
        );

        // Test add_extra_line
        source.add_extra_line("Another-Field: another value".to_string());
        assert_eq!(
            source.extra_lines(),
            vec![
                "Vcs-Svn: https://svn.example.com/repo".to_string(),
                "X-Custom: value".to_string(),
                "Another-Field: another value".to_string(),
            ]
        );

        // Test adding duplicate line (should not add)
        source.add_extra_line("X-Custom: value".to_string());
        assert_eq!(
            source.extra_lines(),
            vec![
                "Vcs-Svn: https://svn.example.com/repo".to_string(),
                "X-Custom: value".to_string(),
                "Another-Field: another value".to_string(),
            ]
        );

        // Test remove_extra_line
        source.remove_extra_line("X-Custom: value");
        assert_eq!(
            source.extra_lines(),
            vec![
                "Vcs-Svn: https://svn.example.com/repo".to_string(),
                "Another-Field: another value".to_string(),
            ]
        );
    }

    #[test]
    fn test_extra_field_manipulation() {
        let mut editor = super::DebcargoEditor::new();
        let mut source = editor.source();

        // Test initial state
        assert_eq!(source.get_extra_field("Vcs-Svn"), None);

        // Test set_extra_field
        source.set_extra_field("Vcs-Svn", "https://svn.example.com/repo");
        assert_eq!(
            source.get_extra_field("Vcs-Svn"),
            Some("https://svn.example.com/repo".to_string())
        );

        // Test updating existing field
        source.set_extra_field("Vcs-Svn", "https://svn.example.com/new-repo");
        assert_eq!(
            source.get_extra_field("Vcs-Svn"),
            Some("https://svn.example.com/new-repo".to_string())
        );
        // Should still have only one Vcs-Svn line
        assert_eq!(
            source.extra_lines(),
            vec!["Vcs-Svn: https://svn.example.com/new-repo".to_string()]
        );

        // Test multiple fields
        source.set_extra_field("X-Custom", "custom value");
        assert_eq!(
            source.get_extra_field("X-Custom"),
            Some("custom value".to_string())
        );
        assert_eq!(
            source.get_extra_field("Vcs-Svn"),
            Some("https://svn.example.com/new-repo".to_string())
        );
        assert_eq!(
            source.extra_lines(),
            vec![
                "Vcs-Svn: https://svn.example.com/new-repo".to_string(),
                "X-Custom: custom value".to_string(),
            ]
        );

        // Test remove_extra_field
        source.remove_extra_field("Vcs-Svn");
        assert_eq!(source.get_extra_field("Vcs-Svn"), None);
        assert_eq!(
            source.get_extra_field("X-Custom"),
            Some("custom value".to_string())
        );
        assert_eq!(
            source.extra_lines(),
            vec!["X-Custom: custom value".to_string()]
        );
    }

    #[test]
    fn test_set_vcs_url() {
        let mut editor = super::DebcargoEditor::new();
        let mut source = editor.source();

        // Test native Git field
        source.set_vcs_url("Git", "https://github.com/example/repo.git");
        assert_eq!(
            source.vcs_git(),
            Some("https://github.com/example/repo.git".to_string())
        );

        // Test native Browser field
        source.set_vcs_url("Browser", "https://github.com/example/repo");
        assert_eq!(
            source.vcs_browser(),
            Some("https://github.com/example/repo".to_string())
        );

        // Test non-native VCS type (should use extra_lines)
        source.set_vcs_url("Svn", "https://svn.example.com/repo");
        assert_eq!(
            source.get_extra_field("Vcs-Svn"),
            Some("https://svn.example.com/repo".to_string())
        );

        // Test another non-native VCS type
        source.set_vcs_url("Bzr", "https://bzr.example.com/repo");
        assert_eq!(
            source.get_extra_field("Vcs-Bzr"),
            Some("https://bzr.example.com/repo".to_string())
        );

        // Test case insensitivity for native fields
        source.set_vcs_url("git", "https://gitlab.com/example/repo.git");
        assert_eq!(
            source.vcs_git(),
            Some("https://gitlab.com/example/repo.git".to_string())
        );

        source.set_vcs_url("browser", "https://gitlab.com/example/repo");
        assert_eq!(
            source.vcs_browser(),
            Some("https://gitlab.com/example/repo".to_string())
        );
    }

    #[test]
    fn test_extra_field_parsing() {
        let mut editor = super::DebcargoEditor::new();
        let mut source = editor.source();

        // Test field with spaces after colon
        source.set_extra_lines(vec!["Vcs-Svn:    https://svn.example.com/repo".to_string()]);
        assert_eq!(
            source.get_extra_field("Vcs-Svn"),
            Some("https://svn.example.com/repo".to_string())
        );

        // Test field with no spaces after colon
        source.set_extra_lines(vec!["Vcs-Bzr:https://bzr.example.com/repo".to_string()]);
        assert_eq!(
            source.get_extra_field("Vcs-Bzr"),
            Some("https://bzr.example.com/repo".to_string())
        );
    }

    #[test]
    fn test_get_vcs_url() {
        let mut editor = super::DebcargoEditor::new();
        let mut source = editor.source();

        // Set various VCS URLs
        source.set_vcs_git("https://github.com/example/repo.git");
        source.set_vcs_browser("https://github.com/example/repo");
        source.set_extra_field("Vcs-Svn", "https://svn.example.com/repo");
        source.set_extra_field("Vcs-Bzr", "https://bzr.example.com/repo");

        // Test getting native Git field
        assert_eq!(
            source.get_vcs_url("Git"),
            Some("https://github.com/example/repo.git".to_string())
        );
        assert_eq!(
            source.get_vcs_url("git"),
            Some("https://github.com/example/repo.git".to_string())
        );

        // Test getting native Browser field
        assert_eq!(
            source.get_vcs_url("Browser"),
            Some("https://github.com/example/repo".to_string())
        );
        assert_eq!(
            source.get_vcs_url("browser"),
            Some("https://github.com/example/repo".to_string())
        );

        // Test getting non-native VCS types from extra_lines
        assert_eq!(
            source.get_vcs_url("Svn"),
            Some("https://svn.example.com/repo".to_string())
        );
        assert_eq!(
            source.get_vcs_url("Bzr"),
            Some("https://bzr.example.com/repo".to_string())
        );

        // Test getting non-existent VCS type
        assert_eq!(source.get_vcs_url("Hg"), None);
    }
}