hint for only bad dependency true literal and rename unit test to match

Author: borngraced

crates/cargo-util-schemas/src/manifest/mod.rs

@@ -783,13 +783,17 @@ impl<'de, P: Deserialize<'de> + Clone> de::Deserialize<'de> for TomlDependency<P
             .expecting(expected)
             .string(|value| Ok(TomlDependency::Simple(value.to_owned())))
             .bool(|value| {
-                let err = format!(
-                    "invalid type: boolean `{value}`, expected {expected}\n\
+                let expected = format!("invalid type: boolean `{value}`, expected {expected}");
+                let err = if value {
+                    format!(
+                        "{expected}\n\
                     note: if you meant to use a workspace member, you can write\n \
-                      dep = {{ workspace = {value} }}\n\
-                    or\n \
-                      dep.workspace = {value}",
-                );
+                      dep.workspace = {value}"
+                    )
+                } else {
+                    expected
+                };
+
                 Err(serde_untagged::de::Error::custom(err))
             })
             .map(|value| value.deserialize().map(TomlDependency::Detailed))

tests/testsuite/bad_config.rs

@@ -2648,7 +2648,7 @@ fn bad_dependency() {
 }
 
 #[cargo_test]
-fn bad_boolean_dependency() {
+fn bad_dependency_true_literal() {
     let p = project()
         .file(
             "Cargo.toml",
@@ -2671,8 +2671,6 @@ fn bad_boolean_dependency() {
         .with_stderr_data(str![[r#"
 [ERROR] invalid type: boolean `true`, expected a version string like "0.9.8" or a detailed dependency like { version = "0.9.8" }
 [NOTE] if you meant to use a workspace member, you can write
- dep = { workspace = true }
-or
  dep.workspace = true
  --> Cargo.toml:9:23
   |