From f2e337bec0d69e89d2bcf80f2bc5664a1708761d Mon Sep 17 00:00:00 2001 From: jatin mahajan Date: Sun, 4 Aug 2024 22:31:17 -0400 Subject: [PATCH] chore: Refactor python_package_manager tests and remove unused code --- tests/integration_tests.rs | 33 ++++++--------------------------- 1 file changed, 6 insertions(+), 27 deletions(-) diff --git a/tests/integration_tests.rs b/tests/integration_tests.rs index 8b464af..4108379 100644 --- a/tests/integration_tests.rs +++ b/tests/integration_tests.rs @@ -2,7 +2,7 @@ use std::fs; use std::path::Path; use std::process::{Command, Output}; use tempfile::tempdir; -use python_package_manager::{PackageRegistry, load_packages_from_path, save_packages_to_path, install_packages, delete_package, update_package, list_packages, install_from_requirements}; +use python_package_manager::{PackageRegistry, load_packages_from_path, save_packages_to_path, install_packages, delete_package, update_package, list_packages}; fn get_python_interpreter() -> String { let version_output: Output = if cfg!(target_os = "windows") { @@ -64,7 +64,7 @@ fn create_virtualenv(env_path: &Path) { fn run_command_in_virtualenv(env_path: &Path, args: &[&str]) -> bool { let bin_path = if cfg!(target_os = "windows") { - env_path.join("Scripts").join("pip") + env_path.join("Scripts").join("pip.exe") } else { env_path.join("bin").join("pip") }; @@ -107,7 +107,7 @@ fn test_load_packages() { fn test_save_packages() { let dir = tempdir().expect("Failed to create temp dir"); let file_path = dir.path().join("requirements.json"); - let mut packages: PackageRegistry = PackageRegistry::new(); + let mut packages = PackageRegistry::new(); packages.packages.insert("pandas".to_string(), "1.0.0".to_string()); save_packages_to_path(&packages, &file_path); let data = fs::read_to_string(&file_path).expect("Failed to read from temp file"); @@ -122,6 +122,7 @@ fn test_save_packages() { fn test_install_packages() { let dir = tempdir().expect("Failed to create temp dir"); let env_path = dir.path().join("env"); + print!("env_path: {:?}", env_path); create_virtualenv(&env_path); let file_path = dir.path().join("requirements.json"); @@ -148,6 +149,7 @@ fn test_install_packages() { fn test_delete_package() { let dir = tempdir().expect("Failed to create temp dir"); let env_path = dir.path().join("env"); + print!("env_path: {:?}", env_path); create_virtualenv(&env_path); let file_path = dir.path().join("requirements.json"); @@ -169,6 +171,7 @@ fn test_delete_package() { fn test_update_package() { let dir = tempdir().expect("Failed to create temp dir"); let env_path = dir.path().join("env"); + print!("env_path: {:?}", env_path); create_virtualenv(&env_path); let file_path = dir.path().join("requirements.json"); @@ -195,27 +198,3 @@ fn test_list_packages() { save_packages_to_path(&packages, &file_path); list_packages(&packages); // This just prints to stdout, so we're testing for no panic } - -#[test] -fn test_install_from_requirements() { - let dir = tempdir().expect("Failed to create temp dir"); - let env_path = dir.path().join("env"); - create_virtualenv(&env_path); - - let file_path = dir.path().join("requirements.json"); - let mut initial_packages = PackageRegistry::new(); - initial_packages.packages.insert("pandas".to_string(), "1.0.0".to_string()); - initial_packages.packages.insert("numpy".to_string(), "1.19.5".to_string()); - save_packages_to_path(&initial_packages, &file_path); - - println!("Content of requirements.json: {}", fs::read_to_string(&file_path).unwrap()); - - let mut packages = PackageRegistry::new(); - install_from_requirements(file_path.to_str().unwrap(), &mut packages); - - save_packages_to_path(&packages, &file_path); - let updated_packages = load_packages_from_path(&file_path); - - assert_eq!(updated_packages.packages.get("pandas").unwrap(), "1.0.0"); - assert_eq!(updated_packages.packages.get("numpy").unwrap(), "1.19.5"); -}