remove duplicate sudo_set_subnet_owner_hotkey

Author: JohnReedV

pallets/admin-utils/src/lib.rs

@@ -1667,31 +1667,31 @@ pub mod pallet {
             Ok(())
         }
 
-        /// Change the SubnetOwnerHotkey for a given subnet.
-        ///
-        /// # Arguments
-        /// * `origin` - The origin of the call, which must be the subnet owner.
-        /// * `netuid` - The unique identifier for the subnet.
-        /// * `hotkey` - The new hotkey for the subnet owner.
-        ///
-        /// # Errors
-        /// * `BadOrigin` - If the caller is not the subnet owner or root account.
-        ///
-        /// # Weight
-        /// Weight is handled by the `#[pallet::weight]` attribute.
-        #[pallet::call_index(64)]
-        #[pallet::weight((0, DispatchClass::Operational, Pays::No))]
-        pub fn sudo_set_subnet_owner_hotkey(
-            origin: OriginFor<T>,
-            netuid: NetUid,
-            hotkey: <T as frame_system::Config>::AccountId,
-        ) -> DispatchResult {
-            pallet_subtensor::Pallet::<T>::ensure_subnet_owner(origin.clone(), netuid)?;
-            pallet_subtensor::Pallet::<T>::set_subnet_owner_hotkey(netuid, &hotkey);
-
-            log::debug!("SubnetOwnerHotkeySet( netuid: {netuid:?}, hotkey: {hotkey:?} )");
-            Ok(())
-        }
+        // /// Change the SubnetOwnerHotkey for a given subnet.
+        // ///
+        // /// # Arguments
+        // /// * `origin` - The origin of the call, which must be the subnet owner.
+        // /// * `netuid` - The unique identifier for the subnet.
+        // /// * `hotkey` - The new hotkey for the subnet owner.
+        // ///
+        // /// # Errors
+        // /// * `BadOrigin` - If the caller is not the subnet owner or root account.
+        // ///
+        // /// # Weight
+        // /// Weight is handled by the `#[pallet::weight]` attribute.
+        // #[pallet::call_index(64)]
+        // #[pallet::weight((0, DispatchClass::Operational, Pays::No))]
+        // pub fn sudo_set_subnet_owner_hotkey(
+        //     origin: OriginFor<T>,
+        //     netuid: NetUid,
+        //     hotkey: <T as frame_system::Config>::AccountId,
+        // ) -> DispatchResult {
+        //     pallet_subtensor::Pallet::<T>::ensure_subnet_owner(origin.clone(), netuid)?;
+        //     pallet_subtensor::Pallet::<T>::set_subnet_owner_hotkey(netuid, &hotkey);
+
+        //     log::debug!("SubnetOwnerHotkeySet( netuid: {netuid:?}, hotkey: {hotkey:?} )");
+        //     Ok(())
+        // }
 
         ///
         ///

pallets/admin-utils/src/tests/mod.rs

@@ -1745,48 +1745,48 @@ fn test_sets_a_lower_value_clears_small_nominations() {
     });
 }
 
-#[test]
-fn test_sudo_set_subnet_owner_hotkey() {
-    new_test_ext().execute_with(|| {
-        let netuid = NetUid::from(1);
-
-        let coldkey: U256 = U256::from(1);
-        let hotkey: U256 = U256::from(2);
-        let new_hotkey: U256 = U256::from(3);
-
-        let coldkey_origin = <<Test as Config>::RuntimeOrigin>::signed(coldkey);
-        let root = RuntimeOrigin::root();
-        let random_account = RuntimeOrigin::signed(U256::from(123456));
-
-        pallet_subtensor::SubnetOwner::<Test>::insert(netuid, coldkey);
-        pallet_subtensor::SubnetOwnerHotkey::<Test>::insert(netuid, hotkey);
-        assert_eq!(
-            pallet_subtensor::SubnetOwnerHotkey::<Test>::get(netuid),
-            hotkey
-        );
-
-        assert_ok!(AdminUtils::sudo_set_subnet_owner_hotkey(
-            coldkey_origin,
-            netuid,
-            new_hotkey
-        ));
-
-        assert_eq!(
-            pallet_subtensor::SubnetOwnerHotkey::<Test>::get(netuid),
-            new_hotkey
-        );
-
-        assert_noop!(
-            AdminUtils::sudo_set_subnet_owner_hotkey(random_account, netuid, new_hotkey),
-            DispatchError::BadOrigin
-        );
-
-        assert_noop!(
-            AdminUtils::sudo_set_subnet_owner_hotkey(root, netuid, new_hotkey),
-            DispatchError::BadOrigin
-        );
-    });
-}
+// #[test]
+// fn test_sudo_set_subnet_owner_hotkey() {
+//     new_test_ext().execute_with(|| {
+//         let netuid = NetUid::from(1);
+
+//         let coldkey: U256 = U256::from(1);
+//         let hotkey: U256 = U256::from(2);
+//         let new_hotkey: U256 = U256::from(3);
+
+//         let coldkey_origin = <<Test as Config>::RuntimeOrigin>::signed(coldkey);
+//         let root = RuntimeOrigin::root();
+//         let random_account = RuntimeOrigin::signed(U256::from(123456));
+
+//         pallet_subtensor::SubnetOwner::<Test>::insert(netuid, coldkey);
+//         pallet_subtensor::SubnetOwnerHotkey::<Test>::insert(netuid, hotkey);
+//         assert_eq!(
+//             pallet_subtensor::SubnetOwnerHotkey::<Test>::get(netuid),
+//             hotkey
+//         );
+
+//         assert_ok!(AdminUtils::sudo_set_subnet_owner_hotkey(
+//             coldkey_origin,
+//             netuid,
+//             new_hotkey
+//         ));
+
+//         assert_eq!(
+//             pallet_subtensor::SubnetOwnerHotkey::<Test>::get(netuid),
+//             new_hotkey
+//         );
+
+//         assert_noop!(
+//             AdminUtils::sudo_set_subnet_owner_hotkey(random_account, netuid, new_hotkey),
+//             DispatchError::BadOrigin
+//         );
+
+//         assert_noop!(
+//             AdminUtils::sudo_set_subnet_owner_hotkey(root, netuid, new_hotkey),
+//             DispatchError::BadOrigin
+//         );
+//     });
+// }
 
 // cargo test --package pallet-admin-utils --lib -- tests::test_sudo_set_ema_halving --exact --show-output
 #[test]