trusty-mpm 1.8.2

trusty-mpm: unified multi-agent orchestration platform (core, daemon, CLI, TUI, Telegram)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107

//! Unit tests for `provisioner::workspace`.
//!
//! Why: extracted from `workspace.rs` so the production module keeps its
//! Why/What/Test documentation density under the mechanical SLOC gate
//! (`scripts/check_line_cap.sh`), whose test/benchmark cap this file's
//! `tests.rs` basename selects. ADR-0055 (#6000) removed the session-workspace
//! provisioner most of this file covered; what remains is the per-project
//! identity binding `content::catalog_sync` depends on.
//! What: #2184 — the resolved identity is applied to every command the backend
//! builds, and an empty identity leaves the command untouched.
//! Test: this file IS the test module; run with `cargo test -p trusty-mpm`.

use super::*;

// ── #2184: RealGitBackend applies the resolved identity to every command ──

/// Why: a `RealGitBackend::default()` (no identity resolved) must build a
/// command carrying ONLY `trusty_common::git::command`'s maintenance-disable
/// args — no identity `-c` args, no env overrides — so every existing
/// production call site (which constructs `RealGitBackend::default()` when it
/// has no project context) is unaffected by #2184 beyond the #7171
/// maintenance-storm fix every git command in the workspace now carries.
/// Test: itself.
#[test]
fn default_identity_produces_plain_git_command() {
    let backend = RealGitBackend::default();
    let cmd = backend.command();
    let args: Vec<&std::ffi::OsStr> = cmd.get_args().collect();
    assert_eq!(
        args,
        vec![
            std::ffi::OsStr::new("-c"),
            std::ffi::OsStr::new("maintenance.auto=false"),
            std::ffi::OsStr::new("-c"),
            std::ffi::OsStr::new("gc.auto=0"),
        ],
        "an empty identity adds no -c args of its own beyond #7171's"
    );
    assert_eq!(
        cmd.get_envs().count(),
        0,
        "no env overrides for an empty identity"
    );
}

/// Why: the resolved `GitIdentity::env` overrides must be applied to every
/// command this backend builds, so `git`/its credential helper authenticate
/// as the right per-project identity.
/// Test: itself.
#[test]
fn git_identity_env_applied_to_command() {
    let identity = crate::core::git_identity::GitIdentity {
        env: vec![("GH_CONFIG_DIR".to_string(), "/cfg/project".to_string())],
        env_remove: vec!["GH_TOKEN".to_string()],
        commit_name: None,
        commit_email: None,
    };
    let backend = RealGitBackend::new(identity);
    let cmd = backend.command();
    let envs: Vec<_> = cmd.get_envs().collect();
    assert!(
        envs.iter().any(|(k, v)| {
            *k == std::ffi::OsStr::new("GH_CONFIG_DIR")
                && *v == Some(std::ffi::OsStr::new("/cfg/project"))
        }),
        "GH_CONFIG_DIR override must be applied: {envs:?}"
    );
    // #6668: a credential helper reads GH_TOKEN ahead of GH_CONFIG_DIR, so the
    // removal has to reach the command too or the binding stays decorative.
    assert!(
        envs.iter()
            .any(|(k, v)| *k == std::ffi::OsStr::new("GH_TOKEN") && v.is_none()),
        "GH_TOKEN must be removed: {envs:?}"
    );
}

/// Why: a resolved commit-identity override must render as `-c user.name=…`/
/// `-c user.email=…` BEFORE any subcommand arg (git only accepts `-c`
/// overrides in that position).
/// Test: itself.
#[test]
fn git_identity_commit_args_applied_to_command() {
    let identity = crate::core::git_identity::GitIdentity {
        env: vec![],
        env_remove: vec![],
        commit_name: Some("Bot".to_string()),
        commit_email: Some("bot@example.com".to_string()),
    };
    let backend = RealGitBackend::new(identity);
    let cmd = backend.command();
    let args: Vec<&std::ffi::OsStr> = cmd.get_args().collect();
    assert_eq!(
        args,
        vec![
            // #7171: trusty_common::git::command's maintenance-disable args
            // come first — every RealGitBackend command carries them.
            std::ffi::OsStr::new("-c"),
            std::ffi::OsStr::new("maintenance.auto=false"),
            std::ffi::OsStr::new("-c"),
            std::ffi::OsStr::new("gc.auto=0"),
            std::ffi::OsStr::new("-c"),
            std::ffi::OsStr::new("user.name=Bot"),
            std::ffi::OsStr::new("-c"),
            std::ffi::OsStr::new("user.email=bot@example.com"),
        ]
    );
}