fix tests

Author: hanabi1224

src/utils/proofs_api/parameters.rs

@@ -90,8 +90,15 @@ pub(super) async fn check_parameter_file(path: &Path, info: &ParameterData) -> a
 // environment variable is set.
 pub(super) fn param_dir(data_dir: &Path) -> PathBuf {
     std::env::var(PathBuf::from(PROOFS_PARAMETER_CACHE_ENV))
-        .map(PathBuf::from)
-        .unwrap_or_else(|_| data_dir.join(PARAM_DIR))
+        .ok()
+        .and_then(|v| {
+            if v.is_empty() {
+                None
+            } else {
+                Some(PathBuf::from(v))
+            }
+        })
+        .unwrap_or_else(|| data_dir.join(PARAM_DIR))
 }
 
 /// Forest uses a set of external crates for verifying the proofs generated by

tests/common/mod.rs

@@ -41,7 +41,10 @@ impl CommonEnv for Command {
     // Always downloads proofs to same location to lower the overall test time
     // (by reducing multiple "fetching param file" steps).
     fn common_env(&mut self) -> &mut Self {
-        self.env("FIL_PROOFS_PARAMETER_CACHE", "/tmp/forest-test-fil-proofs")
+        match std::env::var("FIL_PROOFS_PARAMETER_CACHE").ok() {
+            Some(v) if !v.is_empty() => self,
+            _ => self.env("FIL_PROOFS_PARAMETER_CACHE", "/tmp/forest-test-fil-proofs"),
+        }
     }
 }
 

tests/config.rs

@@ -55,6 +55,7 @@ fn test_download_location_of_proof_parameter_files_default() {
 
     tool()
         .env("FOREST_CONFIG_PATH", config_file.path())
+        .env("FIL_PROOFS_PARAMETER_CACHE", "")
         .arg("fetch-params")
         .arg("--keys")
         .arg("--dry-run")