fixed some clippy issues

gfusee · Oct 19, 2024 · 1ab0e0d · 1ab0e0d
1 parent 1f83a51
commit 1ab0e0d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/caching/src/redis/client.rs b/caching/src/redis/client.rs
@@ -31,7 +31,7 @@ impl RedisClient for redis::Client {
         let options = redis::SetOptions::default()
             .with_expiration(SetExpiry::EX(duration));
 
-        if !connection.set_options::<_, _, ()>(key, value, options).await.is_ok() {
+        if connection.set_options::<_, _, ()>(key, value, options).await.is_err() {
             return Err(CachingRedisError::CannotSetValue)
         }
 
@@ -55,7 +55,7 @@ impl RedisClient for redis::Client {
             return Err(CachingRedisError::CannotGetConnection)
         };
 
-        if !redis::cmd("FLUSHALL").exec_async(&mut connection).await.is_ok() {
+        if redis::cmd("FLUSHALL").exec_async(&mut connection).await.is_err() {
             return Err(CachingRedisError::CannotSetValue)
         };
 

diff --git a/caching/src/redis/redis.rs b/caching/src/redis/redis.rs
@@ -253,7 +253,7 @@ mod test {
         let result = caching.get_or_set_cache::<Vec<u8>, _, NovaXError>(key, async {
             // error if serialized
 
-            Ok(panic!())
+            panic!()
         }).await?;
 
         assert_eq!(result, vec![0u8, 1u8]);