new tests

pallets/regions/src/benchmarking.rs

@@ -1,4 +1,4 @@
-//! Benchmarking setup for pallet-parachain-template
+//! Benchmarking setup for pallet-regions
 
 use super::*;
 

pallets/regions/src/lib.rs

@@ -66,6 +66,8 @@ pub mod pallet {
 			+ From<u32>;
 
 		/// Native currency implementation
+		//
+		// NOTE: Isn't used since we don't have a reward mechanism for ISMP relayers.
 		type NativeCurrency: Mutate<Self::AccountId>;
 
 		/// The Coretime chain from which we read region state.

pallets/regions/src/tests.rs

@@ -4,7 +4,7 @@ use crate::{
 use frame_support::{assert_err, assert_ok, pallet_prelude::*, traits::nonfungible::Mutate};
 use ismp::{
 	module::IsmpModule,
-	router::{GetResponse, Post, Request, Response, Timeout},
+	router::{GetResponse, Post, PostResponse, Request, Response, Timeout},
 };
 use pallet_broker::{CoreMask, RegionId, RegionRecord};
 use std::collections::BTreeMap;
@@ -86,6 +86,36 @@ fn on_response_works() {
 	});
 }
 
+#[test]
+fn on_response_only_handles_get() {
+	new_test_ext().execute_with(|| {
+		let module: IsmpModuleCallback<Test> = IsmpModuleCallback::default();
+
+		let mock_response = Response::Post(PostResponse {
+			post: Post {
+				source: <Test as crate::Config>::CoretimeChain::get(),
+				dest: <Test as crate::Config>::CoretimeChain::get(),
+				nonce: Default::default(),
+				from: Default::default(),
+				to: Default::default(),
+				timeout_timestamp: Default::default(),
+				data: Default::default(),
+			},
+			response: Default::default(),
+			timeout_timestamp: Default::default(),
+		});
+
+		assert_err!(module.on_response(mock_response), IsmpCustomError::NotSupported);
+	});
+}
+
+#[test]
+fn on_timeout_only_handles_get() {
+	new_test_ext().execute_with(|| {
+		// TODO
+	});
+}
+
 #[test]
 fn on_timeout_works() {
 	new_test_ext().execute_with(|| {