Disable compression snapshot tests on windows and use persist instead of rename in config save

Author: Heliozoa

crates/tmc-langs-cli/tests/integration.rs

@@ -138,6 +138,7 @@ fn clean() {
 }
 
 #[test]
+#[cfg(not(target_os = "windows"))] // hash differs on windows
 fn compress_project_tar() {
     test(|exercise| {
         let target = NamedTempFile::new().unwrap();
@@ -164,6 +165,7 @@ fn compress_project_tar() {
 }
 
 #[test]
+#[cfg(not(target_os = "windows"))] // hash differs on windows
 fn compress_project_zip() {
     test(|exercise| {
         let target = NamedTempFile::new().unwrap();
@@ -191,6 +193,7 @@ fn compress_project_zip() {
 }
 
 #[test]
+#[cfg(not(target_os = "windows"))] // hash differs on windows
 fn compress_project_zstd() {
     test(|exercise| {
         let target = NamedTempFile::new().unwrap();

crates/tmc-langs-util/src/file_util.rs

@@ -17,7 +17,7 @@ use std::{
     ops::{Deref, DerefMut},
     path::{Path, PathBuf},
 };
-use tempfile::NamedTempFile;
+use tempfile::{NamedTempFile, TempDir};
 use walkdir::WalkDir;
 
 /// Convenience macro for locking a path.

crates/tmc-langs/src/config/tmc_config.rs

@@ -155,7 +155,7 @@ impl TmcConfig {
         file_util::write_to_file(toml, temp_file.path())?;
 
         log::debug!("Moving new config over old one");
-        file_util::rename(temp_file.path(), path)?;
+        temp_file.persist(path)?;
         Ok(())
     }
 

crates/tmc-langs/src/error.rs

@@ -124,6 +124,8 @@ pub enum LangsError {
     Hmac(#[from] hmac::digest::InvalidLength),
     #[error(transparent)]
     TestMyCodeClient(#[from] tmc_testmycode_client::TestMyCodeClientError),
+    #[error(transparent)]
+    PersistTempFile(#[from] tempfile::PersistError),
 }
 
 /// Error validating TMC params values.