diff --git a/espanso-kvs/src/lib.rs b/espanso-kvs/src/lib.rs
index 05e5fd4..11b08ae 100644
--- a/espanso-kvs/src/lib.rs
+++ b/espanso-kvs/src/lib.rs
@@ -44,7 +44,7 @@ mod tests {
     let dir = TempDir::new("kvstempconfig").unwrap();
 
     callback(
-      &dir.path(),
+      dir.path(),
     );
   }
 
@@ -53,19 +53,19 @@ mod tests {
     use_test_directory(|base_dir| {
       let kvs = get_persistent(base_dir).unwrap();
 
-      assert_eq!(kvs.get::<String>("my_key").unwrap().is_none(), true);
-      assert_eq!(kvs.get::<bool>("another_key").unwrap().is_none(), true);
+      assert!(kvs.get::<String>("my_key").unwrap().is_none());
+      assert!(kvs.get::<bool>("another_key").unwrap().is_none());
 
       kvs.set("my_key", "test".to_string()).unwrap();
       kvs.set("another_key", false).unwrap();
 
       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();
 
-      assert_eq!(kvs.get::<String>("my_key").unwrap().is_none(), true);
-      assert_eq!(kvs.get::<bool>("another_key").unwrap().unwrap(), false);
+      assert!(kvs.get::<String>("my_key").unwrap().is_none());
+      assert!(!kvs.get::<bool>("another_key").unwrap().unwrap());
     });
   }
 
@@ -74,12 +74,12 @@ mod tests {
     use_test_directory(|base_dir| {
       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();
 
-      assert_eq!(kvs.get::<bool>("my_key").is_err(), true);
-      assert_eq!(kvs.get::<String>("my_key").is_ok(), true);
+      assert!(kvs.get::<bool>("my_key").is_err());
+      assert!(kvs.get::<String>("my_key").is_ok());
     });
   }
 
@@ -97,8 +97,8 @@ mod tests {
     use_test_directory(|base_dir| {
       let kvs = get_persistent(base_dir).unwrap();
 
-      assert_eq!(kvs.get::<String>("invalid key name").is_err(), true);
-      assert_eq!(kvs.get::<String>("").is_err(), true);
+      assert!(kvs.get::<String>("invalid key name").is_err());
+      assert!(kvs.get::<String>("").is_err());
     });
   }
 }
diff --git a/espanso-kvs/src/persistent.rs b/espanso-kvs/src/persistent.rs
index ab745f0..034ea3e 100644
--- a/espanso-kvs/src/persistent.rs
+++ b/espanso-kvs/src/persistent.rs
@@ -115,7 +115,7 @@ impl KVS for PersistentJsonKVS {
 }
 
 fn is_valid_key_name(key: &str) -> bool {
-  if key.len() == 0 || key.len() > 200 {
+  if key.is_empty() || key.len() > 200 {
     return false;
   }