Add Intra Workspace Dependency Unit tests

Author: abezukor

crate_universe/src/metadata/dependency.rs

@@ -1009,4 +1009,46 @@ mod test {
             "`mio` is a platform specific dependency and therefore should not be identified under the common configuration."
         );
     }
+
+    #[test]
+    fn intra_workspace_deps() {
+        let metadata = metadata::workspace_path();
+
+        let child_b_node = find_metadata_node("child_b", &metadata);
+        let child_b_depset_with_intra_workspace = DependencySet::new_for_node(
+            child_b_node,
+            &metadata,
+            &TreeResolverMetadata::default(),
+            true,
+        );
+        assert_eq!(
+            child_b_depset_with_intra_workspace
+                .normal_deps
+                .items()
+                .iter()
+                .filter(|(configuration, dep)| {
+                    configuration.is_none() && dep.target_name == "child_a"
+                })
+                .count(),
+            1,
+        );
+
+        let child_b_depset_without_intra_workspace = DependencySet::new_for_node(
+            child_b_node,
+            &metadata,
+            &TreeResolverMetadata::default(),
+            false,
+        );
+        assert_eq!(
+            child_b_depset_without_intra_workspace
+                .normal_deps
+                .items()
+                .iter()
+                .filter(|(configuration, dep)| {
+                    configuration.is_none() && dep.target_name == "child_a"
+                })
+                .count(),
+            0,
+        )
+    }
 }

crate_universe/src/rendering.rs

@@ -2088,8 +2088,58 @@ mod test {
                 tags = ["manual"],
             )
         "#};
-        assert!(build_file_content
-            .replace(' ', "")
-            .contains(&expected.replace(' ', "")));
+        assert!(
+            build_file_content
+                .replace(' ', "")
+                .contains(&expected.replace(' ', "")),
+            "{}\n does not match {}",
+            build_file_content,
+            expected
+        );
+    }
+
+    #[test]
+    fn intra_workspace_deps() {
+        let intra_workspace_dependencies_workspace_toml = Label::Absolute {
+            repository: starlark::Repository::Canonical("test_repo".to_owned()),
+            package: "test_package".to_owned(),
+            target: "Cargo.lock".to_owned(),
+        };
+
+        let render_config = RenderConfig {
+            intra_workspace_dependencies_workspace_toml: Some(
+                intra_workspace_dependencies_workspace_toml,
+            ),
+            ..std::sync::Arc::<RenderConfig>::into_inner(mock_render_config(None)).unwrap()
+        };
+
+        let config = Config {
+            rendering: render_config.clone(),
+            ..Default::default()
+        };
+
+        let annotations = Annotations::new(
+            test::metadata::workspace_path(),
+            test::lockfile::workspace_path(),
+            config,
+            Utf8Path::new("/tmp/bazelworkspace"),
+        )
+        .unwrap();
+        let context = Context::new(annotations, false).unwrap();
+
+        let renderer = Renderer::new(Arc::new(render_config), mock_supported_platform_triples());
+        let output = renderer.render(&context, None).unwrap();
+
+        let expected = indoc! {r#"
+            alias(
+                name = "child_a-0.1.0",
+                actual = "@@test_repo//child_a:child_a",
+                tags = ["manual"],
+            )
+        "#};
+        assert!(output
+            .get(&PathBuf::from("BUILD.bazel"))
+            .unwrap()
+            .contains(expected));
     }
 }

crate_universe/src/test.rs

@@ -204,6 +204,14 @@ pub(crate) mod metadata {
         )))
         .unwrap()
     }
+
+    pub(crate) fn workspace_path() -> cargo_metadata::Metadata {
+        serde_json::from_str(include_str!(concat!(
+            env!("CARGO_MANIFEST_DIR"),
+            "/test_data/metadata/workspace_path/metadata.json"
+        )))
+        .unwrap()
+    }
 }
 
 pub(crate) mod lockfile {
@@ -288,4 +296,12 @@ pub(crate) mod lockfile {
         )))
         .unwrap()
     }
+
+    pub(crate) fn workspace_path() -> cargo_lock::Lockfile {
+        cargo_lock::Lockfile::from_str(include_str!(concat!(
+            env!("CARGO_MANIFEST_DIR"),
+            "/test_data/metadata/workspace_path/Cargo.lock"
+        )))
+        .unwrap()
+    }
 }