From 70801af5cf7463393024fccdadd74038a905a3f7 Mon Sep 17 00:00:00 2001
From: Igor Papandinas <26460174+ipapandinas@users.noreply.github.com>
Date: Tue, 4 Mar 2025 17:26:59 +0100
Subject: [PATCH] remove migration test

---
 pallets/dapp-staking/src/test/tests.rs | 36 --------------------------
 runtime/shiden/src/lib.rs              |  2 +-
 2 files changed, 1 insertion(+), 37 deletions(-)

diff --git a/pallets/dapp-staking/src/test/tests.rs b/pallets/dapp-staking/src/test/tests.rs
index 76fcc2cd8..2922419d9 100644
--- a/pallets/dapp-staking/src/test/tests.rs
+++ b/pallets/dapp-staking/src/test/tests.rs
@@ -4238,42 +4238,6 @@ fn set_static_tier_params_works() {
     })
 }
 
-#[test]
-fn active_update_bonus_status() {
-    ExtBuilder::default()
-        .with_max_bonus_safe_moves(2)
-        .build_and_execute(|| {
-            let account_1 = 1;
-            let contract_1 = MockSmartContract::wasm(1 as AccountId);
-
-            crate::migration::v8::StakerInfo::<Test>::set(
-                &account_1,
-                &contract_1,
-                Some(crate::migration::v8::SingularStakingInfo {
-                    previous_staked: Default::default(),
-                    staked: Default::default(),
-                    loyal_staker: true,
-                }),
-            );
-
-            assert_ok!(crate::Pallet::<Test>::do_update(
-                crate::Pallet::<Test>::max_call_weight()
-            ));
-
-            let expected_bonus_status = *BonusStatusWrapperFor::<Test>::default();
-            let expected_staker_info = SingularStakingInfo {
-                previous_staked: Default::default(),
-                staked: Default::default(),
-                bonus_status: expected_bonus_status,
-            };
-
-            assert_eq!(
-                StakerInfo::<Test>::get(&account_1, &contract_1),
-                Some(expected_staker_info)
-            );
-        })
-}
-
 // Tests a previous bug where previous_stake was storing future stake amounts (amounts that should be eligible in the next era)
 #[test]
 fn previous_stake_unchanged_for_future_era_staking() {
diff --git a/runtime/shiden/src/lib.rs b/runtime/shiden/src/lib.rs
index a5a8affde..664bfb02e 100644
--- a/runtime/shiden/src/lib.rs
+++ b/runtime/shiden/src/lib.rs
@@ -1285,7 +1285,7 @@ pub type Executive = frame_executive::Executive<
 pub type Migrations = (Unreleased, Permanent);
 
 /// Unreleased migrations. Add new ones here:
-pub type Unreleased = (); // TODO: clear old storage for ActiveBonusUpdateState (via kill or take)
+pub type Unreleased = ();
 
 /// Migrations/checks that do not need to be versioned and can run on every upgrade.
 pub type Permanent = (pallet_xcm::migration::MigrateToLatestXcmVersion<Runtime>,);