Remove old migration (#349)

* Remove old migration * Remove migration_add_schedule_to_task benchmarking
AvaProtocol · Apr 13, 2023 · 98f80bd · 98f80bd
1 parent 7d44984
commit 98f80bd
Show file tree

Hide file tree

Showing 11 changed files with 3 additions and 35 deletions.
diff --git a/pallets/automation-time/src/benchmarking.rs b/pallets/automation-time/src/benchmarking.rs
@@ -558,34 +558,6 @@ benchmarks! {
  schedule_notify_tasks::<T>(caller.clone(), vec![new_time_slot], T::MaxTasksPerSlot::get());
  }: { AutomationTime::<T>::shift_missed_tasks(last_time_slot, diff) }
 
- migration_add_schedule_to_task {
- let v in 1..100;
-
- use migrations::add_schedule_to_task::{AccountTasks as OldAccountTasks, AddScheduleToTask, OldAction, OldTask};
- use frame_support::traits::OnRuntimeUpgrade;
-
- for i in 0..v {
- let account_id: T::AccountId = account("Account", 0, i);
- let task_id = Pallet::<T>::generate_task_id(account_id.clone(), vec![0]);
- let task = OldTask::<T> {
- owner_id: account_id.clone(),
- provided_id: vec![1],
- execution_times: vec![10].try_into().unwrap(),
- executions_left: 1,
- action: OldAction::<T>::AutoCompoundDelegatedStake {
- delegator: account_id.clone(),
- collator: account_id.clone(),
- account_minimum: 100u32.into(),
- frequency: 11,
- },
- };
- OldAccountTasks::<T>::insert(account_id.clone(), task_id, task);
- }
- }: { AddScheduleToTask::<T>::on_runtime_upgrade() }
- verify {
- assert_eq!(v, crate::AccountTasks::<T>::iter().count() as u32);
- }
-
  impl_benchmark_test_suite!(
  AutomationTime,
  crate::mock::new_test_ext(crate::tests::START_BLOCK_TIME),

diff --git a/pallets/automation-time/src/migrations/mod.rs b/pallets/automation-time/src/migrations/mod.rs
@@ -1,5 +1,3 @@
 // Migrations
 
 // mod old;
-
-pub mod add_schedule_to_task;
diff --git a/...me/src/migrations/add_schedule_to_task.rs → ...rc/migrations/old/add_schedule_to_task.rs b/...me/src/migrations/add_schedule_to_task.rs → ...rc/migrations/old/add_schedule_to_task.rs
diff --git a/pallets/automation-time/src/migrations/old/mod.rs b/pallets/automation-time/src/migrations/old/mod.rs
@@ -2,3 +2,4 @@
 // Excluded from the module tree to avoid warnings and errors on dead code
 
 mod v4;
+mod add_schedule_to_task;
diff --git a/pallets/xcmp-handler/src/migrations/mod.rs b/pallets/xcmp-handler/src/migrations/mod.rs
@@ -1,5 +1,3 @@
 // Migrations
 
 // mod old;
-
-pub mod add_flow_to_currency_data;
diff --git a/...c/migrations/add_flow_to_currency_data.rs → ...grations/old/add_flow_to_currency_data.rs b/...c/migrations/add_flow_to_currency_data.rs → ...grations/old/add_flow_to_currency_data.rs
diff --git a/pallets/xcmp-handler/src/migrations/old/mod.rs b/pallets/xcmp-handler/src/migrations/old/mod.rs
@@ -1,2 +1,3 @@
 /// Migrations that have already been run
 pub mod add_mgx_tur_chain_currency_combo;
+pub mod add_flow_to_currency_data;
diff --git a/runtime/turing/src/lib.rs b/runtime/turing/src/lib.rs
@@ -137,7 +137,6 @@ pub type Executive = frame_executive::Executive<
  frame_system::ChainContext<Runtime>,
  Runtime,
  AllPalletsWithSystem,
- pallet_xcmp_handler::migrations::add_flow_to_currency_data::AddFlowToCurrencyData<Runtime>,
 >;
 
 /// Opaque types. These are used by the CLI to instantiate machinery that don't need to know

diff --git a/runtime/turing/src/migrations/mod.rs b/runtime/turing/src/migrations/mod.rs
@@ -1,5 +1,3 @@
 // Runtime Migrations
 
 // mod old;
-
-pub mod register_shiden_asset;
diff --git a/runtime/turing/src/migrations/old/mod.rs b/runtime/turing/src/migrations/old/mod.rs
@@ -2,3 +2,4 @@
 // Excluded from the module tree to avoid warnings and errors on dead code
 
 mod assets;
+mod register_shiden_asset;
diff --git a/...g/src/migrations/register_shiden_asset.rs → ...c/migrations/old/register_shiden_asset.rs b/...g/src/migrations/register_shiden_asset.rs → ...c/migrations/old/register_shiden_asset.rs