From 903d37a254f931e79ecded26f30a3b517c6d27da Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 21 Jun 2026 03:16:34 +0900 Subject: test(repository): port InstalledRepositoryTest testAddRepository maps the expected LogicException to #[should_panic] since InstalledRepository::add_repository guards with assert!. testFindPackagesWithReplacersAndProviders is #[ignore] because that assert omits InstalledRepositoryInterface, so adding an InstalledArrayRepository panics before the lookup runs. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../shirabe/tests/util/http/request_proxy_test.rs | 53 ++++++++++++++++++---- 1 file changed, 43 insertions(+), 10 deletions(-) (limited to 'crates/shirabe/tests/util/http/request_proxy_test.rs') diff --git a/crates/shirabe/tests/util/http/request_proxy_test.rs b/crates/shirabe/tests/util/http/request_proxy_test.rs index 3674396..502ec31 100644 --- a/crates/shirabe/tests/util/http/request_proxy_test.rs +++ b/crates/shirabe/tests/util/http/request_proxy_test.rs @@ -64,7 +64,11 @@ fn test_get_status() { let cases: Vec<(Option<&str>, Option<&str>, &str)> = vec![ (None, Some(format), ""), (Some("http://proxy.com:80"), None, "http://proxy.com:80"), - (Some("http://proxy.com:80"), Some(format), "proxy (http://proxy.com:80)"), + ( + Some("http://proxy.com:80"), + Some(format), + "proxy (http://proxy.com:80)", + ), ]; for (url, format, expected) in cases { @@ -92,7 +96,10 @@ fn test_get_curl_options() { Some("http://proxy.com:80"), None, curl_options(&[ - (CURLOPT_PROXY, PhpMixed::String("http://proxy.com:80".to_string())), + ( + CURLOPT_PROXY, + PhpMixed::String("http://proxy.com:80".to_string()), + ), (CURLOPT_NOPROXY, PhpMixed::String(String::new())), ]), ), @@ -100,10 +107,16 @@ fn test_get_curl_options() { Some("http://proxy.com:80"), Some("user:p%40ss"), curl_options(&[ - (CURLOPT_PROXY, PhpMixed::String("http://proxy.com:80".to_string())), + ( + CURLOPT_PROXY, + PhpMixed::String("http://proxy.com:80".to_string()), + ), (CURLOPT_NOPROXY, PhpMixed::String(String::new())), (CURLOPT_PROXYAUTH, PhpMixed::Int(CURLAUTH_BASIC)), - (CURLOPT_PROXYUSERPWD, PhpMixed::String("user:p%40ss".to_string())), + ( + CURLOPT_PROXYUSERPWD, + PhpMixed::String("user:p%40ss".to_string()), + ), ]), ), ]; @@ -118,20 +131,34 @@ fn test_get_curl_options() { #[ignore = "secure proxies require curl_version, which is a todo!() in the php-shim"] fn test_get_curl_options_with_ssl() { let mut cafile_opts: IndexMap = IndexMap::new(); - cafile_opts.insert("cafile".to_string(), PhpMixed::String("/certs/bundle.pem".to_string())); + cafile_opts.insert( + "cafile".to_string(), + PhpMixed::String("/certs/bundle.pem".to_string()), + ); let mut capath_opts: IndexMap = IndexMap::new(); capath_opts.insert("capath".to_string(), PhpMixed::String("/certs".to_string())); - let cases: Vec<(&str, Option<&str>, IndexMap, IndexMap)> = vec![ + let cases: Vec<( + &str, + Option<&str>, + IndexMap, + IndexMap, + )> = vec![ ( "https://proxy.com:443", None, cafile_opts, curl_options(&[ - (CURLOPT_PROXY, PhpMixed::String("https://proxy.com:443".to_string())), + ( + CURLOPT_PROXY, + PhpMixed::String("https://proxy.com:443".to_string()), + ), (CURLOPT_NOPROXY, PhpMixed::String(String::new())), - (CURLOPT_PROXY_CAINFO, PhpMixed::String("/certs/bundle.pem".to_string())), + ( + CURLOPT_PROXY_CAINFO, + PhpMixed::String("/certs/bundle.pem".to_string()), + ), ]), ), ( @@ -139,10 +166,16 @@ fn test_get_curl_options_with_ssl() { Some("user:p%40ss"), capath_opts, curl_options(&[ - (CURLOPT_PROXY, PhpMixed::String("https://proxy.com:443".to_string())), + ( + CURLOPT_PROXY, + PhpMixed::String("https://proxy.com:443".to_string()), + ), (CURLOPT_NOPROXY, PhpMixed::String(String::new())), (CURLOPT_PROXYAUTH, PhpMixed::Int(CURLAUTH_BASIC)), - (CURLOPT_PROXYUSERPWD, PhpMixed::String("user:p%40ss".to_string())), + ( + CURLOPT_PROXYUSERPWD, + PhpMixed::String("user:p%40ss".to_string()), + ), (CURLOPT_PROXY_CAPATH, PhpMixed::String("/certs".to_string())), ]), ), -- cgit v1.3.1