From b7f5392867c1454ac2bda1334cd815f5d01b6301 Mon Sep 17 00:00:00 2001
From: mulhern <amulhern@redhat.com>
Date: Mon, 26 Aug 2024 22:14:15 -0400
Subject: [PATCH] Remove unnecessary mut modifiers in a test

Signed-off-by: mulhern <amulhern@redhat.com>
---
 .../strat_engine/backstore/cache_tier.rs      | 30 ++++++++-----------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git a/src/engine/strat_engine/backstore/cache_tier.rs b/src/engine/strat_engine/backstore/cache_tier.rs
index 9a6bc4027d..493f9c2cf3 100644
--- a/src/engine/strat_engine/backstore/cache_tier.rs
+++ b/src/engine/strat_engine/backstore/cache_tier.rs
@@ -412,9 +412,8 @@ mod tests {
             // the tier.
             let cache_metadata_size = cache_tier.meta_segments.size();
 
-            let mut metadata_size = cache_tier.block_mgr.sizer().metadata_size();
-            let mut size = cache_tier.block_mgr.sizer().size();
-            let mut allocated = cache_tier.cache_segments.size();
+            let metadata_size = cache_tier.block_mgr.sizer().metadata_size();
+            let size = cache_tier.block_mgr.sizer().size();
 
             assert_eq!(
                 cache_tier.block_mgr.sizer().avail_space(),
@@ -422,7 +421,7 @@ mod tests {
             );
             assert_eq!(
                 size.sectors() - metadata_size.sectors(),
-                allocated + cache_metadata_size
+                cache_tier.cache_segments.size() + cache_metadata_size
             );
 
             let (_, (cache, meta)) = cache_tier
@@ -442,12 +441,10 @@ mod tests {
                 cache_tier.block_mgr.sizer().metadata_size().sectors() > metadata_size.sectors()
             );
 
-            metadata_size = cache_tier.block_mgr.sizer().metadata_size();
-            size = cache_tier.block_mgr.sizer().size();
-            allocated = cache_tier.cache_segments.size();
             assert_eq!(
-                size.sectors() - metadata_size.sectors(),
-                allocated + cache_metadata_size
+                cache_tier.block_mgr.sizer().size().sectors()
+                    - cache_tier.block_mgr.sizer().metadata_size().sectors(),
+                cache_tier.cache_segments.size() + cache_metadata_size
             );
 
             cache_tier.destroy().unwrap();
@@ -497,9 +494,8 @@ mod tests {
             // the tier.
             let cache_metadata_size = cache_tier.meta_segments.size();
 
-            let mut metadata_size = cache_tier.block_mgr.sizer().metadata_size();
-            let mut size = cache_tier.block_mgr.sizer().size();
-            let mut allocated = cache_tier.cache_segments.size();
+            let metadata_size = cache_tier.block_mgr.sizer().metadata_size();
+            let size = cache_tier.block_mgr.sizer().size();
 
             assert_eq!(
                 cache_tier.block_mgr.sizer().avail_space(),
@@ -507,7 +503,7 @@ mod tests {
             );
             assert_eq!(
                 size.sectors() - metadata_size.sectors(),
-                allocated + cache_metadata_size
+                cache_tier.cache_segments.size() + cache_metadata_size
             );
 
             let (_, (cache, meta)) = cache_tier.add(pool_uuid, devices2).unwrap();
@@ -525,12 +521,10 @@ mod tests {
                 cache_tier.block_mgr.sizer().metadata_size().sectors() > metadata_size.sectors()
             );
 
-            metadata_size = cache_tier.block_mgr.sizer().metadata_size();
-            size = cache_tier.block_mgr.sizer().size();
-            allocated = cache_tier.cache_segments.size();
             assert_eq!(
-                size.sectors() - metadata_size.sectors(),
-                allocated + cache_metadata_size
+                cache_tier.block_mgr.sizer().size().sectors()
+                    - cache_tier.block_mgr.sizer().metadata_size().sectors(),
+                cache_tier.cache_segments.size() + cache_metadata_size
             );
 
             cache_tier.destroy().unwrap();