Fix statefulness in tests

xcm/xcm-executor/integration-tests/src/lib.rs

@@ -80,13 +80,13 @@ fn transact_recursion_limit_works() {
 		.build();
 
 	let mut msg = Xcm(vec![ClearOrigin]);
-	let mut max_weight = <XcmConfig as xcm_executor::Config>::Weigher::weight(&mut msg).unwrap();
+	let max_weight = <XcmConfig as xcm_executor::Config>::Weigher::weight(&mut msg).unwrap();
 	let mut call = polkadot_test_runtime::RuntimeCall::Xcm(
 		pallet_xcm::Call::execute { message: Box::new(VersionedXcm::from(msg)), max_weight },
 	);
 
 	for _ in 0..10 {
-		msg = Xcm(vec![
+		let mut msg = Xcm(vec![
 			WithdrawAsset((Parent, 1_000_000_000).into()),
 			BuyExecution { fees: (Parent, 1_000_000_000).into(), weight_limit: Unlimited },
 			Transact {
@@ -95,7 +95,7 @@ fn transact_recursion_limit_works() {
 				call: call.encode().into(),
 			},
 		]);
-		max_weight = <XcmConfig as xcm_executor::Config>::Weigher::weight(&mut msg).unwrap();
+		let max_weight = <XcmConfig as xcm_executor::Config>::Weigher::weight(&mut msg).unwrap();
 		call = polkadot_test_runtime::RuntimeCall::Xcm(
 			pallet_xcm::Call::execute { message: Box::new(VersionedXcm::from(msg)), max_weight }
 		);
@@ -119,6 +119,7 @@ fn transact_recursion_limit_works() {
 		.expect("imports the block");
 
 	client.state_at(block_hash).expect("state should exist").inspect_state(|| {
+		panic!("{:?}", polkadot_test_runtime::System::events());
 		assert!(polkadot_test_runtime::System::events().iter().any(|r| matches!(
 			r.event,
 			polkadot_test_runtime::RuntimeEvent::Xcm(pallet_xcm::Event::Attempted(