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
use derive_builder::Builder;
use git_checks_core::impl_prelude::*;
#[derive(Builder, Debug, Default, Clone, Copy)]
#[builder(field(private))]
pub struct RejectSeparateRoot {}
impl RejectSeparateRoot {
pub fn builder() -> RejectSeparateRootBuilder {
RejectSeparateRootBuilder::default()
}
}
impl Check for RejectSeparateRoot {
fn name(&self) -> &str {
"reject-separate-root"
}
fn check(&self, _: &CheckGitContext, commit: &Commit) -> Result<CheckResult, Box<dyn Error>> {
let mut result = CheckResult::new();
if commit.parents.is_empty() {
result.add_error(format!(
"commit {} not allowed; it is a root commit.",
commit.sha1,
));
}
Ok(result)
}
}
#[cfg(feature = "config")]
pub(crate) mod config {
use git_checks_config::{register_checks, CommitCheckConfig, IntoCheck};
use serde::Deserialize;
#[cfg(test)]
use serde_json::json;
use crate::RejectSeparateRoot;
#[derive(Deserialize, Debug)]
pub struct RejectSeparateRootConfig {}
impl IntoCheck for RejectSeparateRootConfig {
type Check = RejectSeparateRoot;
fn into_check(self) -> Self::Check {
RejectSeparateRoot::default()
}
}
register_checks! {
RejectSeparateRootConfig {
"reject_separate_root" => CommitCheckConfig,
},
}
#[test]
fn test_reject_separate_root_config_empty() {
let json = json!({});
let check: RejectSeparateRootConfig = serde_json::from_value(json).unwrap();
let _ = check.into_check();
}
}
#[cfg(test)]
mod tests {
use git_checks_core::Check;
use crate::test::*;
use crate::RejectSeparateRoot;
const NO_ROOT_TOPIC: &str = "ba3dc3cb09a558c88282742413a2dccb17d444fc";
const WITH_ROOT_TOPIC: &str = "ff560e8798ef7a9d10bf43660695f7155b49b398";
#[test]
fn test_reject_separate_root_builder_default() {
assert!(RejectSeparateRoot::builder().build().is_ok());
}
#[test]
fn test_reject_separate_root_name_commit() {
let check = RejectSeparateRoot::default();
assert_eq!(Check::name(&check), "reject-separate-root");
}
#[test]
fn test_reject_separate_root_no_root() {
let check = RejectSeparateRoot::default();
run_check_ok("test_reject_separate_root_no_root", NO_ROOT_TOPIC, check);
}
#[test]
fn test_reject_separate_root_with_root() {
let check = RejectSeparateRoot::default();
let result = run_check(
"test_reject_separate_root_with_root",
WITH_ROOT_TOPIC,
check,
);
test_result_errors(
result,
&["commit ff560e8798ef7a9d10bf43660695f7155b49b398 not allowed; it is a root commit."],
);
}
}