Repository: paritytech/substrate
File: utils/frame/try-runtime/cli/tests/create_snapshot.rs (Rust)

Code before:
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.

// You should have received a copy of the GNU General Public License
// along with this program. If not, see <https://www.gnu.org/licenses/>.

#![cfg(unix)]
#[cfg(feature = "try-runtime")]
mod tests {
	use assert_cmd::cargo::cargo_bin;
	use regex::Regex;
	use std::{
		path::{Path, PathBuf},
		process::{self},
		time::Duration,
	};
	use substrate_cli_test_utils as common;
	use tempfile::tempdir;
	use tokio::process::{Child, Command};

	#[tokio::test]
	async fn create_snapshot_works() {
		// Build substrate so binaries used in the test use the latest code.
		common::build_substrate(&["--features=try-runtime"]);

		let temp_dir = tempdir().expect("Failed to create a tempdir");
		let snap_file_path = temp_dir.path().join("snapshot.snap");

		common::run_with_timeout(Duration::from_secs(60), async move {
			fn create_snapshot(ws_url: &str, snap_file: &PathBuf) -> Child {
				Command::new(cargo_bin("substrate"))
					.stdout(process::Stdio::piped())
					.stderr(process::Stdio::piped())
					.args(&["try-runtime", "--runtime=existing"])
					.args(&["create-snapshot", format!("--uri={}", ws_url).as_str()])
					.arg(snap_file)
					.kill_on_drop(true)
					.spawn()
					.unwrap()
			}

			// Start a node and wait for it to begin finalizing blocks
			let mut node = common::KillChildOnDrop(common::start_node());
			let ws_url = common::extract_info_from_output(node.stderr.take().unwrap()).0.ws_url;
			common::wait_n_finalized_blocks(1, &ws_url).await;

			// Try to create a snapshot.
			let mut snapshot_creation = create_snapshot(&ws_url, &snap_file_path);
			let re = Regex::new(r#".*writing snapshot of (\d+) bytes to .*"#).unwrap();
			let matched =
				common::wait_for_stream_pattern_match(snapshot_creation.stderr.take().unwrap(), re)

Review comment (bug):
`tempdir` supports passing a prefix, can we give it one like "try-runtime-cli-test-dir"? Just in case something goes wrong and it gets left hanging around it's clear what it is.

Code after:
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.

// You should have received a copy of the GNU General Public License
// along with this program. If not, see <https://www.gnu.org/licenses/>.

#![cfg(unix)]
#![cfg(feature = "try-runtime")]

use assert_cmd::cargo::cargo_bin;
use node_primitives::Hash;
use regex::Regex;
use remote_externalities::{Builder, Mode, OfflineConfig, SnapshotConfig};
use sp_runtime::testing::{Block as RawBlock, ExtrinsicWrapper};
use std::{
	path::{Path, PathBuf},
	process,
	time::Duration,
};
use substrate_cli_test_utils as common;
use tokio::process::{Child, Command};

type Block = RawBlock<ExtrinsicWrapper<Hash>>;

#[tokio::test]
async fn create_snapshot_works() {
	// Build substrate so binaries used in the test use the latest code.
	common::build_substrate(&["--features=try-runtime"]);

	let temp_dir = tempfile::Builder::new()
		.prefix("try-runtime-cli-test-dir")
		.tempdir()
		.expect("Failed to create a tempdir");
	let snap_file_path = temp_dir.path().join("snapshot.snap");

	common::run_with_timeout(Duration::from_secs(60), async move {
		fn create_snapshot(ws_url: &str, snap_file: &PathBuf, at: Hash) -> Child {
			Command::new(cargo_bin("substrate"))
				.stdout(process::Stdio::piped())
				.stderr(process::Stdio::piped())
				.args(&["try-runtime", "--runtime=existing"])
				.args(&["create-snapshot", format!("--uri={}", ws_url).as_str()])
				.arg(snap_file)
				.args(&["--at", format!("{:?}", at).as_str()])
				.kill_on_drop(true)
				.spawn()
				.unwrap()
		}

		// Start a node and wait for it to begin finalizing blocks
		let mut node = common::KillChildOnDrop(common::start_node());