Skip to content

Commit cd0f987

Browse files
committed
feat: clean up logging a bit
Signed-off-by: James Petersen <jpetersenames@gmail.com>
1 parent 14e3f1c commit cd0f987

2 files changed

Lines changed: 12 additions & 9 deletions

File tree

src/lib.rs

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -50,23 +50,26 @@ impl CheckGroupResult {
5050
pub fn log_group(&self) {
5151
let name = &self.name;
5252
let result = &self.result;
53+
let s = format!("[{}] {}", name, result);
5354
match result {
54-
CheckResultValue::Passed => info!("{}: {}", name, result),
55-
CheckResultValue::Failed(_) => warn!("{}: {}", name, result),
56-
CheckResultValue::Errored(_) => error!("{}: {}", name, result),
57-
CheckResultValue::Unknown => warn!("{}: {}", name, result),
55+
CheckResultValue::Passed => info!("{}", s),
56+
CheckResultValue::Failed(_) => warn!("{}", s),
57+
CheckResultValue::Errored(_) => error!("{}", s),
58+
CheckResultValue::Unknown => warn!("{}", s),
5859
}
5960
}
6061

6162
pub fn log_individual_checks(&self) {
63+
let group_name = &self.name;
6264
for check_result in self.results.iter() {
6365
let name = &check_result.name;
6466
let result = &check_result.result;
67+
let s = format!("[{}] {}: {}", group_name, name, result);
6568
match result {
66-
CheckResultValue::Passed => info!("{}: {}", name, result),
67-
CheckResultValue::Failed(_) => warn!("{}: {}", name, result),
68-
CheckResultValue::Errored(_) => error!("{}: {}", name, result),
69-
CheckResultValue::Unknown => warn!("{}: {}", name, result),
69+
CheckResultValue::Passed => info!("{}", s),
70+
CheckResultValue::Failed(_) => warn!("{}", s),
71+
CheckResultValue::Errored(_) => error!("{}", s),
72+
CheckResultValue::Unknown => warn!("{}", s),
7073
}
7174
}
7275
}

src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ fn main() -> Result<()> {
1717

1818
// Run each check group
1919
for check in checks {
20-
info!("Running {} - {}", check.name(), check.description());
20+
info!("Running Group [{}] - {}", check.name(), check.description());
2121
let check_group_result = check.run();
2222
check_group_result.log_group();
2323
if env::var("EDERA_PREFLIGHT_VERBOSE").unwrap_or_default() == "true" {

0 commit comments

Comments
 (0)