diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-21 12:40:10 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-21 12:40:10 +0900 |
| commit | 2b2ca914cc5f2f7fde3b6e75faabbe4913fd7264 (patch) | |
| tree | ff8bbb00238526c376f521647adc9ea9f486b84c /crates/shirabe/tests/repository/vcs/git_driver_test.rs | |
| parent | 4b92ecafd7634ad99aa432d58fbc1958d1f01270 (diff) | |
| download | php-shirabe-2b2ca914cc5f2f7fde3b6e75faabbe4913fd7264.tar.gz php-shirabe-2b2ca914cc5f2f7fde3b6e75faabbe4913fd7264.tar.zst php-shirabe-2b2ca914cc5f2f7fde3b6e75faabbe4913fd7264.zip | |
test(tests): port setUp/tearDown as set_up/tear_down with TearDown
Port PHP setUp/tearDown across the ported integration tests using
same-named set_up()/tear_down() functions and a TearDown struct whose
Drop runs tear_down(). Fixture-init setUp returns its fixtures;
tmpdir-style setUp/tearDown carry state in TearDown fields. Parts that
depend on unported infrastructure (PHPUnit mocks, Config::merge, the PHP
error handler) stay todo!() and are only wired into ignored stubs to
avoid breaking live tests.
Also fix shirabe-php-shim putenv to handle the no-'=' form (PHP unsets
the variable), which Platform::clear_env relies on for the env-clearing
tearDowns.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/tests/repository/vcs/git_driver_test.rs')
| -rw-r--r-- | crates/shirabe/tests/repository/vcs/git_driver_test.rs | 65 |
1 files changed, 65 insertions, 0 deletions
diff --git a/crates/shirabe/tests/repository/vcs/git_driver_test.rs b/crates/shirabe/tests/repository/vcs/git_driver_test.rs index 85c9f0b..c36470c 100644 --- a/crates/shirabe/tests/repository/vcs/git_driver_test.rs +++ b/crates/shirabe/tests/repository/vcs/git_driver_test.rs @@ -4,11 +4,76 @@ // that reaches curl_multi_init, todo!()) to feed git command output; mocking is not // available here. +use indexmap::IndexMap; +use shirabe::config::Config; +use shirabe::util::filesystem::Filesystem; +use shirabe::util::platform::Platform; +use shirabe_php_shim::PhpMixed; +use tempfile::TempDir; + +struct SetUp { + home: TempDir, + config: Config, + network_env: Option<String>, +} + +fn set_up() -> SetUp { + let home = TempDir::new().unwrap(); + let mut config = Config::new(true, None); + let mut top: IndexMap<String, PhpMixed> = IndexMap::new(); + let mut config_section: IndexMap<String, PhpMixed> = IndexMap::new(); + config_section.insert( + "home".to_string(), + PhpMixed::String(home.path().to_string_lossy().into_owned()), + ); + top.insert("config".to_string(), PhpMixed::Array(config_section)); + config.merge(&top, Config::SOURCE_UNKNOWN); + let network_env = Platform::get_env("COMPOSER_DISABLE_NETWORK"); + + SetUp { + home, + config, + network_env, + } +} + +fn tear_down(home: &std::path::Path, network_env: &Option<String>) { + let mut fs = Filesystem::new(None); + fs.remove_directory(home).unwrap(); + match network_env { + None => Platform::clear_env("COMPOSER_DISABLE_NETWORK"), + Some(network_env) => Platform::put_env("COMPOSER_DISABLE_NETWORK", network_env), + } +} + +struct TearDown { + home: std::path::PathBuf, + network_env: Option<String>, +} + +impl TearDown { + fn new(home: std::path::PathBuf, network_env: Option<String>) -> Self { + TearDown { home, network_env } + } +} + +impl Drop for TearDown { + fn drop(&mut self) { + tear_down(&self.home, &self.network_env); + } +} + macro_rules! git_stub { ($name:ident) => { #[test] #[ignore = "constructs a GitDriver and mocks a ProcessExecutor/HttpDownloader (curl_multi_init todo!())"] fn $name() { + let SetUp { + home, + config: _, + network_env, + } = set_up(); + let _tear_down = TearDown::new(home.path().to_path_buf(), network_env); todo!() } }; |
