From 752dd6a20a734da6f86f9cf0c59c080f778f8af1 Mon Sep 17 00:00:00 2001
From: Marcin S <marcin@realemail.net>
Date: Fri, 20 Jan 2023 12:16:19 +0100
Subject: [PATCH] Minor: Update output validity tests

Quick follow-up to https://github.com/paritytech/substrate/pull/13183.

Mainly, I wanted to double check that the `test_return_max_memory_offset` test doesn't pass just
because the output length is 0.

I also:

- Organized these tests into a module.
- Added a comment explaining why we don't use the `wasm_export_functions` macro.
---
 client/executor/runtime-test/src/lib.rs      | 62 ++++++++++++--------
 client/executor/src/integration_tests/mod.rs |  2 +-
 2 files changed, 37 insertions(+), 27 deletions(-)

diff --git a/client/executor/runtime-test/src/lib.rs b/client/executor/runtime-test/src/lib.rs
index 0aa30e4bc9675..25fe317f602ea 100644
--- a/client/executor/runtime-test/src/lib.rs
+++ b/client/executor/runtime-test/src/lib.rs
@@ -29,8 +29,6 @@ use sp_runtime::{
 	print,
 	traits::{BlakeTwo256, Hash},
 };
-#[cfg(not(feature = "std"))]
-use sp_runtime_interface::pack_ptr_and_len;
 
 extern "C" {
 	#[allow(dead_code)]
@@ -344,31 +342,43 @@ sp_core::wasm_export_functions! {
 	}
 }
 
-// Returns a huge len. It should result in an error, and not an allocation.
-#[no_mangle]
-#[cfg(not(feature = "std"))]
-pub extern "C" fn test_return_huge_len(_params: *const u8, _len: usize) -> u64 {
-	pack_ptr_and_len(0, u32::MAX)
-}
+// Tests that check output validity. We explicitly return the ptr and len, so we avoid using the
+// `wasm_export_functions` macro.
+mod output_validity {
+	#[cfg(not(feature = "std"))]
+	use super::WASM_PAGE_SIZE;
 
-// Returns an offset right at the edge of the wasm memory boundary. With length 0, it should
-// succeed.
-#[no_mangle]
-#[cfg(not(feature = "std"))]
-pub extern "C" fn test_return_max_memory_offset(_params: *const u8, _len: usize) -> u64 {
-	pack_ptr_and_len((core::arch::wasm32::memory_size(0) * WASM_PAGE_SIZE) as u32, 0)
-}
+	#[cfg(not(feature = "std"))]
+	use sp_runtime_interface::pack_ptr_and_len;
 
-// Returns an offset right at the edge of the wasm memory boundary. With length 1, it should fail.
-#[no_mangle]
-#[cfg(not(feature = "std"))]
-pub extern "C" fn test_return_max_memory_offset_plus_one(_params: *const u8, _len: usize) -> u64 {
-	pack_ptr_and_len((core::arch::wasm32::memory_size(0) * WASM_PAGE_SIZE) as u32, 1)
-}
+	// Returns a huge len. It should result in an error, and not an allocation.
+	#[no_mangle]
+	#[cfg(not(feature = "std"))]
+	pub extern "C" fn test_return_huge_len(_params: *const u8, _len: usize) -> u64 {
+		pack_ptr_and_len(0, u32::MAX)
+	}
 
-// Returns an output that overflows the u32 range. It should result in an error.
-#[no_mangle]
-#[cfg(not(feature = "std"))]
-pub extern "C" fn test_return_overflow(_params: *const u8, _len: usize) -> u64 {
-	pack_ptr_and_len(u32::MAX, 1)
+	// Returns an offset right before the edge of the wasm memory boundary. It should succeed.
+	#[no_mangle]
+	#[cfg(not(feature = "std"))]
+	pub extern "C" fn test_return_max_memory_offset(_params: *const u8, _len: usize) -> u64 {
+		pack_ptr_and_len((core::arch::wasm32::memory_size(0) * WASM_PAGE_SIZE) as u32 - 1, 1)
+	}
+
+	// Returns an offset right after the edge of the wasm memory boundary. It should fail.
+	#[no_mangle]
+	#[cfg(not(feature = "std"))]
+	pub extern "C" fn test_return_max_memory_offset_plus_one(
+		_params: *const u8,
+		_len: usize,
+	) -> u64 {
+		pack_ptr_and_len((core::arch::wasm32::memory_size(0) * WASM_PAGE_SIZE) as u32, 1)
+	}
+
+	// Returns an output that overflows the u32 range. It should result in an error.
+	#[no_mangle]
+	#[cfg(not(feature = "std"))]
+	pub extern "C" fn test_return_overflow(_params: *const u8, _len: usize) -> u64 {
+		pack_ptr_and_len(u32::MAX, 1)
+	}
 }
diff --git a/client/executor/src/integration_tests/mod.rs b/client/executor/src/integration_tests/mod.rs
index 22e651abe98e2..048c189b8bd74 100644
--- a/client/executor/src/integration_tests/mod.rs
+++ b/client/executor/src/integration_tests/mod.rs
@@ -811,7 +811,7 @@ fn return_max_memory_offset(wasm_method: WasmExecutionMethod) {
 
 	assert_eq!(
 		call_in_wasm("test_return_max_memory_offset", &[], wasm_method, &mut ext).unwrap(),
-		().encode()
+		vec![0]
 	);
 }