From 52d1670b90fe2c58a66ff73cc0dc90f3452ff04b Mon Sep 17 00:00:00 2001 From: onur-ozkan Date: Wed, 11 Oct 2023 19:08:26 +0300 Subject: [PATCH] add unit test for `Config::verify` Signed-off-by: onur-ozkan --- src/bootstrap/config/tests.rs | 23 ++++++++++++++++++++++- src/bootstrap/download.rs | 2 +- 2 files changed, 23 insertions(+), 2 deletions(-) diff --git a/src/bootstrap/config/tests.rs b/src/bootstrap/config/tests.rs index d091f33eee44d..ae8363b6de9c4 100644 --- a/src/bootstrap/config/tests.rs +++ b/src/bootstrap/config/tests.rs @@ -3,7 +3,12 @@ use crate::config::TomlConfig; use super::{Config, Flags}; use clap::CommandFactory; use serde::Deserialize; -use std::{env, path::Path}; +use std::{ + env, + fs::{remove_file, File}, + io::Write, + path::Path, +}; fn parse(config: &str) -> Config { Config::parse_inner(&["check".to_owned(), "--config=/does/not/exist".to_owned()], |&_| { @@ -196,3 +201,19 @@ fn rust_optimize() { fn invalid_rust_optimize() { parse("rust.optimize = \"a\""); } + +#[test] +fn verify_file_integrity() { + let config = parse(""); + + let tempfile = config.tempdir().join(".tmp-test-file"); + File::create(&tempfile).unwrap().write_all(b"dummy value").unwrap(); + assert!(tempfile.exists()); + + assert!( + config + .verify(&tempfile, "7e255dd9542648a8779268a0f268b891a198e9828e860ed23f826440e786eae5") + ); + + remove_file(tempfile).unwrap(); +} diff --git a/src/bootstrap/download.rs b/src/bootstrap/download.rs index 2653793da17d4..756fb6900c989 100644 --- a/src/bootstrap/download.rs +++ b/src/bootstrap/download.rs @@ -320,7 +320,7 @@ impl Config { } /// Returns whether the SHA256 checksum of `path` matches `expected`. - fn verify(&self, path: &Path, expected: &str) -> bool { + pub(crate) fn verify(&self, path: &Path, expected: &str) -> bool { use sha2::Digest; self.verbose(&format!("verifying {}", path.display()));