Add ignored test case for conflicting deps in unrelated ws members

scarb/tests/metadata.rs

@@ -2,6 +2,7 @@ use std::collections::BTreeMap;
 
 use assert_fs::prelude::*;
 use indoc::indoc;
+use itertools::Itertools;
 use serde_json::json;
 
 use scarb_metadata::{Cfg, DepKind, ManifestMetadataBuilder, Metadata, PackageMetadata};
@@ -31,6 +32,18 @@ fn packages_and_deps(meta: Metadata) -> BTreeMap<String, Vec<String>> {
         .collect::<BTreeMap<_, _>>()
 }
 
+fn units_and_components(meta: Metadata) -> BTreeMap<String, Vec<String>> {
+    meta.compilation_units
+        .iter()
+        .map(|cu| {
+            (
+                cu.target.name.clone(),
+                cu.components.iter().map(|c| c.name.clone()).collect_vec(),
+            )
+        })
+        .collect::<BTreeMap<_, _>>()
+}
+
 #[test]
 fn simple() {
     let t = assert_fs::TempDir::new().unwrap();
@@ -259,6 +272,81 @@ fn dev_dependencies() {
     );
 }
 
+#[test]
+#[ignore = "not implemented yet"]
+fn dev_deps_are_not_propagated() {
+    // TODO(maciektr): Make sure dev-deps are not propagated.
+    let t = assert_fs::TempDir::new().unwrap();
+
+    let dep1 = t.child("dep1");
+    ProjectBuilder::start().name("dep1").build(&dep1);
+
+    let dep2 = t.child("dep2");
+    ProjectBuilder::start()
+        .name("dep2")
+        .dev_dep("dep1", &dep1)
+        .build(&dep2);
+
+    let pkg = t.child("pkg");
+    ProjectBuilder::start()
+        .name("x")
+        .dev_dep("dep2", &dep2)
+        .build(&pkg);
+
+    let metadata = Scarb::quick_snapbox()
+        .arg("--json")
+        .arg("metadata")
+        .arg("--format-version")
+        .arg("1")
+        .current_dir(&pkg)
+        .stdout_json::<Metadata>();
+
+    assert_eq!(
+        packages_and_deps(metadata.clone()),
+        BTreeMap::from_iter([
+            ("core".to_string(), vec![]),
+            ("test_plugin".to_string(), vec![]),
+            (
+                "x".to_string(),
+                vec![
+                    "core".to_string(),
+                    "dep2".to_string(),
+                    "test_plugin".to_string(),
+                ]
+            ),
+            (
+                "dep1".to_string(),
+                vec!["core".to_string(), "test_plugin".to_string()]
+            ),
+            (
+                "dep2".to_string(),
+                vec![
+                    "core".to_string(),
+                    "dep1".to_string(),
+                    "test_plugin".to_string()
+                ]
+            )
+        ])
+    );
+
+    assert_eq!(
+        units_and_components(metadata),
+        BTreeMap::from_iter(vec![
+            ("x".to_string(), vec!["core".to_string(), "x".to_string()]),
+            (
+                "x_unittest".to_string(),
+                vec![
+                    "core".to_string(),
+                    // With dev-deps propagation enabled, this would be included
+                    // "dep1".to_string(),
+                    "dep2".to_string(),
+                    "x".to_string()
+                ]
+            ),
+        ])
+    );
+}
+
 #[test]
 fn no_dep() {
     let t = assert_fs::TempDir::new().unwrap();
@@ -557,6 +645,69 @@ fn workspace_with_root() {
     )
 }
 
+#[test]
+#[ignore = "not implemented yet"]
+fn workspace_with_conflicting_dep() {
+    // TODO(#2): Allow using conflicting dependencies in unrelated workspace members.
+    let t = assert_fs::TempDir::new().unwrap().child("test_workspace");
+    let dep1 = t.child("dep1");
+    ProjectBuilder::start()
+        .name("some_dep")
+        .version("1.0.0")
+        .build(&dep1);
+    let dep2 = t.child("dep2");
+    ProjectBuilder::start()
+        .name("some_dep")
+        .version("2.0.0")
+        .build(&dep2);
+    let pkg1 = t.child("first");
+    ProjectBuilder::start()
+        .name("first")
+        .dep("some_dep", &dep1)
+        .build(&pkg1);
+    let pkg2 = t.child("second");
+    ProjectBuilder::start()
+        .name("second")
+        .dep("some_dep", &dep2)
+        .build(&pkg2);
+    WorkspaceBuilder::start()
+        .add_member("first")
+        .add_member("second")
+        .build(&t);
+
+    let metadata = Scarb::quick_snapbox()
+        .args(["--json", "metadata", "--format-version=1"])
+        .current_dir(&t)
+        .stdout_json::<Metadata>();
+    assert_eq!(
+        packages_and_deps(metadata),
+        BTreeMap::from_iter([
+            ("core".to_string(), vec![]),
+            ("test_plugin".to_string(), vec![]),
+            (
+                "first".to_string(),
+                vec![
+                    "core".to_string(),
+                    "some_dep".to_string(),
+                    "test_plugin".to_string()
+                ]
+            ),
+            (
+                "second".to_string(),
+                vec![
+                    "core".to_string(),
+                    "some_dep".to_string(),
+                    "test_plugin".to_string()
+                ]
+            ),
+            (
+                "some_dep".to_string(),
+                vec!["core".to_string(), "test_plugin".to_string()]
+            ),
+        ])
+    )
+}
+
 #[test]
 fn workspace_as_dep() {
     let t = assert_fs::TempDir::new().unwrap();