[Fix] Config : don't clobber existing failure-persistence config duri…

…ng contextualization
proptest-rs · Feb 18, 2024 · 33b1c25 · 33b1c25
1 parent b712538
commit 33b1c25
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/proptest/CHANGELOG.md b/proptest/CHANGELOG.md
@@ -5,6 +5,9 @@
 - Setting `PROPTEST_MAX_DEFAULT_SIZE_RANGE` now customizes the default `SizeRange`
   used by the default strategies for collections (like `Vec`). The default remains 100.
 
+### Bug Fixes
+- Fixed issue where config contextualization would clobber existing failure persistence config
+
 ## 1.4.0
 
 ### Breaking Changes

diff --git a/proptest/src/test_runner/config.rs b/proptest/src/test_runner/config.rs
@@ -22,8 +22,6 @@ use std::str::FromStr;
 use crate::test_runner::result_cache::{noop_result_cache, ResultCache};
 use crate::test_runner::rng::RngAlgorithm;
 use crate::test_runner::FailurePersistence;
-#[cfg(feature = "std")]
-use crate::test_runner::FileFailurePersistence;
 
 #[cfg(feature = "std")]
 const CASES: &str = "PROPTEST_CASES";
@@ -80,8 +78,6 @@ pub fn contextualize_config(mut result: Config) -> Config {
         }
     }
 
-    result.failure_persistence =
-        Some(Box::new(FileFailurePersistence::default()));
     for (var, value) in
         env::vars_os().filter_map(|(k, v)| k.into_string().ok().map(|k| (k, v)))
     {