test(priv_dep): A test case for adding the same dependency as private directly and as public indirectly in the same time.

Author: linyihai

tests/testsuite/pub_priv.rs

@@ -480,6 +480,9 @@ fn allow_priv_in_custom_build() {
         .run()
 }
 
+
+// A indirectly add D as private dependency.
+// A -> B -> C -> D
 #[cargo_test(nightly, reason = "exported_private_dependencies lint is unstable")]
 fn recursive_package_pub_no_warning() {
     Package::new("grandparent_bar", "0.1.0")
@@ -549,6 +552,8 @@ fn recursive_package_pub_no_warning() {
         .run()
 }
 
+// A indirectly add D as private dependency.
+// A -> B -> C -> D
 #[cargo_test(nightly, reason = "exported_private_dependencies lint is unstable")]
 fn recursive_package_priv_warning() {
     Package::new("grandparent_bar", "0.1.0")
@@ -617,3 +622,78 @@ fn recursive_package_priv_warning() {
         )
         .run()
 }
+
+// A indirectly add D as public dependency, then directly add D as private dependency. 
+// A -> B -> C -> D
+//  \ _ _ _ _ _ /|\
+#[cargo_test(nightly, reason = "exported_private_dependencies lint is unstable")]
+fn recursive_package_pub_priv_together() {
+    Package::new("grandparent_bar", "0.1.0")
+        .file("src/lib.rs", "pub struct FromPriv;")
+        .publish();
+    Package::new("parent_bar", "0.1.0")
+        .cargo_feature("public-dependency")
+        .add_dep(Dependency::new("grandparent_bar", "0.1.0").public(true))
+        .file(
+            "src/lib.rs",
+            "
+            extern crate grandparent_bar;
+            pub use grandparent_bar::*;
+        ",
+        )
+        .publish();
+    Package::new("pub_dep", "0.1.0")
+        .cargo_feature("public-dependency")
+        .add_dep(Dependency::new("parent_bar", "0.1.0").public(true))
+        .file(
+            "src/lib.rs",
+            "
+            extern crate parent_bar;
+            pub use parent_bar::*;
+        ",
+        )
+        .publish();
+    let p = project()
+        .file(
+            "Cargo.toml",
+            r#"
+            cargo-features = ["public-dependency"]
+
+            [package]
+            name = "foo"
+            version = "0.0.1"
+
+            [dependencies]
+            pub_dep = {version = "0.1.0", public = true}
+            grandparent_bar = "0.1.0"
+        "#,
+        )
+        .file(
+            "src/lib.rs",
+            "
+            extern crate pub_dep;
+            extern crate grandparent_bar;
+
+            pub fn use_pub(_: grandparent_bar::FromPriv) {}
+        ",
+        )
+        .build();
+
+    p.cargo("check --message-format=short")
+        .masquerade_as_nightly_cargo(&["public-dependency"])
+        .with_stderr(
+            "\
+[UPDATING] `[..]` index
+[DOWNLOADING] crates ...
+[DOWNLOADED] pub_dep v0.1.0 ([..])
+[DOWNLOADED] parent_bar v0.1.0 ([..])
+[DOWNLOADED] grandparent_bar v0.1.0 ([..])
+[CHECKING] grandparent_bar v0.1.0
+[CHECKING] parent_bar v0.1.0
+[CHECKING] pub_dep v0.1.0
+[CHECKING] foo v0.0.1 ([..])
+[FINISHED] dev [unoptimized + debuginfo] target(s) in [..]
+",
+        )
+        .run()
+}