fix: do not remove snapshots directory before running the test suite (foundry-rs#9645)

* do not remove snapshots directory before running the test suite, the side effect is that any custom group names or file name changes are not reflected - this is delegated to the end user

* do not remove the `snapshots` directory upon running `forge clean`

Author: zerosnacks

crates/config/src/lib.rs

@@ -1059,12 +1059,6 @@ impl Config {
         remove_test_dir(&self.fuzz.failure_persist_dir);
         remove_test_dir(&self.invariant.failure_persist_dir);
 
-        // Remove snapshot directory.
-        let snapshot_dir = project.root().join(&self.snapshots);
-        if snapshot_dir.exists() {
-            let _ = fs::remove_dir_all(&snapshot_dir);
-        }
-
         Ok(())
     }
 

crates/forge/bin/cmd/test/mod.rs

@@ -301,17 +301,6 @@ impl TestArgs {
         // Create test options from general project settings and compiler output.
         let project_root = &project.paths.root;
 
-        // Remove the snapshots directory if it exists.
-        // This is to ensure that we don't have any stale snapshots.
-        // If `FORGE_SNAPSHOT_CHECK` is set, we don't remove the snapshots directory as it is
-        // required for comparison.
-        if std::env::var_os("FORGE_SNAPSHOT_CHECK").is_none() {
-            let snapshot_dir = project_root.join(&config.snapshots);
-            if snapshot_dir.exists() {
-                let _ = fs::remove_dir_all(project_root.join(&config.snapshots));
-            }
-        }
-
         let should_debug = self.debug;
         let should_draw = self.flamegraph || self.flamechart;