Merge 'Move all checksum tests behind the feature flag' from Avinash Sajjanshetty

Closes #3704

core/storage/checksum.rs

@@ -94,6 +94,7 @@ impl Default for ChecksumContext {
 }
 
 #[cfg(test)]
+#[cfg(feature = "checksum")]
 mod tests {
     use super::*;
 
@@ -110,7 +111,6 @@ mod tests {
     }
 
     #[test]
-    #[cfg(feature = "checksum")]
     fn test_add_checksum_to_page() {
         let ctx = ChecksumContext::new();
         let mut page = get_random_page();
@@ -128,7 +128,7 @@ mod tests {
     }
 
     #[test]
-    fn test_verify_and_strip_checksum_valid() {
+    fn test_verify_checksum_valid() {
         let ctx = ChecksumContext::new();
         let mut page = get_random_page();
 
@@ -139,8 +139,7 @@ mod tests {
     }
 
     #[test]
-    #[cfg(feature = "checksum")]
-    fn test_verify_and_strip_checksum_mismatch() {
+    fn test_verify_checksum_mismatch() {
         let ctx = ChecksumContext::new();
         let mut page = get_random_page();
 
@@ -165,8 +164,7 @@ mod tests {
     }
 
     #[test]
-    #[cfg(feature = "checksum")]
-    fn test_verify_and_strip_checksum_corrupted_checksum() {
+    fn test_verify_checksum_corrupted_checksum() {
         let ctx = ChecksumContext::new();
         let mut page = get_random_page();