From 8117e5450693d19726da877bce8aacf5c7aa53af Mon Sep 17 00:00:00 2001 From: nsfisis Date: Thu, 25 Jun 2026 15:12:06 +0900 Subject: test: port 44 vcs/downloader/version tests using mock infra Port git, version_guesser, gitlab_driver, github_driver, and git_downloader tests using the ProcessExecutor/HttpDownloader mocks and IO/Config stubs. Fix production regex-porting bugs surfaced by the now-reachable paths: Url::sanitize and Response::find_header_value had non-delimited PCRE patterns; implement array_search_mixed non-strict branch and a datetime format mapping. Add HttpDownloader::__new_mock so mocked downloaders skip curl construction. Co-Authored-By: Claude Opus 4.8 (1M context) --- crates/shirabe/src/util/http_downloader.rs | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) (limited to 'crates/shirabe/src/util/http_downloader.rs') diff --git a/crates/shirabe/src/util/http_downloader.rs b/crates/shirabe/src/util/http_downloader.rs index edc84cc..782f04a 100644 --- a/crates/shirabe/src/util/http_downloader.rs +++ b/crates/shirabe/src/util/http_downloader.rs @@ -883,6 +883,30 @@ impl HttpDownloader { && function_exists("curl_multi_init") } + /// For testing only. Builds an HttpDownloader whose request methods are fully + /// short-circuited by the mock (see [`HttpDownloader::__expects`]), without + /// constructing the real curl/rfs backends. Mirrors HttpDownloaderMock, which + /// extends HttpDownloader but never performs curl I/O. + pub fn __new_mock( + io: std::rc::Rc>, + config: std::rc::Rc>, + ) -> Self { + Self { + io, + config, + jobs: IndexMap::new(), + options: IndexMap::new(), + running_jobs: 0, + max_jobs: 12, + curl: None, + rfs: None, + id_gen: 0, + disabled: false, + allow_async: false, + mock: None, + } + } + /// For testing only. Mirrors HttpDownloaderMock::expects: installs the expectation queue, /// strict flag and default handler used by the mock request path. pub fn __expects( -- cgit v1.3.1