diff options
| author | Matthias Beyer <mail@beyermatthias.de> | 2019-12-01 13:15:37 +0100 |
|---|---|---|
| committer | Matthias Beyer <mail@beyermatthias.de> | 2019-12-22 02:03:35 +0100 |
| commit | 05051ba38393253c41c8832d45efe53c1559eec7 (patch) | |
| tree | 741be8bd3c5035924562840227ce95c36a77a5d0 | |
| parent | d381a59a7b6d2de029977695e5b961b696e916f4 (diff) | |
Fix: Use unwrap_or_else(panic!(...)) as recommended by clippy
instead of expect()ing errors, clippy suggests to use
unwrap_or_else(|| panic!(...)) - use that instead
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
| -rw-r--r-- | tests/version-sync/src/lib.rs | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/version-sync/src/lib.rs b/tests/version-sync/src/lib.rs index b167c74a..7e74f80d 100644 --- a/tests/version-sync/src/lib.rs +++ b/tests/version-sync/src/lib.rs @@ -70,8 +70,8 @@ fn test_all_cargotoml_files() { None }) .for_each(|cargotoml| { - let filecontent = std::fs::read_to_string(&cargotoml).expect(&format!("Failed to read {}", cargotoml.display())); - let toml = filecontent.parse::<Value>().expect(&format!("Failed to parse toml: {}", cargotoml.display())); + let filecontent = std::fs::read_to_string(&cargotoml).unwrap_or_else(|_| panic!("Failed to read {}", cargotoml.display())); + let toml = filecontent.parse::<Value>().unwrap_or_else(|_| panic!("Failed to parse toml: {}", cargotoml.display())); match toml.get("dependencies") { Some(Value::Table(ref tab)) => { @@ -80,7 +80,7 @@ fn test_all_cargotoml_files() { match v { Value::String(s) => assert!(s.contains(current_version)), Value::Table(ref dep) => { - let version = dep.get("version").expect(&format!("No 'version' key for dependencies at {}", cargotoml.display())); + let version = dep.get("version").unwrap_or_else(|| panic!("No 'version' key for dependencies at {}", cargotoml.display())); let version_str = version.as_str().unwrap(); assert!(version_str.contains(current_version), "failed for: {} ('{}')", cargotoml.display(), version_str) }, @@ -100,7 +100,7 @@ fn test_all_cargotoml_files() { match v { Value::String(s) => assert!(s.contains(current_version)), Value::Table(ref dep) => { - let version = dep.get("version").expect(&format!("No 'version' key for dev-dependencies at {}", cargotoml.display())); + let version = dep.get("version").unwrap_or_else(|| panic!("No 'version' key for dev-dependencies at {}", cargotoml.display())); let version_str = version.as_str().unwrap(); assert!(version_str.contains(current_version), "failed for: {} ('{}')", cargotoml.display(), version_str) }, |