fix: only increment nonce by 1 on force_change_authorities (#722)

* fix: updates for light client read

* fix: only increment nonce by 1 on force_change_authorities

pallets/dkg-metadata/src/lib.rs

@@ -1375,7 +1375,10 @@ pub mod pallet {
 			// to sign our own key as a means of jumpstarting the mechanism.
 			if let Some(pub_key) = next_pub_key {
 				ShouldSubmitProposerVote::<T>::put(true);
-				let next_nonce = Self::refresh_nonce() + 1u32;
+				// we use the current nonce as the next nonce since the refreshNonce
+				// has been incremented in `change_authorities` incrementing here will lead to a
+				// difference of two
+				let next_nonce = Self::refresh_nonce();
 				let data = Self::create_refresh_proposal(pub_key.1.into(), next_nonce)?;
 				match T::ProposalHandler::handle_unsigned_proposal(data) {
 					Ok(()) => {

pallets/dkg-metadata/src/tests.rs

@@ -18,7 +18,7 @@ use crate::{
 	mock::*, AggregatedMisbehaviourReports, AggregatedPublicKeys, Authorities,
 	AuthorityReputations, BestAuthorities, Config, Error, Event, JailedKeygenAuthorities,
 	NextAuthorities, NextBestAuthorities, NextKeygenThreshold, NextSignatureThreshold,
-	INITIAL_REPUTATION, REPUTATION_INCREMENT,
+	RefreshNonce, INITIAL_REPUTATION, REPUTATION_INCREMENT,
 };
 use codec::Encode;
 use dkg_runtime_primitives::{keccak_256, utils::ecdsa, MisbehaviourType, KEY_TYPE};
@@ -875,3 +875,37 @@ fn reputation_is_set_correctly() {
 		}
 	})
 }
+
+#[test]
+fn force_change_authorities_increments_nonce_correctly() {
+	new_test_ext(vec![1, 2, 3, 4, 5]).execute_with(|| {
+		let session_id = 1;
+
+		// prep the next authorities
+		let mut next_authorities: BoundedVec<_, _> = Default::default();
+		let mut next_authorities_raw: Vec<_> = Default::default();
+		for _ in 1..=5 {
+			let authority_id = mock_pub_key();
+			let dkg_id = DKGId::from(authority_id);
+			next_authorities_raw.push(authority_id);
+			next_authorities.try_push(dkg_id).unwrap();
+		}
+
+		let keygen_threshold = 5;
+		let signature_threshold = 3;
+
+		RefreshNonce::<Test>::put(1);
+		let input: BoundedVec<_, _> = mock_pub_key().to_raw_vec().try_into().unwrap();
+		crate::pallet::NextDKGPublicKey::<Test>::put((session_id, input));
+		NextKeygenThreshold::<Test>::put(keygen_threshold);
+		NextSignatureThreshold::<Test>::put(signature_threshold);
+		NextAuthorities::<Test>::put(&next_authorities);
+
+		// force change authorities
+		assert_ok!(DKGMetadata::force_change_authorities(RuntimeOrigin::root(),));
+
+		// the new refresh nonce should only increment by one
+		let expected_refresh_nonce: u32 = 2;
+		assert_eq!(RefreshNonce::<Test>::get(), expected_refresh_nonce);
+	});
+}

standalone/runtime/src/lib.rs

@@ -300,7 +300,7 @@ parameter_types! {
 #[cfg(not(feature = "integration-tests"))]
 parameter_types! {
   // How often we trigger a new session.
-  pub const Period: BlockNumber = HOURS;
+  pub const Period: BlockNumber = MINUTES;
   pub const Offset: BlockNumber = 0;
 }