matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

Enhance store config errors by printing (println)

Browse source
This commit is contained in:
Matthias Beyer 2016-03-25 15:08:44 +01:00
parent f8c9234a7a
commit 6a407365bd
1 changed files with 41 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

46
libimagstore/src/configuration.rs
View file

@ -42,6 +42,8 @@ use toml::Value;
/// ///
pub fn config_is_valid(config: &Option<Value>) -> bool { pub fn config_is_valid(config: &Option<Value>) -> bool {
use std::collections::BTreeMap; use std::collections::BTreeMap;
use std::io::Write;
use std::io::stderr;
if config.is_none() { if config.is_none() {
return true; return true;
@ -56,8 +58,15 @@ pub fn config_is_valid(config: &Option<Value>) -> bool {
_ => false, _ => false,
} }
}), }),
_ => false _ => {
}).unwrap_or(false) write!(stderr(), "Key '{}' in store config should contain an array", key)
.ok();
false
}
}).unwrap_or_else(|| {
write!(stderr(), "Required key '{}' is not in store config", key).ok();
false
})
} }
/// Check that /// Check that
@ -79,8 +88,8 @@ pub fn config_is_valid(config: &Option<Value>) -> bool {
match section_table { // which is match section_table { // which is
&Value::Table(ref section_table) => // a table &Value::Table(ref section_table) => // a table
section_table section_table
.values() // which has values, .iter() // which has values,
.all(|cfg| { // and all of these values .all(|(inner_key, cfg)| { // and all of these values
match cfg { match cfg {
&Value::Table(ref hook_config) => { // are tables &Value::Table(ref hook_config) => { // are tables
hook_config.get(key) // with a key hook_config.get(key) // with a key
@ -88,13 +97,25 @@ pub fn config_is_valid(config: &Option<Value>) -> bool {
.map(|hook_aspect| f(&hook_aspect)) .map(|hook_aspect| f(&hook_aspect))
.unwrap_or(false) .unwrap_or(false)
}, },
_ => false, _ => {
write!(stderr(), "Store config expects '{}' to be in '{}.{}', but isn't.",
key, section, inner_key).ok();
false
}
} }
}), }),
_ => false, _ => {
write!(stderr(), "Store config expects '{}' to be a Table, but isn't.",
section).ok();
false
}
} }
}) })
.unwrap_or(false) .unwrap_or_else(|| {
write!(stderr(), "Store config expects section '{}' to be present, but isn't.",
section).ok();
false
})
} }
match config { match config {
@ -113,16 +134,21 @@ pub fn config_is_valid(config: &Option<Value>) -> bool {
// The section "hooks" has maps which have a key "aspect" which has a value of type // The section "hooks" has maps which have a key "aspect" which has a value of type
// String // String
check_all_inner_maps_have_key_with(t, "hooks", "aspect", |asp| { check_all_inner_maps_have_key_with(t, "hooks", "aspect", |asp| {
match asp { &Value::String(_) => true, _ => false } let res = match asp { &Value::String(_) => true, _ => false };
res
}) && }) &&
// The section "aspects" has maps which have a key "parllel" which has a value of type // The section "aspects" has maps which have a key "parllel" which has a value of type
// Boolean // Boolean
check_all_inner_maps_have_key_with(t, "aspects", "parallel", |asp| { check_all_inner_maps_have_key_with(t, "aspects", "parallel", |asp| {
match asp { &Value::Boolean(_) => true, _ => false, } let res = match asp { &Value::Boolean(_) => true, _ => false, };
res
}) })
} }
_ => false, _ => {
write!(stderr(), "Store config is no table").ok();
false
},
} }
} }