test: correct extract test implementations

Author: brianheineman

postgresql_archive/tests/archive.rs

@@ -2,7 +2,7 @@ use postgresql_archive::configuration::theseus;
 use postgresql_archive::extract;
 use postgresql_archive::{get_archive, get_version};
 use semver::VersionReq;
-use std::fs::{create_dir_all, remove_dir_all};
+use std::fs::remove_dir_all;
 use test_log::test;
 
 #[test(tokio::test)]
@@ -32,7 +32,6 @@ async fn test_get_archive_and_extract() -> anyhow::Result<()> {
     assert!(version_req.matches(&archive_version));
 
     let out_dir = tempfile::tempdir()?.path().to_path_buf();
-    create_dir_all(&out_dir)?;
     extract(url, &archive, &out_dir).await?;
     remove_dir_all(&out_dir)?;
     Ok(())

postgresql_archive/tests/blocking.rs

@@ -5,7 +5,7 @@ use postgresql_archive::configuration::theseus;
 #[cfg(feature = "blocking")]
 use postgresql_archive::VersionReq;
 #[cfg(feature = "blocking")]
-use std::fs::{create_dir_all, remove_dir_all};
+use std::fs::remove_dir_all;
 #[cfg(feature = "blocking")]
 use test_log::test;
 
@@ -30,7 +30,6 @@ fn test_get_archive_and_extract() -> anyhow::Result<()> {
     assert!(version_req.matches(&archive_version));
 
     let out_dir = tempfile::tempdir()?.path().to_path_buf();
-    create_dir_all(&out_dir)?;
     extract(url, &archive, &out_dir)?;
     remove_dir_all(&out_dir)?;
     Ok(())

postgresql_archive/tests/zonky.rs

@@ -2,7 +2,7 @@ use postgresql_archive::configuration::zonky;
 use postgresql_archive::extract;
 use postgresql_archive::{get_archive, get_version};
 use semver::VersionReq;
-use std::fs::{create_dir_all, remove_dir_all};
+use std::fs::remove_dir_all;
 use test_log::test;
 
 #[test(tokio::test)]
@@ -32,7 +32,6 @@ async fn test_get_archive_and_extract() -> anyhow::Result<()> {
     assert!(version_req.matches(&archive_version));
 
     let out_dir = tempfile::tempdir()?.path().to_path_buf();
-    create_dir_all(&out_dir)?;
     extract(url, &archive, &out_dir).await?;
     remove_dir_all(&out_dir)?;
     Ok(())