Use temp directory for BuildConfig in sui-core/src/test_utils.rs (Mys…

…tenLabs#7893)

crates/sui-core/src/test_utils.rs

@@ -148,7 +148,7 @@ pub fn compile_basics_package() -> CompiledPackage {
     let mut path = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
     path.push("../../sui_programmability/examples/basics");
 
-    let build_config = BuildConfig::default();
+    let build_config = BuildConfig::new_for_testing();
     sui_framework::build_move_package(&path, build_config).unwrap()
 }
 

crates/sui-framework-build/Cargo.toml

@@ -11,6 +11,7 @@ publish = false
 anyhow = { version = "1.0.64", features = ["backtrace"] }
 fastcrypto = { workspace = true }
 once_cell = "1.16"
+tempfile = "3.3.0"
 
 serde-reflection = "0.3.6"
 sui-types = { path = "../sui-types" }

crates/sui-framework-build/src/compiled_package.rs

@@ -48,6 +48,12 @@ pub struct BuildConfig {
 }
 
 impl BuildConfig {
+    pub fn new_for_testing() -> Self {
+        let mut build_config: Self = Default::default();
+        build_config.config.install_dir = Some(tempfile::TempDir::new().unwrap().into_path());
+        build_config
+    }
+
     /// Given a `path` and a `build_config`, build the package in that path, including its dependencies.
     /// If we are building the Sui framework, we skip the check that the addresses should be 0
     pub fn build(self, path: PathBuf) -> SuiResult<CompiledPackage> {

crates/test-utils/src/messages.rs

@@ -284,8 +284,7 @@ pub fn create_publish_move_package_transaction(
     keypair: &AccountKeyPair,
     gas_price: Option<u64>,
 ) -> VerifiedTransaction {
-    let mut build_config = BuildConfig::default();
-    build_config.config.install_dir = Some(tempfile::TempDir::new().unwrap().into_path());
+    let build_config = BuildConfig::new_for_testing();
     let all_module_bytes = sui_framework::build_move_package(&path, build_config)
         .unwrap()
         .get_package_bytes(/* with_unpublished_deps */ false);