aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/tests/dependency_resolver
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-07-11 16:33:05 +0900
committernsfisis <nsfisis@gmail.com>2026-07-11 16:33:05 +0900
commit27d00055df8691a6bd99aaf38633a7338b16cc6a (patch)
tree4af17ccff5f8c2d09156fa62c559e60e3e683dac /crates/shirabe/tests/dependency_resolver
parent1ec2220def43e37e5a65b96dde93c19b493258f4 (diff)
downloadphp-shirabe-27d00055df8691a6bd99aaf38633a7338b16cc6a.tar.gz
php-shirabe-27d00055df8691a6bd99aaf38633a7338b16cc6a.tar.zst
php-shirabe-27d00055df8691a6bd99aaf38633a7338b16cc6a.zip
chore: use fully-qualified name for Rc/RefCell
Diffstat (limited to 'crates/shirabe/tests/dependency_resolver')
-rw-r--r--crates/shirabe/tests/dependency_resolver/pool_builder_test.rs10
-rw-r--r--crates/shirabe/tests/dependency_resolver/pool_optimizer_test.rs3
-rw-r--r--crates/shirabe/tests/dependency_resolver/rule_set_iterator_test.rs20
-rw-r--r--crates/shirabe/tests/dependency_resolver/rule_set_test.rs28
-rw-r--r--crates/shirabe/tests/dependency_resolver/solver_test.rs38
5 files changed, 51 insertions, 48 deletions
diff --git a/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs b/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs
index 548cb0dc..e6897ff7 100644
--- a/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/pool_builder_test.rs
@@ -22,9 +22,7 @@ use shirabe::repository::repository_set::{RepositorySet, RootAliasInput};
use shirabe_external_packages::composer::pcre::preg::Preg;
use shirabe_php_shim::PREG_SPLIT_DELIM_CAPTURE;
use shirabe_php_shim::PhpMixed;
-use std::cell::RefCell;
use std::path::PathBuf;
-use std::rc::Rc;
/// Maps the PHP `$loadPackage` closure: pops the optional `id` from the data, loads the
/// package and records it in `package_ids` keyed by that id (erroring on duplicates).
@@ -451,8 +449,9 @@ fn run_test_pool_builder(
IndexMap::new(),
IndexMap::new(),
);
- let config = Rc::new(RefCell::new(Config::new(false, None)));
- let io: Rc<RefCell<dyn IOInterface>> = Rc::new(RefCell::new(NullIO::new()));
+ let config = std::rc::Rc::new(std::cell::RefCell::new(Config::new(false, None)));
+ let io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>> =
+ std::rc::Rc::new(std::cell::RefCell::new(NullIO::new()));
let mut rm = RepositoryFactory::manager(io.clone(), &config, None, None, None).unwrap();
// PHP: foreach ($packageRepos as $packages)
@@ -605,7 +604,8 @@ fn run_test_pool_builder(
message
);
- let mut optimizer = PoolOptimizer::new(Rc::new(DefaultPolicy::new(false, false, None)));
+ let mut optimizer =
+ PoolOptimizer::new(std::rc::Rc::new(DefaultPolicy::new(false, false, None)));
let optimized = optimizer.optimize(&request, &pool);
let mut result = get_package_result_set(&optimized, &package_ids);
let mut expect_optimized = expect_optimized.as_list().unwrap().clone();
diff --git a/crates/shirabe/tests/dependency_resolver/pool_optimizer_test.rs b/crates/shirabe/tests/dependency_resolver/pool_optimizer_test.rs
index 3c4952e7..08d5f989 100644
--- a/crates/shirabe/tests/dependency_resolver/pool_optimizer_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/pool_optimizer_test.rs
@@ -15,7 +15,6 @@ use shirabe_external_packages::composer::pcre::preg::Preg;
use shirabe_php_shim::PREG_SPLIT_DELIM_CAPTURE;
use shirabe_php_shim::PhpMixed;
use std::path::PathBuf;
-use std::rc::Rc;
fn load_package(package_data: &PhpMixed) -> BasePackageHandle {
let loader = ArrayLoader::new(None, false);
@@ -237,7 +236,7 @@ fn run_test_pool_optimizer(
IndexMap::new(),
IndexMap::new(),
);
- let mut pool_optimizer = PoolOptimizer::new(Rc::new(DefaultPolicy::new(
+ let mut pool_optimizer = PoolOptimizer::new(std::rc::Rc::new(DefaultPolicy::new(
prefer_stable,
prefer_lowest,
None,
diff --git a/crates/shirabe/tests/dependency_resolver/rule_set_iterator_test.rs b/crates/shirabe/tests/dependency_resolver/rule_set_iterator_test.rs
index 9c9f85bd..93e25962 100644
--- a/crates/shirabe/tests/dependency_resolver/rule_set_iterator_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/rule_set_iterator_test.rs
@@ -7,13 +7,11 @@ use shirabe::dependency_resolver::rule::{RULE_LEARNED, RULE_ROOT_REQUIRE, Reason
use shirabe::dependency_resolver::rule_set::RuleSet;
use shirabe::dependency_resolver::rule_set_iterator::RuleSetIterator;
use shirabe_semver::constraint::MatchAllConstraint;
-use std::cell::RefCell;
-use std::rc::Rc;
-type Rules = IndexMap<i64, Vec<Rc<RefCell<Rule>>>>;
+type Rules = IndexMap<i64, Vec<std::rc::Rc<std::cell::RefCell<Rule>>>>;
-fn root_require_rule() -> Rc<RefCell<Rule>> {
- Rc::new(RefCell::new(Rule::Generic(GenericRule::new(
+fn root_require_rule() -> std::rc::Rc<std::cell::RefCell<Rule>> {
+ std::rc::Rc::new(std::cell::RefCell::new(Rule::Generic(GenericRule::new(
vec![],
RULE_ROOT_REQUIRE,
ReasonData::RootRequire {
@@ -40,11 +38,9 @@ fn set_up() -> (Pool, Rules) {
);
rules.insert(
RuleSet::TYPE_LEARNED,
- vec![Rc::new(RefCell::new(Rule::Generic(GenericRule::new(
- vec![],
- RULE_LEARNED,
- ReasonData::Int(1),
- ))))],
+ vec![std::rc::Rc::new(std::cell::RefCell::new(Rule::Generic(
+ GenericRule::new(vec![], RULE_LEARNED, ReasonData::Int(1)),
+ )))],
);
rules.insert(RuleSet::TYPE_PACKAGE, vec![]);
@@ -56,7 +52,7 @@ fn test_foreach() {
let (_pool, rules) = set_up();
let mut rule_set_iterator = RuleSetIterator::new(rules.clone());
- let mut result: Vec<Rc<RefCell<Rule>>> = Vec::new();
+ let mut result: Vec<std::rc::Rc<std::cell::RefCell<Rule>>> = Vec::new();
while rule_set_iterator.valid() {
result.push(rule_set_iterator.current());
rule_set_iterator.next();
@@ -70,7 +66,7 @@ fn test_foreach() {
assert_eq!(expected.len(), result.len());
for (e, r) in expected.iter().zip(result.iter()) {
- assert!(Rc::ptr_eq(e, r));
+ assert!(std::rc::Rc::ptr_eq(e, r));
}
}
diff --git a/crates/shirabe/tests/dependency_resolver/rule_set_test.rs b/crates/shirabe/tests/dependency_resolver/rule_set_test.rs
index 3ed974e6..0a9582dc 100644
--- a/crates/shirabe/tests/dependency_resolver/rule_set_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/rule_set_test.rs
@@ -7,8 +7,6 @@ use shirabe::dependency_resolver::{
};
use shirabe::repository::RepositorySet;
use shirabe_semver::constraint::{MatchAllConstraint, MatchNoneConstraint};
-use std::cell::RefCell;
-use std::rc::Rc;
fn root_require_reason() -> ReasonData {
ReasonData::RootRequire {
@@ -17,16 +15,16 @@ fn root_require_reason() -> ReasonData {
}
}
-fn rule(literals: Vec<i64>) -> Rc<RefCell<Rule>> {
- Rc::new(RefCell::new(Rule::Generic(GenericRule::new(
+fn rule(literals: Vec<i64>) -> std::rc::Rc<std::cell::RefCell<Rule>> {
+ std::rc::Rc::new(std::cell::RefCell::new(Rule::Generic(GenericRule::new(
literals,
RULE_ROOT_REQUIRE,
root_require_reason(),
))))
}
-fn learned_rule() -> Rc<RefCell<Rule>> {
- Rc::new(RefCell::new(Rule::Generic(GenericRule::new(
+fn learned_rule() -> std::rc::Rc<std::cell::RefCell<Rule>> {
+ std::rc::Rc::new(std::cell::RefCell::new(Rule::Generic(GenericRule::new(
vec![],
RULE_LEARNED,
ReasonData::Int(1),
@@ -67,11 +65,11 @@ fn test_add() {
);
let request = &rules[&RuleSet::TYPE_REQUEST];
assert_eq!(2, request.len());
- assert!(Rc::ptr_eq(&request[0], &request0));
- assert!(Rc::ptr_eq(&request[1], &request1));
+ assert!(std::rc::Rc::ptr_eq(&request[0], &request0));
+ assert!(std::rc::Rc::ptr_eq(&request[1], &request1));
let learned = &rules[&RuleSet::TYPE_LEARNED];
assert_eq!(1, learned.len());
- assert!(Rc::ptr_eq(&learned[0], &learned0));
+ assert!(std::rc::Rc::ptr_eq(&learned[0], &learned0));
}
#[test]
@@ -110,7 +108,7 @@ fn test_rule_by_id() {
let rule = rule(vec![]);
rule_set.add(rule.clone(), RuleSet::TYPE_REQUEST).unwrap();
- assert!(Rc::ptr_eq(&rule_set.rule_by_id[&0], &rule));
+ assert!(std::rc::Rc::ptr_eq(&rule_set.rule_by_id[&0], &rule));
}
#[test]
@@ -122,9 +120,9 @@ fn test_get_iterator() {
rule_set.add(rule2.clone(), RuleSet::TYPE_LEARNED).unwrap();
let mut iterator = rule_set.get_iterator();
- assert!(Rc::ptr_eq(&iterator.current(), &rule1));
+ assert!(std::rc::Rc::ptr_eq(&iterator.current(), &rule1));
iterator.next();
- assert!(Rc::ptr_eq(&iterator.current(), &rule2));
+ assert!(std::rc::Rc::ptr_eq(&iterator.current(), &rule2));
}
#[test]
@@ -136,7 +134,7 @@ fn test_get_iterator_for() {
rule_set.add(rule2.clone(), RuleSet::TYPE_LEARNED).unwrap();
let iterator = rule_set.get_iterator_for(vec![RuleSet::TYPE_LEARNED]);
- assert!(Rc::ptr_eq(&iterator.current(), &rule2));
+ assert!(std::rc::Rc::ptr_eq(&iterator.current(), &rule2));
}
#[test]
@@ -148,7 +146,7 @@ fn test_get_iterator_without() {
rule_set.add(rule2.clone(), RuleSet::TYPE_LEARNED).unwrap();
let iterator = rule_set.get_iterator_without(vec![RuleSet::TYPE_REQUEST]);
- assert!(Rc::ptr_eq(&iterator.current(), &rule2));
+ assert!(std::rc::Rc::ptr_eq(&iterator.current(), &rule2));
}
// The constraint is MatchNoneConstraint, so what_provides returns no packages and the
@@ -178,7 +176,7 @@ fn test_pretty_string() {
let mut rule_set = RuleSet::new();
let literal = p.get_id();
- let rule = Rc::new(RefCell::new(Rule::Generic(GenericRule::new(
+ let rule = std::rc::Rc::new(std::cell::RefCell::new(Rule::Generic(GenericRule::new(
vec![literal],
RULE_ROOT_REQUIRE,
ReasonData::RootRequire {
diff --git a/crates/shirabe/tests/dependency_resolver/solver_test.rs b/crates/shirabe/tests/dependency_resolver/solver_test.rs
index c20fd4c5..b8be2287 100644
--- a/crates/shirabe/tests/dependency_resolver/solver_test.rs
+++ b/crates/shirabe/tests/dependency_resolver/solver_test.rs
@@ -16,8 +16,6 @@ use shirabe::repository::handle::{LockArrayRepositoryHandle, RepositoryInterface
use shirabe::repository::lock_array_repository::LockArrayRepository;
use shirabe::repository::repository_set::RepositorySet;
use shirabe_semver::constraint::{AnyConstraint, MatchAllConstraint, MultiConstraint};
-use std::cell::RefCell;
-use std::rc::Rc;
#[allow(dead_code)]
struct Fixtures {
@@ -122,13 +120,18 @@ fn check_solver_result_repo_set(
expected: Vec<ExpectedJob>,
) {
// createSolver()
- let io: Rc<RefCell<dyn IOInterface>> = Rc::new(RefCell::new(NullIO::new()));
+ let io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>> =
+ std::rc::Rc::new(std::cell::RefCell::new(NullIO::new()));
let pool = repo_set
.create_pool(&mut request, io.clone(), None, None, vec![], None, None)
.unwrap();
- let policy: Rc<dyn PolicyInterface> = Rc::new(DefaultPolicy::new(false, false, None));
- let mut solver =
- shirabe::dependency_resolver::solver::Solver::new(policy, Rc::new(RefCell::new(pool)), io);
+ let policy: std::rc::Rc<dyn PolicyInterface> =
+ std::rc::Rc::new(DefaultPolicy::new(false, false, None));
+ let mut solver = shirabe::dependency_resolver::solver::Solver::new(
+ policy,
+ std::rc::Rc::new(std::cell::RefCell::new(pool)),
+ io,
+ );
let transaction = solver.solve(&request, None).unwrap().unwrap();
@@ -197,7 +200,7 @@ struct SolveError {
exception: SolverProblemsException,
repo_set: RepositorySet,
request: Request,
- pool: Rc<RefCell<Pool>>,
+ pool: std::rc::Rc<std::cell::RefCell<Pool>>,
}
/// ref: SolverTest::createSolver + solve, returning the caught SolverProblemsException.
@@ -212,13 +215,15 @@ fn solve_expecting_problems(
.unwrap();
repo_set.add_repository(repo_locked.into()).unwrap();
- let io: Rc<RefCell<dyn IOInterface>> = Rc::new(RefCell::new(NullIO::new()));
- let pool = Rc::new(RefCell::new(
+ let io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>> =
+ std::rc::Rc::new(std::cell::RefCell::new(NullIO::new()));
+ let pool = std::rc::Rc::new(std::cell::RefCell::new(
repo_set
.create_pool(&mut request, io.clone(), None, None, vec![], None, None)
.unwrap(),
));
- let policy: Rc<dyn PolicyInterface> = Rc::new(DefaultPolicy::new(false, false, None));
+ let policy: std::rc::Rc<dyn PolicyInterface> =
+ std::rc::Rc::new(DefaultPolicy::new(false, false, None));
let mut solver = shirabe::dependency_resolver::solver::Solver::new(policy, pool.clone(), io);
let exception = match solver.solve(&request, None).unwrap() {
@@ -2489,13 +2494,18 @@ fn test_learn_positive_literal() {
let mut request = fixtures.request;
request.require_name("A", None).unwrap();
- let io: Rc<RefCell<dyn IOInterface>> = Rc::new(RefCell::new(NullIO::new()));
+ let io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>> =
+ std::rc::Rc::new(std::cell::RefCell::new(NullIO::new()));
let pool = repo_set
.create_pool(&mut request, io.clone(), None, None, vec![], None, None)
.unwrap();
- let policy: Rc<dyn PolicyInterface> = Rc::new(DefaultPolicy::new(false, false, None));
- let mut solver =
- shirabe::dependency_resolver::solver::Solver::new(policy, Rc::new(RefCell::new(pool)), io);
+ let policy: std::rc::Rc<dyn PolicyInterface> =
+ std::rc::Rc::new(DefaultPolicy::new(false, false, None));
+ let mut solver = shirabe::dependency_resolver::solver::Solver::new(
+ policy,
+ std::rc::Rc::new(std::cell::RefCell::new(pool)),
+ io,
+ );
// check correct setup for assertion later
assert!(!solver.test_flag_learned_positive_literal);