handle unsafe set_var call

adriangb · adriangb · commit ad58e54e9665 · 2025-09-23T15:54:41.000-07:00
diff --git a/src/test_utils/insta.rs b/src/test_utils/insta.rs
@@ -12,7 +12,8 @@ macro_rules! assert_snapshot {
 }
 
 pub fn settings() -> insta::Settings {
-    env::set_var("INSTA_WORKSPACE_ROOT", env!("CARGO_MANIFEST_DIR"));
+    // Safety: this is only used in tests, it may panic if used in parallel with other tests.
+    unsafe { env::set_var("INSTA_WORKSPACE_ROOT", env!("CARGO_MANIFEST_DIR")) };
     let mut settings = insta::Settings::clone_current();
     let cwd = env::current_dir().unwrap();
     let cwd = cwd.to_str().unwrap();

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,8 @@ macro_rules! assert_snapshot {`
`12`	`12`	`}`
`13`	`13`
`14`	`14`	`pub fn settings() -> insta::Settings {`
`15`		`- env::set_var("INSTA_WORKSPACE_ROOT", env!("CARGO_MANIFEST_DIR"));`
	`15`	`+ // Safety: this is only used in tests, it may panic if used in parallel with other tests.`
	`16`	`+ unsafe { env::set_var("INSTA_WORKSPACE_ROOT", env!("CARGO_MANIFEST_DIR")) };`
`16`	`17`	`let mut settings = insta::Settings::clone_current();`
`17`	`18`	`let cwd = env::current_dir().unwrap();`
`18`	`19`	`let cwd = cwd.to_str().unwrap();`