monochromatic-git-policy-cli 0.5.29

Repository-owned Git policy wrapper: forwards Git commands and checks commits against configured policies.
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

//! Controls for the arguments real Git is given for a commit it builds, and for reading the parents of what it made.

use super::*;

fn arguments(words: &[&str]) -> Vec<OsString> {
    return words.iter().map(OsString::from).collect();
}

fn strings(words: &[&str]) -> Vec<String> {
    return words.iter().map(|word| { return (*word).to_string(); }).collect();
}

/// The parents come from the header block only, in order; a line in the message that looks like a parent is not one.
#[test]
fn parents_come_from_the_header_block_in_order() {
    let merge: &[u8] = b"tree 4b825dc642cb6eb9a060e54bf8d69288fbee4904\nparent 1111\nparent 2222\nauthor A <a@b> 1 +0000\n\nparent 3333\n";
    assert_eq!(parents_of(merge), strings(&["1111", "2222"]));
    let root: &[u8] = b"tree 4b825dc642cb6eb9a060e54bf8d69288fbee4904\nauthor A <a@b> 1 +0000\n\nparent 9999\n";
    assert_eq!(parents_of(root), Vec::<String>::new());
}

/// A commit of named paths is given neither the paths nor the separator nor `--only`, and keeps its message options.
#[test]
fn named_paths_and_the_separator_are_left_out_for_git() {
    let (globals, commit) = private_invocation(
        &arguments(&["commit", "--only", "-m", "subject", "--", "a.txt", "dir/b.txt"]),
        &strings(&["a.txt", "dir/b.txt"]),
        TransactionMode::ExplicitPath,
        false,
        false,
    );
    assert!(globals.is_empty());
    assert_eq!(commit, strings(&["-m", "subject"]));
}

/// Global options that would send Git elsewhere are dropped, the others are kept before the command.
#[test]
fn global_options_that_redirect_git_are_dropped() {
    let (globals, commit) = private_invocation(
        &arguments(&["-C", "/elsewhere", "-c", "core.editor=true", "commit", "-S", "-m", "signed"]),
        &[],
        TransactionMode::Index,
        false,
        false,
    );
    assert_eq!(globals, strings(&["-c", "core.editor=true"]));
    assert_eq!(commit, strings(&["-S", "-m", "signed"]));
}

/// A selection the private index already holds is not given to Git again, with its pathspec file and its pathspecs.
#[test]
fn a_selection_the_private_index_holds_is_not_given_to_git_again() {
    let line: Vec<OsString> = arguments(&["commit", "-i", "-m", "all", "--pathspec-from-file=list", "--pathspec-file-nul", "--", "a.txt"]);
    assert_eq!(private_invocation(&line, &strings(&["a.txt"]), TransactionMode::Index, false, true).1, strings(&["-m", "all"]));
    let patch: Vec<OsString> = arguments(&["commit", "--patch", "-m", "p"]);
    assert_eq!(private_invocation(&patch, &[], TransactionMode::Index, false, true).1, strings(&["-m", "p"]));
    assert_eq!(private_invocation(&patch, &[], TransactionMode::Index, false, false).1, strings(&["--patch", "-m", "p"]));
}

/// `-a` is removed once it has staged into the private index, whether alone or in a cluster, and not otherwise.
#[test]
fn commit_all_is_removed_only_when_the_index_already_holds_it() {
    let line: Vec<OsString> = arguments(&["commit", "-am", "all"]);
    assert_eq!(private_invocation(&line, &[], TransactionMode::Index, true, false).1, strings(&["-m", "all"]));
    assert_eq!(private_invocation(&line, &[], TransactionMode::Index, false, false).1, strings(&["-am", "all"]));
}