diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-25 16:44:29 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-26 00:20:05 +0900 |
| commit | f5f429dbae0a3e2d8224c0b1e4edcef54805d286 (patch) | |
| tree | 9f837baeeae6efa0ed926b181b8c273128d86c49 /crates/shirabe/tests/command | |
| parent | 3a0d9340810a8808d963135a884f50d08442ac67 (diff) | |
| download | php-shirabe-f5f429dbae0a3e2d8224c0b1e4edcef54805d286.tar.gz php-shirabe-f5f429dbae0a3e2d8224c0b1e4edcef54805d286.tar.zst php-shirabe-f5f429dbae0a3e2d8224c0b1e4edcef54805d286.zip | |
feat(http): reimplement CurlDownloader on reqwest; port 15 more tests
Replace the libcurl-shim CurlDownloader with a reqwest+tokio implementation per
the .ken sketch, resolving the construction panic that blocked command tests
(mock path via __new_mock is untouched). Port remote_filesystem (7), hg/svn
driver (4), zip_archiver/git_exclude_filter (4) tests. Fix hg/svn/git_exclude
regex-delimiter and svn result-propagation porting bugs.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/tests/command')
| -rw-r--r-- | crates/shirabe/tests/command/bump_command_test.rs | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/crates/shirabe/tests/command/bump_command_test.rs b/crates/shirabe/tests/command/bump_command_test.rs index 8866990..96c8baf 100644 --- a/crates/shirabe/tests/command/bump_command_test.rs +++ b/crates/shirabe/tests/command/bump_command_test.rs @@ -41,11 +41,10 @@ fn test_bump_fails_on_non_existing_composer_file() { #[test] #[serial] -#[ignore = "BumpCommand::initialize constructs a full Composer (Factory::create_http_downloader \ - -> CurlDownloader::new -> curl_multi_init), and the curl subsystem in shirabe-php-shim \ - is still todo!(). Only reachable once the HTTP/curl layer is ported. The companion \ - test_bump_fails_on_non_existing_composer_file covers the same error-output capture path \ - without reaching curl (the missing composer.json makes Composer construction a no-op)."] +#[ignore = "CurlDownloader::new no longer panics (HTTP layer is now reqwest-based), but this path \ + still reaches an unrelated todo!() in shirabe-php-shim stream_set_blocking (stream.rs), \ + which requires fcntl(2). The companion test_bump_fails_on_non_existing_composer_file \ + covers the same error-output capture path without reaching that shim."] fn test_bump_fails_on_write_error_to_composer_file() { if shirabe_php_shim::function_exists("posix_getuid") && shirabe_php_shim::posix_getuid() == 0 { // ref: $this->markTestSkipped('Cannot run as root'); |
