From 432472808051cb4f1bb9517b858dbc810aaa5a63 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sat, 25 Jul 2026 16:16:33 +0900 Subject: refactor: replace redundant clones with moves Co-Authored-By: Claude Opus 5 --- .../dependency_resolver/default_policy_test.rs | 5 +- .../tests/dependency_resolver/pool_builder_test.rs | 2 +- .../security_advisory_pool_filter_test.rs | 4 +- .../tests/dependency_resolver/solver_test.rs | 64 +++++++++++----------- .../tests/dependency_resolver/transaction_test.rs | 38 ++++++------- 5 files changed, 55 insertions(+), 58 deletions(-) (limited to 'crates/shirabe/tests/dependency_resolver') diff --git a/crates/shirabe/tests/dependency_resolver/default_policy_test.rs b/crates/shirabe/tests/dependency_resolver/default_policy_test.rs index 2717954c..44724b33 100644 --- a/crates/shirabe/tests/dependency_resolver/default_policy_test.rs +++ b/crates/shirabe/tests/dependency_resolver/default_policy_test.rs @@ -505,10 +505,7 @@ fn test_select_local_repos_first() { package_a_alias_important.set_root_package_alias(true); fixtures.repo.add_package(package_a).unwrap(); - fixtures - .repo - .add_package(package_a_alias.clone().into()) - .unwrap(); + fixtures.repo.add_package(package_a_alias.into()).unwrap(); repo_important.add_package(package_a_important).unwrap(); repo_important .add_package(package_a_alias_important.clone().into()) diff --git a/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs b/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs index 9b82908d..40ae115d 100644 --- a/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs +++ b/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs @@ -539,7 +539,7 @@ fn run_test_pool_builder( } } - let mut request = Request::new(Some(locked_repo.clone())); + let mut request = Request::new(Some(locked_repo)); for (package, constraint) in request_map["require"].as_array().unwrap() { request .require_name( diff --git a/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs b/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs index 15eace79..37403b19 100644 --- a/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs +++ b/crates/shirabe/tests/dependency_resolver/security_advisory_pool_filter_test.rs @@ -75,7 +75,7 @@ fn test_filter_packages_by_advisories() { let pool = Pool::new( vec![ - package.clone(), + package, expected_package1.clone(), expected_package2.clone(), ], @@ -323,7 +323,7 @@ fn test_dont_filter_packages_with_abandoned_package() { let pool = Pool::new( vec![ expected_package.clone(), - abandoned_package.clone(), + abandoned_package, ignore_abandoned_package.clone(), ], vec![], diff --git a/crates/shirabe/tests/dependency_resolver/solver_test.rs b/crates/shirabe/tests/dependency_resolver/solver_test.rs index be8df4db..4a449c7f 100644 --- a/crates/shirabe/tests/dependency_resolver/solver_test.rs +++ b/crates/shirabe/tests/dependency_resolver/solver_test.rs @@ -329,7 +329,7 @@ fn test_solver_install_same_package_from_different_repositories() { let foo1 = get_package("foo", "1"); let foo2 = get_package("foo", "1"); repo1.add_package(foo1.clone()).unwrap(); - repo2.add_package(foo2.clone()).unwrap(); + repo2.add_package(foo2).unwrap(); repo_set .add_repository(RepositoryInterfaceHandle::new(repo1)) @@ -362,7 +362,7 @@ fn test_solver_install_with_deps() { let package_b = get_package("B", "1.0"); fixtures.repo.add_package(package_b.clone()).unwrap(); let new_package_b = get_package("B", "1.1"); - fixtures.repo.add_package(new_package_b.clone()).unwrap(); + fixtures.repo.add_package(new_package_b).unwrap(); package_a .as_complete_package() @@ -404,13 +404,13 @@ fn test_solver_install_honours_not_equal_operator() { let package_a = get_package("A", "1.0"); fixtures.repo.add_package(package_a.clone()).unwrap(); let package_b = get_package("B", "1.0"); - fixtures.repo.add_package(package_b.clone()).unwrap(); + fixtures.repo.add_package(package_b).unwrap(); let new_package_b11 = get_package("B", "1.1"); fixtures.repo.add_package(new_package_b11.clone()).unwrap(); let new_package_b12 = get_package("B", "1.2"); - fixtures.repo.add_package(new_package_b12.clone()).unwrap(); + fixtures.repo.add_package(new_package_b12).unwrap(); let new_package_b13 = get_package("B", "1.3"); - fixtures.repo.add_package(new_package_b13.clone()).unwrap(); + fixtures.repo.add_package(new_package_b13).unwrap(); package_a .as_complete_package() @@ -585,7 +585,7 @@ fn test_solver_multi_package_name_version_resolution_depends_on_require_order() vec![ ExpectedJob::Single { job: "install", - package: php80.clone(), + package: php80, }, ExpectedJob::Single { job: "install", @@ -595,7 +595,7 @@ fn test_solver_multi_package_name_version_resolution_depends_on_require_order() ); // now we flip the requirements around: we request "ext-foobar" before "php" - let mut request = Request::new(Some(fixtures.repo_locked.clone())); + let mut request = Request::new(Some(fixtures.repo_locked)); request.require_name("ourcustom/ext-foobar", None).unwrap(); request.require_name("ourcustom/PHP", None).unwrap(); @@ -620,7 +620,7 @@ fn test_solver_multi_package_name_version_resolution_is_independent_of_require_o { let fixtures = set_up(); let php74 = get_package("ourcustom/PHP", "7.4"); - fixtures.repo.add_package(php74.clone()).unwrap(); + fixtures.repo.add_package(php74).unwrap(); let php80 = get_package("ourcustom/PHP", "8.0"); fixtures.repo.add_package(php80.clone()).unwrap(); // note we are inserting this one into the repo first, unlike in the previous test @@ -689,7 +689,7 @@ fn test_solver_multi_package_name_version_resolution_is_independent_of_require_o ); // unlike in the previous test, the order of requirements no longer matters now - let mut request = Request::new(Some(fixtures.repo_locked.clone())); + let mut request = Request::new(Some(fixtures.repo_locked)); request.require_name("ourcustom/ext-foobar", None).unwrap(); request.require_name("ourcustom/PHP", None).unwrap(); @@ -716,7 +716,7 @@ fn test_solver_fix_locked() { fixtures.repo_locked.add_package(package_a.clone()).unwrap(); let mut request = fixtures.request; - request.fix_package(package_a.clone()); + request.fix_package(package_a); check_solver_result( fixtures.repo_set, @@ -735,7 +735,7 @@ fn test_solver_fix_locked_with_alternative() { fixtures.repo_locked.add_package(package_a.clone()).unwrap(); let mut request = fixtures.request; - request.fix_package(package_a.clone()); + request.fix_package(package_a); check_solver_result( fixtures.repo_set, @@ -770,7 +770,7 @@ fn test_solver_update_does_only_update() { )])); let mut request = fixtures.request; - request.fix_package(package_a.clone()); + request.fix_package(package_a); request .require_name("B", Some(get_version_constraint("=", "1.1.0.0"))) .unwrap(); @@ -899,11 +899,11 @@ fn test_solver_update_only_updates_selected_package() { let package_a_newer = get_package("A", "1.1"); fixtures.repo.add_package(package_a_newer.clone()).unwrap(); let package_b_newer = get_package("B", "1.1"); - fixtures.repo.add_package(package_b_newer.clone()).unwrap(); + fixtures.repo.add_package(package_b_newer).unwrap(); let mut request = fixtures.request; request.require_name("A", None).unwrap(); - request.fix_package(package_b.clone()); + request.fix_package(package_b); check_solver_result( fixtures.repo_set, @@ -1019,7 +1019,7 @@ fn test_solver_all_jobs() { let package_b = get_package("B", "1.0"); fixtures.repo.add_package(package_b.clone()).unwrap(); let new_package_b = get_package("B", "1.1"); - fixtures.repo.add_package(new_package_b.clone()).unwrap(); + fixtures.repo.add_package(new_package_b).unwrap(); let package_c = get_package("C", "1.1"); fixtures.repo.add_package(package_c.clone()).unwrap(); fixtures.repo.add_package(get_package("D", "1.0")).unwrap(); @@ -1074,9 +1074,9 @@ fn test_solver_three_alternative_require_and_conflict() { let middle_package_b = get_package("B", "1.0"); fixtures.repo.add_package(middle_package_b.clone()).unwrap(); let new_package_b = get_package("B", "1.1"); - fixtures.repo.add_package(new_package_b.clone()).unwrap(); + fixtures.repo.add_package(new_package_b).unwrap(); let old_package_b = get_package("B", "0.9"); - fixtures.repo.add_package(old_package_b.clone()).unwrap(); + fixtures.repo.add_package(old_package_b).unwrap(); package_a .as_complete_package() .unwrap() @@ -1170,7 +1170,7 @@ fn test_install_one_of_two_alternatives() { let package_a = get_package("A", "1.0"); fixtures.repo.add_package(package_a.clone()).unwrap(); let package_b = get_package("A", "1.0"); - fixtures.repo.add_package(package_b.clone()).unwrap(); + fixtures.repo.add_package(package_b).unwrap(); let mut request = fixtures.request; request.require_name("A", None).unwrap(); @@ -1337,7 +1337,7 @@ fn test_skip_replaced_package_if_replacer_is_selected() { let package_q = get_package("Q", "1.0"); fixtures.repo.add_package(package_q.clone()).unwrap(); let package_b = get_package("B", "1.0"); - fixtures.repo.add_package(package_b.clone()).unwrap(); + fixtures.repo.add_package(package_b).unwrap(); package_a .as_complete_package() .unwrap() @@ -1509,7 +1509,7 @@ fn test_install_circular_require() { let package_a = get_package("A", "1.0"); fixtures.repo.add_package(package_a.clone()).unwrap(); let package_b1 = get_package("B", "0.9"); - fixtures.repo.add_package(package_b1.clone()).unwrap(); + fixtures.repo.add_package(package_b1).unwrap(); let package_b2 = get_package("B", "1.1"); fixtures.repo.add_package(package_b2.clone()).unwrap(); package_a @@ -1675,7 +1675,7 @@ fn test_use_replacer_if_necessary() { let package_a = get_package("A", "1.0"); fixtures.repo.add_package(package_a.clone()).unwrap(); let package_b = get_package("B", "1.0"); - fixtures.repo.add_package(package_b.clone()).unwrap(); + fixtures.repo.add_package(package_b).unwrap(); let package_d = get_package("D", "1.0"); fixtures.repo.add_package(package_d.clone()).unwrap(); let package_d2 = get_package("D", "1.1"); @@ -1779,11 +1779,11 @@ fn test_use_replacer_if_necessary() { fn test_issue265() { let fixtures = set_up(); let package_a1 = get_package("A", "2.0.999999-dev"); - fixtures.repo.add_package(package_a1.clone()).unwrap(); + fixtures.repo.add_package(package_a1).unwrap(); let package_a2 = get_package("A", "2.1-dev"); - fixtures.repo.add_package(package_a2.clone()).unwrap(); + fixtures.repo.add_package(package_a2).unwrap(); let package_a3 = get_package("A", "2.2-dev"); - fixtures.repo.add_package(package_a3.clone()).unwrap(); + fixtures.repo.add_package(package_a3).unwrap(); let package_b1 = get_package("B", "2.0.10"); fixtures.repo.add_package(package_b1.clone()).unwrap(); let package_b2 = get_package("B", "2.0.9"); @@ -1898,7 +1898,7 @@ fn test_conflict_result_empty() { let package_a = get_package("A", "1.0"); fixtures.repo.add_package(package_a.clone()).unwrap(); let package_b = get_package("B", "1.0"); - fixtures.repo.add_package(package_b.clone()).unwrap(); + fixtures.repo.add_package(package_b).unwrap(); package_a .as_complete_package() .unwrap() @@ -1955,7 +1955,7 @@ fn test_unsatisfiable_requires() { let package_a = get_package("A", "1.0"); fixtures.repo.add_package(package_a.clone()).unwrap(); let package_b = get_package("B", "1.0"); - fixtures.repo.add_package(package_b.clone()).unwrap(); + fixtures.repo.add_package(package_b).unwrap(); package_a .as_complete_package() @@ -2093,11 +2093,11 @@ fn test_require_mismatch_exception() { fn test_learn_literals_with_sorted_rule_literals() { let fixtures = set_up(); let package_twig2 = get_package("twig/twig", "2.0"); - fixtures.repo.add_package(package_twig2.clone()).unwrap(); + fixtures.repo.add_package(package_twig2).unwrap(); let package_twig16 = get_package("twig/twig", "1.6"); fixtures.repo.add_package(package_twig16.clone()).unwrap(); let package_twig15 = get_package("twig/twig", "1.5"); - fixtures.repo.add_package(package_twig15.clone()).unwrap(); + fixtures.repo.add_package(package_twig15).unwrap(); let package_symfony = get_package("symfony/symfony", "2.0"); fixtures.repo.add_package(package_symfony.clone()).unwrap(); let package_twig_bridge = get_package("symfony/twig-bridge", "2.0"); @@ -2158,7 +2158,7 @@ fn test_learn_literals_with_sorted_rule_literals() { fn test_install_recursive_alias_dependencies() { let fixtures = set_up(); let package_a = get_package("A", "1.0"); - fixtures.repo.add_package(package_a.clone()).unwrap(); + fixtures.repo.add_package(package_a).unwrap(); let package_b = get_package("B", "2.0"); fixtures.repo.add_package(package_b.clone()).unwrap(); let package_a2 = get_package("A", "2.0"); @@ -2363,13 +2363,13 @@ fn test_learn_positive_literal() { let package_f1 = get_package("F", "1.0"); fixtures.repo.add_package(package_f1.clone()).unwrap(); let package_f2 = get_package("F", "2.0"); - fixtures.repo.add_package(package_f2.clone()).unwrap(); + fixtures.repo.add_package(package_f2).unwrap(); let package_g1 = get_package("G", "1.0"); - fixtures.repo.add_package(package_g1.clone()).unwrap(); + fixtures.repo.add_package(package_g1).unwrap(); let package_g2 = get_package("G", "2.0"); fixtures.repo.add_package(package_g2.clone()).unwrap(); let package_g3 = get_package("G", "3.0"); - fixtures.repo.add_package(package_g3.clone()).unwrap(); + fixtures.repo.add_package(package_g3).unwrap(); package_a .as_complete_package() diff --git a/crates/shirabe/tests/dependency_resolver/transaction_test.rs b/crates/shirabe/tests/dependency_resolver/transaction_test.rs index 3b1006ea..b6c28cc3 100644 --- a/crates/shirabe/tests/dependency_resolver/transaction_test.rs +++ b/crates/shirabe/tests/dependency_resolver/transaction_test.rs @@ -110,8 +110,8 @@ fn test_transaction_generation_and_sorting() { let dl_modifying_plugin2_dep = get_package("x/downloads-modifying2-dep", "1.0.0"); let dl_modifying_plugin2 = get_package("x/downloads-modifying2", "1.0.0"); let result_packages = vec![ - package_a.clone(), - package_a_alias.clone(), + package_a, + package_a_alias, package_b_new.clone(), package_d.clone(), package_f.clone(), @@ -216,67 +216,67 @@ fn test_transaction_generation_and_sorting() { let expected_operations = vec![ OperationEntry::Job { job: "uninstall".to_string(), - package: package_c.clone(), + package: package_c, }, OperationEntry::Job { job: "uninstall".to_string(), - package: package_e.clone(), + package: package_e, }, OperationEntry::Job { job: "markAliasUninstalled".to_string(), - package: package_e_alias.clone(), + package: package_e_alias, }, OperationEntry::Job { job: "install".to_string(), - package: dl_modifying_plugin.clone(), + package: dl_modifying_plugin, }, OperationEntry::Job { job: "install".to_string(), - package: dl_modifying_plugin2_dep.clone(), + package: dl_modifying_plugin2_dep, }, OperationEntry::Job { job: "install".to_string(), - package: dl_modifying_plugin2.clone(), + package: dl_modifying_plugin2, }, OperationEntry::Job { job: "install".to_string(), - package: plugin.clone(), + package: plugin, }, OperationEntry::Job { job: "install".to_string(), - package: plugin2_dep.clone(), + package: plugin2_dep, }, OperationEntry::Job { job: "install".to_string(), - package: plugin2.clone(), + package: plugin2, }, OperationEntry::Job { job: "install".to_string(), - package: package_a0_first.clone(), + package: package_a0_first, }, OperationEntry::Update { - from: package_b.clone(), - to: package_b_new.clone(), + from: package_b, + to: package_b_new, }, OperationEntry::Job { job: "install".to_string(), - package: package_g.clone(), + package: package_g, }, OperationEntry::Job { job: "install".to_string(), - package: package_f.clone(), + package: package_f, }, OperationEntry::Job { job: "markAliasInstalled".to_string(), - package: package_f_alias2.clone(), + package: package_f_alias2, }, OperationEntry::Job { job: "markAliasInstalled".to_string(), - package: package_f_alias1.clone(), + package: package_f_alias1, }, OperationEntry::Job { job: "install".to_string(), - package: package_d.clone(), + package: package_d, }, ]; -- cgit v1.3.1