GameCult/Huginn
Watch
9
0
Fork
You've already forked Huginn
 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

mind-body dependency graph test: read every target, not the host's

Browse source 
cargo unifies features per target, so a target-specific line that restores
eureka-state's default features left a Linux graph clean while a Windows
build compiled the huginn CLI's parser. The test asks cargo tree for
--target all, for the control and for mind-body.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Meta Vens 2026-10-10 20:15:18 +02:00
commit cb6f1a5a4e
1 changed files with 12 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

19
crates/mind-body/tests/dependency_graph.rs
View file

@ -1,25 +1,30 @@
//! The huginn CLI's argument parser is not in mind-body's build. `eureka-state`
//! carries it behind its default `cli` feature; mind-body takes the client
//! core only, so its organs and the `mind-control` binary compile none of it.
//!
//! The graph is read for every target (`--target all`), not the host's: cargo
//! unifies features per target, so a `[target.'cfg(windows)'.dependencies]` line
//! that restores the default features would put the parser into a Windows build
//! and leave a Linux graph clean.
use std::process::Command;
/// The crates `package` pulls in through normal and build edges, one name per line.
/// The crates `package` pulls in through normal and build edges, on any target,
/// one name per line.
fn dependencies_of(package: &str) -> Vec<String> {
let cargo = std::env::var_os("CARGO").expect("cargo sets CARGO for its tests");
let output = Command::new(cargo)
let output = Command::new(env!("CARGO"))
.current_dir(env!("CARGO_MANIFEST_DIR"))
.args(["tree", "--offline", "--locked", "-p", package, "-e", "normal,build", "--prefix", "none"])
.args(["tree", "--locked", "-p", package, "-e", "normal,build", "--target", "all", "--prefix", "none"])
.output()
.expect("cargo tree runs");
assert!(output.status.success(), "cargo tree -p {package} failed");
assert!(output.status.success(), "cargo tree -p {package} failed: {}", String::from_utf8_lossy(&output.stderr));
String::from_utf8(output.stdout).unwrap().lines().filter_map(|line| line.split_whitespace().next()).map(str::to_string).collect()
}
#[test]
fn clap_is_not_in_mind_bodys_graph_though_the_cli_crate_has_it() {
fn clap_is_not_in_mind_bodys_graph_on_any_target_though_the_cli_crate_has_it() {
// The control: with its default features the client crate does carry clap, so
// an empty answer for mind-body is the dependency's doing and not a blind probe.
assert!(dependencies_of("eureka-state").iter().any(|name| name == "clap"), "eureka-state's default build has clap");
assert!(!dependencies_of("mind-body").iter().any(|name| name == "clap"), "mind-body compiles the huginn CLI's parser");
assert!(!dependencies_of("mind-body").iter().any(|name| name == "clap"), "mind-body compiles the huginn CLI's parser on some target");
}