fix: platform-specific dependencies are not shown (#259)

Author: lhenry-dev

src/parsers/manifest.rs

@@ -32,6 +32,18 @@ struct CargoTomlWorkspace {
     members: Vec<RelativePathBuf>,
 }
 
+#[derive(Debug, Deserialize, Serialize)]
+pub struct CargoTomlTargetDependencies {
+    #[serde(default)]
+    dependencies: IndexMap<String, CargoTomlDependency>,
+    #[serde(rename = "dev-dependencies")]
+    #[serde(default)]
+    dev_dependencies: IndexMap<String, CargoTomlDependency>,
+    #[serde(rename = "build-dependencies")]
+    #[serde(default)]
+    build_dependencies: IndexMap<String, CargoTomlDependency>,
+}
+
 #[derive(Serialize, Deserialize, Debug)]
 struct CargoToml {
     #[serde(default)]
@@ -46,6 +58,21 @@ struct CargoToml {
     #[serde(rename = "build-dependencies")]
     #[serde(default)]
     build_dependencies: IndexMap<String, CargoTomlDependency>,
+    #[serde(default)]
+    target: IndexMap<String, CargoTomlTargetDependencies>,
+}
+
+fn extract_target_dependencies_into(
+    target: IndexMap<String, CargoTomlTargetDependencies>,
+    deps: &mut IndexMap<String, CargoTomlDependency>,
+    dev_deps: &mut IndexMap<String, CargoTomlDependency>,
+    build_deps: &mut IndexMap<String, CargoTomlDependency>,
+) {
+    for target_deps in target.into_values() {
+        deps.extend(target_deps.dependencies);
+        dev_deps.extend(target_deps.dev_dependencies);
+        build_deps.extend(target_deps.build_dependencies);
+    }
 }
 
 fn convert_dependency(
@@ -84,14 +111,21 @@ fn convert_dependency(
 }
 
 pub fn parse_manifest_toml(input: &str) -> Result<CrateManifest, Error> {
-    let cargo_toml = toml::de::from_str::<CargoToml>(input)?;
+    let mut cargo_toml = toml::de::from_str::<CargoToml>(input)?;
 
     let mut package_part = None;
     let mut workspace_part = None;
 
     if let Some(package) = cargo_toml.package {
         let crate_name = package.name.parse::<CrateName>()?;
 
+        extract_target_dependencies_into(
+            cargo_toml.target,
+            &mut cargo_toml.dependencies,
+            &mut cargo_toml.dev_dependencies,
+            &mut cargo_toml.build_dependencies,
+        );
+
         let dependencies = cargo_toml
             .dependencies
             .into_iter()
@@ -192,3 +226,45 @@ symbolic-common_crate = { version = "2.0.6", package = "symbolic-common" }
         }
     }
 }
+
+#[test]
+fn parse_manifest_with_target_dependencies() {
+    let toml = r#"[package]
+name = "platform-specific"
+
+[dependencies]
+serde = "1.0"
+
+[target.'cfg(unix)'.dependencies]
+nix = { version = "0.28", features = ["sched"] }
+
+[target.'cfg(windows)'.dev-dependencies]
+winapi = "0.3"
+
+[target.'cfg(any(target_os = "android", target_os = "dragonfly", target_os = "freebsd", target_os = "linux"))'.build-dependencies]
+cc = "1.0"
+"#;
+
+    let manifest = parse_manifest_toml(toml).unwrap();
+
+    match manifest {
+        CrateManifest::Package(name, deps) => {
+            assert_eq!(name.as_ref(), "platform-specific");
+
+            assert_eq!(deps.main.len(), 2);
+            let serde_name: CrateName = "serde".parse().unwrap();
+            assert!(deps.main.get(&serde_name).is_some());
+            let nix_name: CrateName = "nix".parse().unwrap();
+            assert!(deps.main.get(&nix_name).is_some());
+
+            assert_eq!(deps.dev.len(), 1);
+            let winapi_name: CrateName = "winapi".parse().unwrap();
+            assert!(deps.dev.get(&winapi_name).is_some());
+
+            assert_eq!(deps.build.len(), 1);
+            let cc_name: CrateName = "cc".parse().unwrap();
+            assert!(deps.build.get(&cc_name).is_some());
+        }
+        _ => panic!("expected package manifest"),
+    }
+}