[FRAME] Test for sane genesis default

substrate/frame/babe/src/lib.rs

@@ -323,7 +323,7 @@ pub mod pallet {
 	#[pallet::genesis_config]
 	pub struct GenesisConfig<T: Config> {
 		pub authorities: Vec<(AuthorityId, BabeAuthorityWeight)>,
-		pub epoch_config: Option<BabeEpochConfiguration>,
+		pub epoch_config: BabeEpochConfiguration,
 		#[serde(skip)]
 		pub _config: sp_std::marker::PhantomData<T>,
 	}
@@ -333,9 +333,7 @@ pub mod pallet {
 		fn build(&self) {
 			SegmentIndex::<T>::put(0);
 			Pallet::<T>::initialize_genesis_authorities(&self.authorities);
-			EpochConfig::<T>::put(
-				self.epoch_config.clone().expect("epoch_config must not be None"),
-			);
+			EpochConfig::<T>::put(&self.epoch_config);
 		}
 	}
 

substrate/frame/support/procedural/src/construct_runtime/expand/config.rs

@@ -99,6 +99,17 @@ pub fn expand_outer_config(
 				<AllPalletsWithSystem as #scrate::traits::OnGenesis>::on_genesis();
 			}
 		}
+
+		/// Test the `Default` derive impl of the `RuntimeGenesisConfig`.
+		#[cfg(test)]
+		#[test]
+		fn test_genesis_config_builds() {
+			#scrate::__private::sp_io::TestExternalities::default().execute_with(|| {
+				<RuntimeGenesisConfig as #scrate::traits::BuildGenesisConfig>::build(
+					&RuntimeGenesisConfig::default()
+				);
+			});
+		}
 	}
 }
 

substrate/frame/support/src/lib.rs

@@ -780,7 +780,7 @@ macro_rules! assert_err_with_weight {
 			$crate::assert_err!($call.map(|_| ()).map_err(|e| e.error), $err);
 			assert_eq!(dispatch_err_with_post.post_info.actual_weight, $weight);
 		} else {
-			panic!("expected Err(_), got Ok(_).")
+			::core::panic!("expected Err(_), got Ok(_).")
 		}
 	};
 }

substrate/primitives/consensus/babe/src/lib.rs

@@ -256,6 +256,12 @@ pub struct BabeEpochConfiguration {
 	pub allowed_slots: AllowedSlots,
 }
 
+impl Default for BabeEpochConfiguration {
+	fn default() -> Self {
+		Self { c: (1, 4), allowed_slots: AllowedSlots::PrimaryAndSecondaryPlainSlots }
+	}
+}
+
 /// Verifies the equivocation proof by making sure that: both headers have
 /// different hashes, are targetting the same slot, and have valid signatures by
 /// the same authority.