open.source/espanso
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(kvs): fix warnings

Browse Source
This commit is contained in:
Federico Terzi 2021-10-05 22:06:26 +02:00
parent 2ecf86d9fa
commit e112633509
2 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
espanso-kvs/src/lib.rs
View File

@ -44,7 +44,7 @@ mod tests {
let dir = TempDir::new("kvstempconfig").unwrap(); let dir = TempDir::new("kvstempconfig").unwrap();
callback( callback(
&dir.path(), dir.path(),
); );
} }
@ -53,19 +53,19 @@ mod tests {
use_test_directory(|base_dir| { use_test_directory(|base_dir| {
let kvs = get_persistent(base_dir).unwrap(); let kvs = get_persistent(base_dir).unwrap();
assert_eq!(kvs.get::<String>("my_key").unwrap().is_none(), true); assert!(kvs.get::<String>("my_key").unwrap().is_none());
assert_eq!(kvs.get::<bool>("another_key").unwrap().is_none(), true); assert!(kvs.get::<bool>("another_key").unwrap().is_none());
kvs.set("my_key", "test".to_string()).unwrap(); kvs.set("my_key", "test".to_string()).unwrap();
kvs.set("another_key", false).unwrap(); kvs.set("another_key", false).unwrap();
assert_eq!(kvs.get::<String>("my_key").unwrap().unwrap(), "test"); assert_eq!(kvs.get::<String>("my_key").unwrap().unwrap(), "test");
assert_eq!(kvs.get::<bool>("another_key").unwrap().unwrap(), false); assert!(!kvs.get::<bool>("another_key").unwrap().unwrap());
kvs.delete("my_key").unwrap(); kvs.delete("my_key").unwrap();
assert_eq!(kvs.get::<String>("my_key").unwrap().is_none(), true); assert!(kvs.get::<String>("my_key").unwrap().is_none());
assert_eq!(kvs.get::<bool>("another_key").unwrap().unwrap(), false); assert!(!kvs.get::<bool>("another_key").unwrap().unwrap());
}); });
} }
@ -74,12 +74,12 @@ mod tests {
use_test_directory(|base_dir| { use_test_directory(|base_dir| {
let kvs = get_persistent(base_dir).unwrap(); let kvs = get_persistent(base_dir).unwrap();
assert_eq!(kvs.get::<String>("my_key").unwrap().is_none(), true); assert!(kvs.get::<String>("my_key").unwrap().is_none());
kvs.set("my_key", "test".to_string()).unwrap(); kvs.set("my_key", "test".to_string()).unwrap();
assert_eq!(kvs.get::<bool>("my_key").is_err(), true); assert!(kvs.get::<bool>("my_key").is_err());
assert_eq!(kvs.get::<String>("my_key").is_ok(), true); assert!(kvs.get::<String>("my_key").is_ok());
}); });
} }
@ -97,8 +97,8 @@ mod tests {
use_test_directory(|base_dir| { use_test_directory(|base_dir| {
let kvs = get_persistent(base_dir).unwrap(); let kvs = get_persistent(base_dir).unwrap();
assert_eq!(kvs.get::<String>("invalid key name").is_err(), true); assert!(kvs.get::<String>("invalid key name").is_err());
assert_eq!(kvs.get::<String>("").is_err(), true); assert!(kvs.get::<String>("").is_err());
}); });
} }
} }

2
espanso-kvs/src/persistent.rs
View File

@ -115,7 +115,7 @@ impl KVS for PersistentJsonKVS {
} }
fn is_valid_key_name(key: &str) -> bool { fn is_valid_key_name(key: &str) -> bool {
if key.len() == 0 || key.len() > 200 { if key.is_empty() || key.len() > 200 {
return false; return false;
} }