From 4b92ecafd7634ad99aa432d58fbc1958d1f01270 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 21 Jun 2026 05:17:53 +0900 Subject: test: port remaining repository and util tests as stubs All ignored: Composer/Filesystem/Platform/Vcs repositories and the AuthHelper/Bitbucket/Git/Perforce/RemoteFilesystem/Svn utils mock IO/Config/HttpDownloader(curl)/ProcessExecutor or use reflection; Filesystem/ProcessExecutor/StreamContextFactory/ProxyManager need on-disk fixtures or proxy env. setUp/tearDown not ported. Co-Authored-By: Claude Opus 4.8 (1M context) --- crates/shirabe/tests/config_test.rs | 35 +++++++++++++++++++++++++---------- 1 file changed, 25 insertions(+), 10 deletions(-) (limited to 'crates/shirabe/tests/config_test.rs') diff --git a/crates/shirabe/tests/config_test.rs b/crates/shirabe/tests/config_test.rs index 0b88c6b..f44b746 100644 --- a/crates/shirabe/tests/config_test.rs +++ b/crates/shirabe/tests/config_test.rs @@ -12,10 +12,7 @@ fn repo(r#type: &str, url: &str) -> PhpMixed { } fn map(pairs: Vec<(&str, PhpMixed)>) -> IndexMap { - pairs - .into_iter() - .map(|(k, v)| (k.to_string(), v)) - .collect() + pairs.into_iter().map(|(k, v)| (k.to_string(), v)).collect() } fn disable(name: &str) -> PhpMixed { @@ -73,7 +70,10 @@ fn data_add_packagist_repository() -> Vec { ("packagist.org", packagist()), ]), local: map(vec![]), - system: Some(map(vec![("example.com", repo("composer", "http://example.com"))])), + system: Some(map(vec![( + "example.com", + repo("composer", "http://example.com"), + )])), }, // local config can disable repos by name and re-add them anonymously to bring them above system config Case { @@ -85,7 +85,10 @@ fn data_add_packagist_repository() -> Vec { ("0", disable("packagist.org")), ("1", repo("composer", "http://packagist.org")), ]), - system: Some(map(vec![("example.com", repo("composer", "http://example.com"))])), + system: Some(map(vec![( + "example.com", + repo("composer", "http://example.com"), + )])), }, // local config can override by name to bring a repo above system config Case { @@ -93,8 +96,14 @@ fn data_add_packagist_repository() -> Vec { ("packagist.org", repo("composer", "http://packagistnew.org")), ("example.com", repo("composer", "http://example.com")), ]), - local: map(vec![("packagist.org", repo("composer", "http://packagistnew.org"))]), - system: Some(map(vec![("example.com", repo("composer", "http://example.com"))])), + local: map(vec![( + "packagist.org", + repo("composer", "http://packagistnew.org"), + )]), + system: Some(map(vec![( + "example.com", + repo("composer", "http://example.com"), + )])), }, // local config redefining packagist.org by URL override it if no named keys are used Case { @@ -104,8 +113,14 @@ fn data_add_packagist_repository() -> Vec { }, // local config redefining packagist.org by URL override it also with named keys Case { - expected: map(vec![("example", repo("composer", "https://repo.packagist.org"))]), - local: map(vec![("example", repo("composer", "https://repo.packagist.org"))]), + expected: map(vec![( + "example", + repo("composer", "https://repo.packagist.org"), + )]), + local: map(vec![( + "example", + repo("composer", "https://repo.packagist.org"), + )]), system: None, }, // incorrect local config does not cause ErrorException -- cgit v1.3.1