[TKN-714] add unit tests to validate anchor account discriminators (#1094)

Author: jshiohaha

.changeset/fruity-impalas-sit.md

@@ -0,0 +1,5 @@
+---
+"@orca-so/whirlpools-program": patch
+---
+
+Add unit tests to validate account discriminators

programs/whirlpool/src/state/adaptive_fee_tier.rs

@@ -149,6 +149,25 @@ impl AdaptiveFeeTier {
     }
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = AdaptiveFeeTier::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:AdaptiveFeeTier | sha256sum | cut -c 1-16
+        // 931090742f92952e
+        assert_eq!(
+            discriminator,
+            [0x93, 0x10, 0x90, 0x74, 0x2f, 0x92, 0x95, 0x2e]
+        );
+    }
+}
+
 #[cfg(test)]
 mod data_layout_tests {
     use anchor_lang::Discriminator;

programs/whirlpool/src/state/config.rs

@@ -99,6 +99,25 @@ pub enum ConfigFeatureFlag {
     TokenBadge(bool),
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = WhirlpoolsConfig::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:WhirlpoolsConfig | sha256sum | cut -c 1-16
+        // 9d1431e0d957c1fe
+        assert_eq!(
+            discriminator,
+            [0x9d, 0x14, 0x31, 0xe0, 0xd9, 0x57, 0xc1, 0xfe]
+        );
+    }
+}
+
 #[cfg(test)]
 mod data_layout_tests {
     use anchor_lang::Discriminator;

programs/whirlpool/src/state/config_extension.rs

@@ -61,6 +61,25 @@ mod whirlpools_config_extension_initialize_tests {
     }
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = WhirlpoolsConfigExtension::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:WhirlpoolsConfigExtension | sha256sum | cut -c 1-16
+        // 0263d7a3f01a993a
+        assert_eq!(
+            discriminator,
+            [0x02, 0x63, 0xd7, 0xa3, 0xf0, 0x1a, 0x99, 0x3a]
+        );
+    }
+}
+
 #[cfg(test)]
 mod whirlpools_config_extension_update_tests {
     use super::*;

programs/whirlpool/src/state/dynamic_tick_array.rs

@@ -949,7 +949,10 @@ mod discriminator_tests {
         // The discriminator is determined by the struct name and not depending on the program id.
         // $ echo -n account:DynamicTickArray | sha256sum | cut -c 1-16
         // 11d8f68ee1c7da38
-        assert_eq!(discriminator, [17, 216, 246, 142, 225, 199, 218, 56]);
+        assert_eq!(
+            discriminator,
+            [0x11, 0xd8, 0xf6, 0x8e, 0xe1, 0xc7, 0xda, 0x38]
+        );
     }
 }
 

programs/whirlpool/src/state/fee_tier.rs

@@ -38,6 +38,25 @@ impl FeeTier {
     }
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = FeeTier::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:FeeTier | sha256sum | cut -c 1-16
+        // 384b9f4c8e44be69
+        assert_eq!(
+            discriminator,
+            [0x38, 0x4b, 0x9f, 0x4c, 0x8e, 0x44, 0xbe, 0x69]
+        );
+    }
+}
+
 #[cfg(test)]
 mod data_layout_tests {
     use anchor_lang::Discriminator;

programs/whirlpool/src/state/fixed_tick_array.rs

@@ -253,6 +253,25 @@ mod array_update_tests {
     }
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = TickArray::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:TickArray | sha256sum | cut -c 1-16
+        // 4561bdbe6e0742bb
+        assert_eq!(
+            discriminator,
+            [0x45, 0x61, 0xbd, 0xbe, 0x6e, 0x07, 0x42, 0xbb]
+        );
+    }
+}
+
 #[cfg(test)]
 mod data_layout_tests {
     use crate::state::NUM_REWARDS;

programs/whirlpool/src/state/lock_config.rs

@@ -85,6 +85,25 @@ mod lock_config_initialize_tests {
     }
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = LockConfig::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:LockConfig | sha256sum | cut -c 1-16
+        // 6a2fee9f7c0ca0c0
+        assert_eq!(
+            discriminator,
+            [0x6a, 0x2f, 0xee, 0x9f, 0x7c, 0x0c, 0xa0, 0xc0]
+        );
+    }
+}
+
 #[cfg(test)]
 mod data_layout_tests {
     use anchor_lang::Discriminator;

programs/whirlpool/src/state/oracle.rs

@@ -480,6 +480,25 @@ impl<'info> OracleAccessor<'info> {
     }
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = Oracle::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:Oracle | sha256sum | cut -c 1-16
+        // 8bc283b38cb3e5f4
+        assert_eq!(
+            discriminator,
+            [0x8b, 0xc2, 0x83, 0xb3, 0x8c, 0xb3, 0xe5, 0xf4]
+        );
+    }
+}
+
 #[cfg(test)]
 mod data_layout_tests {
     use super::*;

programs/whirlpool/src/state/position.rs

@@ -332,6 +332,25 @@ pub mod position_builder {
     }
 }
 
+#[cfg(test)]
+mod discriminator_tests {
+    use anchor_lang::Discriminator;
+
+    use super::*;
+
+    #[test]
+    fn test_discriminator() {
+        let discriminator = Position::discriminator();
+        // The discriminator is determined by the struct name and not depending on the program id.
+        // $ echo -n account:Position | sha256sum | cut -c 1-16
+        // aabc8fe47a40f7d0
+        assert_eq!(
+            discriminator,
+            [0xaa, 0xbc, 0x8f, 0xe4, 0x7a, 0x40, 0xf7, 0xd0]
+        );
+    }
+}
+
 #[cfg(test)]
 mod data_layout_tests {
     use anchor_lang::Discriminator;