From 2b2ca914cc5f2f7fde3b6e75faabbe4913fd7264 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 21 Jun 2026 12:40:10 +0900 Subject: 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 --- crates/shirabe/tests/util/filesystem_test.rs | 48 ++++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) (limited to 'crates/shirabe/tests/util/filesystem_test.rs') diff --git a/crates/shirabe/tests/util/filesystem_test.rs b/crates/shirabe/tests/util/filesystem_test.rs index 4ed9c14..e101c79 100644 --- a/crates/shirabe/tests/util/filesystem_test.rs +++ b/crates/shirabe/tests/util/filesystem_test.rs @@ -3,6 +3,54 @@ // These exercise Filesystem path helpers and on-disk operations (sizes, copy, symlinks and // junctions over a temp tree). The filesystem fixtures and platform-specific symlink/junction // behaviour are not ported. +use shirabe::util::filesystem::Filesystem; +use shirabe_php_shim::{dirname, is_dir, is_file}; + +#[allow(dead_code)] +struct SetUp { + fs: Filesystem, + working_dir: String, + test_file: String, +} + +#[allow(dead_code)] +fn set_up() -> SetUp { + let fs = Filesystem::new(None); + // getUniqueTmpDirectory is base TestCase infrastructure that is not ported. + let working_dir: String = todo!(); + #[allow(unreachable_code)] + let unique_tmp: String = todo!(); + #[allow(unreachable_code)] + let test_file: String = format!("{unique_tmp}/composer_test_file"); + #[allow(unreachable_code)] + SetUp { + fs, + working_dir, + test_file, + } +} + +#[allow(dead_code)] +fn tear_down(set_up: &mut SetUp) { + if is_dir(&set_up.working_dir) { + let _ = set_up.fs.remove_directory(&set_up.working_dir); + } + if is_file(&set_up.test_file) { + let _ = set_up.fs.remove_directory(dirname(&set_up.test_file)); + } +} + +#[allow(dead_code)] +struct TearDown { + set_up: SetUp, +} + +impl Drop for TearDown { + fn drop(&mut self) { + tear_down(&mut self.set_up); + } +} + macro_rules! stub { ($name:ident) => { #[test] -- cgit v1.3.1