Put feature gate attribute under test attribute

Either way is fine, just pick one and be uniform.

bitcoin/src/blockdata/transaction.rs

@@ -1361,8 +1361,8 @@ mod tests {
     }
 
     // We temporarily abuse `Transaction` for testing consensus serde adapter.
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn consensus_serde() {
         use crate::consensus::serde as con_serde;
         let json = "\"010000000001010000000000000000000000000000000000000000000000000000000000000000ffffffff3603da1b0e00045503bd5704c7dd8a0d0ced13bb5785010800000000000a636b706f6f6c122f4e696e6a61506f6f6c2f5345475749542fffffffff02b4e5a212000000001976a914876fbb82ec05caa6af7a3b5e5a983aae6c6cc6d688ac0000000000000000266a24aa21a9edf91c46b49eb8a29089980f02ee6b57e7d63d33b18b4fddac2bcd7db2a39837040120000000000000000000000000000000000000000000000000000000000000000000000000\"";

bitcoin/src/crypto/key.rs

@@ -1222,8 +1222,8 @@ mod tests {
         assert!(upk.wpubkey_hash().is_err());
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn test_key_serde() {
         use serde_test::{assert_tokens, Configure, Token};
 

bitcoin/src/network/mod.rs

@@ -490,8 +490,8 @@ mod tests {
         }
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn serde_as_core_arg() {
         #[derive(Serialize, Deserialize, PartialEq, Debug)]
         struct T {

bitcoin/src/pow.rs

@@ -1643,8 +1643,8 @@ mod tests {
         assert_eq!(got, want);
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn u256_serde() {
         let check = |uint, hex| {
             let json = format!("\"{}\"", hex);
@@ -1932,8 +1932,8 @@ mod tests {
         assert_eq!(back, target)
     }
 
-    #[cfg(feature = "std")]
     #[test]
+    #[cfg(feature = "std")]
     fn work_log2() {
         // Compare work log2 to historical Bitcoin Core values found in Core logs.
         let tests: &[(u128, f64)] = &[

hashes/src/hash160.rs

@@ -101,8 +101,8 @@ mod tests {
         }
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn ripemd_serde() {
         use serde_test::{assert_tokens, Configure, Token};
 

hashes/src/hmac.rs

@@ -289,8 +289,8 @@ mod tests {
         }
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn hmac_sha512_serde() {
         use serde_test::{assert_tokens, Configure, Token};
 

hashes/src/ripemd160.rs

@@ -493,8 +493,8 @@ mod tests {
         }
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn ripemd_serde() {
         use serde_test::{assert_tokens, Configure, Token};
 

hashes/src/sha1.rs

@@ -197,8 +197,8 @@ mod tests {
         }
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn sha1_serde() {
         use serde_test::{assert_tokens, Configure, Token};
 

hashes/src/sha256.rs

@@ -1063,8 +1063,8 @@ mod tests {
         assert_eq!(got, want);
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn sha256_serde() {
         use serde_test::{assert_tokens, Configure, Token};
 

hashes/src/sha256d.rs

@@ -107,8 +107,8 @@ mod tests {
         assert_eq!(rinsed, hash)
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn sha256_serde() {
         use serde_test::{assert_tokens, Configure, Token};
 

hashes/src/sha512.rs

@@ -384,8 +384,8 @@ mod tests {
         }
     }
 
-    #[cfg(feature = "serde")]
     #[test]
+    #[cfg(feature = "serde")]
     fn sha512_serde() {
         use serde_test::{assert_tokens, Configure, Token};