govctl 0.19.0

Project governance CLI for RFC, ADR, and Work Item management
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
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127

use super::*;
use crate::config::Config;
use std::fs;

fn test_config(temp_dir: &tempfile::TempDir) -> Config {
    let mut config = Config {
        gov_root: temp_dir.path().join("gov"),
        ..Config::default()
    };
    config.paths.docs_output = temp_dir.path().join("docs");
    config
}

#[test]
fn execute_ops_removes_created_files_when_later_apply_fails()
-> Result<(), Box<dyn std::error::Error>> {
    let temp_dir = tempfile::tempdir()?;
    let config = test_config(&temp_dir);
    fs::create_dir_all(&config.gov_root)?;
    let created = config.gov_root.join("created.txt");
    let bad_target = config.gov_root.join("bad-target");
    fs::create_dir_all(&bad_target)?;

    let result = execute_ops(
        &config,
        &[
            FileOp::Write {
                path: created.clone(),
                content: "created".to_string(),
            },
            FileOp::Write {
                path: bad_target,
                content: "cannot replace directory".to_string(),
            },
        ],
    );

    assert!(result.is_err());
    assert!(
        !created.exists(),
        "created migration target should be removed on rollback"
    );
    assert!(!config.gov_root.join(".migrate-stage").exists());
    assert!(!config.gov_root.join(".migrate-backup").exists());
    Ok(())
}

#[test]
fn execute_ops_restores_modified_and_deleted_files_when_later_apply_fails()
-> Result<(), Box<dyn std::error::Error>> {
    let temp_dir = tempfile::tempdir()?;
    let config = test_config(&temp_dir);
    fs::create_dir_all(&config.gov_root)?;
    let modified = config.gov_root.join("modified.txt");
    let deleted = config.gov_root.join("deleted.txt");
    let bad_target = config.gov_root.join("bad-target");
    fs::write(&modified, "old")?;
    fs::write(&deleted, "gone")?;
    fs::create_dir_all(&bad_target)?;

    let result = execute_ops(
        &config,
        &[
            FileOp::Write {
                path: modified.clone(),
                content: "new".to_string(),
            },
            FileOp::Delete {
                path: deleted.clone(),
            },
            FileOp::Write {
                path: bad_target,
                content: "cannot replace directory".to_string(),
            },
        ],
    );

    assert!(result.is_err());
    assert_eq!(fs::read_to_string(modified)?, "old");
    assert_eq!(fs::read_to_string(deleted)?, "gone");
    assert!(!config.gov_root.join(".migrate-stage").exists());
    assert!(!config.gov_root.join(".migrate-backup").exists());
    Ok(())
}

#[test]
fn execute_ops_retains_backup_and_reports_e0903_when_rollback_fails()
-> Result<(), Box<dyn std::error::Error>> {
    let temp_dir = tempfile::tempdir()?;
    let config = test_config(&temp_dir);
    fs::create_dir_all(&config.gov_root)?;
    let victim = config.gov_root.join("victim");
    let bad_target = config.gov_root.join("bad-target");
    fs::write(&victim, "old")?;
    fs::create_dir(&bad_target)?;

    let error = match execute_ops(
        &config,
        &[
            FileOp::Write {
                path: victim.clone(),
                content: "new".to_string(),
            },
            FileOp::Delete {
                path: victim.clone(),
            },
            FileOp::Write {
                path: victim.join("child"),
                content: "forces victim to become a directory".to_string(),
            },
            FileOp::Write {
                path: bad_target,
                content: "cannot replace directory".to_string(),
            },
        ],
    ) {
        Ok(()) => return Err("rollback unexpectedly succeeded".into()),
        Err(error) => error,
    };

    assert_eq!(error.code, DiagnosticCode::E0903UnexpectedError);
    assert!(error.message.contains("restoration may be incomplete"));
    assert!(error.message.contains("recovery backup retained at"));
    assert!(!config.gov_root.join(".migrate-stage").exists());
    assert!(config.gov_root.join(".migrate-backup").exists());
    Ok(())
}