Revert bad rename for SubnetNotExists error

Author: gztensor

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

@@ -2638,7 +2638,7 @@ fn test_trim_to_max_allowed_uids() {
                 NetUid::from(42),
                 new_max_n
             ),
-            pallet_subtensor::Error::<Test>::MechanismDoesNotExist
+            pallet_subtensor::Error::<Test>::SubnetNotExists
         );
 
         // New max n less than lower bound

pallets/subtensor/src/coinbase/root.rs

@@ -369,7 +369,7 @@ impl<T: Config> Pallet<T> {
         // 1. --- The network exists?
         ensure!(
             Self::if_subnet_exist(netuid) && netuid != NetUid::ROOT,
-            Error::<T>::MechanismDoesNotExist
+            Error::<T>::SubnetNotExists
         );
 
         // 2. --- Perform the cleanup before removing the network.

pallets/subtensor/src/lib.rs

@@ -1985,7 +1985,7 @@ pub enum CustomTransactionError {
     ColdkeyInSwapSchedule,
     StakeAmountTooLow,
     BalanceTooLow,
-    SubnetDoesntExist,
+    SubnetNotExists,
     HotkeyAccountDoesntExist,
     NotEnoughStakeToWithdraw,
     RateLimitExceeded,
@@ -2010,7 +2010,7 @@ impl From<CustomTransactionError> for u8 {
             CustomTransactionError::ColdkeyInSwapSchedule => 0,
             CustomTransactionError::StakeAmountTooLow => 1,
             CustomTransactionError::BalanceTooLow => 2,
-            CustomTransactionError::SubnetDoesntExist => 3,
+            CustomTransactionError::SubnetNotExists => 3,
             CustomTransactionError::HotkeyAccountDoesntExist => 4,
             CustomTransactionError::NotEnoughStakeToWithdraw => 5,
             CustomTransactionError::RateLimitExceeded => 6,

pallets/subtensor/src/staking/recycle_alpha.rs

@@ -23,10 +23,7 @@ impl<T: Config> Pallet<T> {
     ) -> DispatchResult {
         let coldkey: T::AccountId = ensure_signed(origin)?;
 
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
 
         ensure!(
             !netuid.is_root(),
@@ -91,10 +88,7 @@ impl<T: Config> Pallet<T> {
     ) -> DispatchResult {
         let coldkey = ensure_signed(origin)?;
 
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
 
         ensure!(
             !netuid.is_root(),

pallets/subtensor/src/staking/remove_stake.rs

@@ -443,10 +443,7 @@ impl<T: Config> Pallet<T> {
 
     pub fn destroy_alpha_in_out_stakes(netuid: NetUid) -> DispatchResult {
         // 1) Ensure the subnet exists.
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
 
         // 2) Owner / lock cost.
         let owner_coldkey: T::AccountId = SubnetOwner::<T>::get(netuid);

pallets/subtensor/src/staking/set_children.rs

@@ -62,10 +62,7 @@ impl<T: Config> Pallet<T> {
         );
 
         // Check that the network we are trying to create the child on exists.
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
 
         // Check that the coldkey owns the hotkey.
         ensure!(

pallets/subtensor/src/subnets/registration.rs

@@ -78,10 +78,7 @@ impl<T: Config> Pallet<T> {
             !netuid.is_root(),
             Error::<T>::RegistrationNotPermittedOnRootSubnet
         );
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
 
         // --- 3. Ensure the passed network allows registrations.
         ensure!(
@@ -236,10 +233,7 @@ impl<T: Config> Pallet<T> {
             !netuid.is_root(),
             Error::<T>::RegistrationNotPermittedOnRootSubnet
         );
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
 
         // --- 3. Ensure the passed network allows registrations.
         ensure!(

pallets/subtensor/src/subnets/subnet.rs

@@ -343,18 +343,15 @@ impl<T: Config> Pallet<T> {
     ///
     /// # Raises
     ///
-    /// * `Error::<T>::MechanismDoesNotExist`: If the subnet does not exist.
+    /// * `Error::<T>::SubnetNotExists`: If the subnet does not exist.
     /// * `DispatchError::BadOrigin`: If the caller is not the subnet owner.
     /// * `Error::<T>::FirstEmissionBlockNumberAlreadySet`: If the last emission block number has already been set.
     ///
     /// # Returns
     ///
     /// * `DispatchResult`: A result indicating the success or failure of the operation.
     pub fn do_start_call(origin: T::RuntimeOrigin, netuid: NetUid) -> DispatchResult {
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
         Self::ensure_subnet_owner(origin, netuid)?;
         ensure!(
             FirstEmissionBlockNumber::<T>::get(netuid).is_none(),

pallets/subtensor/src/subnets/uids.rs

@@ -133,10 +133,7 @@ impl<T: Config> Pallet<T> {
 
     pub fn trim_to_max_allowed_uids(netuid: NetUid, max_n: u16) -> DispatchResult {
         // Reasonable limits
-        ensure!(
-            Self::if_subnet_exist(netuid),
-            Error::<T>::MechanismDoesNotExist
-        );
+        ensure!(Self::if_subnet_exist(netuid), Error::<T>::SubnetNotExists);
         ensure!(
             max_n >= MinAllowedUids::<T>::get(netuid),
             Error::<T>::InvalidValue

pallets/subtensor/src/tests/children.rs

@@ -62,7 +62,7 @@ fn test_do_set_child_singular_network_does_not_exist() {
                 netuid,
                 vec![(proportion, child)]
             ),
-            Error::<Test>::MechanismDoesNotExist
+            Error::<Test>::SubnetNotExists
         );
     });
 }
@@ -328,7 +328,7 @@ fn test_add_singular_child() {
                 netuid,
                 vec![(u64::MAX, child)]
             ),
-            Err(Error::<Test>::MechanismDoesNotExist.into())
+            Err(Error::<Test>::SubnetNotExists.into())
         );
         add_network(netuid, 1, 0);
         step_rate_limit(&TransactionType::SetChildren, netuid);
@@ -472,7 +472,7 @@ fn test_do_set_empty_children_network_does_not_exist() {
                 netuid,
                 vec![]
             ),
-            Error::<Test>::MechanismDoesNotExist
+            Error::<Test>::SubnetNotExists
         );
     });
 }
@@ -601,7 +601,7 @@ fn test_do_schedule_children_multiple_network_does_not_exist() {
                 netuid,
                 vec![(proportion, child1)]
             ),
-            Error::<Test>::MechanismDoesNotExist
+            Error::<Test>::SubnetNotExists
         );
     });
 }
@@ -1200,7 +1200,7 @@ fn test_do_revoke_children_multiple_network_does_not_exist() {
                 netuid,
                 vec![(u64::MAX / 2, child1), (u64::MAX / 2, child2)]
             ),
-            Error::<Test>::MechanismDoesNotExist
+            Error::<Test>::SubnetNotExists
         );
     });
 }