From: Carlhuda Date: Fri, 11 Apr 2014 22:14:37 +0000 (-0700) Subject: Whitespace X-Git-Tag: archive/raspbian/0.35.0-2+rpi1~3^2^2^2^2^2^2^2~1117 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=40353e83c6b0982f01567e9dc63281639f3306f5;p=cargo.git Whitespace --- diff --git a/src/cargo/core/resolver.rs b/src/cargo/core/resolver.rs index fcf167eed..09d1e48da 100644 --- a/src/cargo/core/resolver.rs +++ b/src/cargo/core/resolver.rs @@ -4,87 +4,93 @@ use {CargoResult}; #[allow(dead_code)] pub fn resolve(deps: &Vec, registry: &core::Registry) -> CargoResult> { - let mut remaining = deps.clone(); - let mut resolve = HashMap::<&str, &core::Package>::new(); + let mut remaining = deps.clone(); + let mut resolve = HashMap::<&str, &core::Package>::new(); - loop { - let curr = match remaining.pop() { - Some(curr) => curr, - None => return Ok(resolve.values().map(|v| (*v).clone()).collect()) - }; + loop { + let curr = match remaining.pop() { + Some(curr) => curr, + None => return Ok(resolve.values().map(|v| (*v).clone()).collect()) + }; - let opts = registry.query(curr.get_name()); + let opts = registry.query(curr.get_name()); - assert!(!resolve.contains_key_equiv(&curr.get_name()), "already traversed {}", curr.get_name()); - // Temporary, but we must have exactly one option to satisfy the dep - assert!(opts.len() == 1, "invalid num of results {}", opts.len()); + assert!(!resolve.contains_key_equiv(&curr.get_name()), "already traversed {}", curr.get_name()); + // Temporary, but we must have exactly one option to satisfy the dep + assert!(opts.len() == 1, "invalid num of results {}", opts.len()); - let pkg = opts.get(0); - resolve.insert(pkg.get_name(), *pkg); - } + let pkg = opts.get(0); + resolve.insert(pkg.get_name(), *pkg); + } } #[cfg(test)] mod test { - use hamcrest::{ - assert_that, - equal_to, - of_len, - contains - }; - - use core::{ - MemRegistry, - Dependency, - Package - }; - - use super::{ - resolve - }; - - fn pkg(name: &str) -> Package { - Package::new(name, &Vec::::new()) - } - - fn dep(name: &str) -> Dependency { - Dependency::new(name) - } - - fn registry(pkgs: Vec) -> MemRegistry { - MemRegistry::new(&pkgs) - } - - #[test] - pub fn test_resolving_empty_dependency_list() { - let res = resolve(&vec!(), ®istry(vec!())).unwrap(); - - assert_that(&res, equal_to(&Vec::::new())); - } - - #[test] - pub fn test_resolving_only_package() { - let reg = registry(vec!(pkg("foo"))); - let res = resolve(&vec!(dep("foo")), ®); - - assert_that(&res.unwrap(), equal_to(&vec!(pkg("foo")))); - } - - #[test] - pub fn test_resolving_one_dep() { - let reg = registry(vec!(pkg("foo"), pkg("bar"))); - let res = resolve(&vec!(dep("foo")), ®); - - assert_that(&res.unwrap(), equal_to(&vec!(pkg("foo")))); - } - - #[test] - pub fn test_resolving_multiple_deps() { - let reg = registry(vec!(pkg("foo"), pkg("bar"), pkg("baz"))); - let res = resolve(&vec!(dep("foo"), dep("baz")), ®).unwrap(); - - assert_that(&res, of_len(2)); - assert_that(&res, contains(vec!(pkg("foo"), pkg("baz"))).exactly()); - } + use hamcrest::{ + assert_that, + equal_to, + of_len, + contains + }; + + use core::{ + MemRegistry, + Dependency, + Package + }; + + use super::{ + resolve + }; + + + + fn pkg(name: &str) -> Package { + Package::new(name, &Vec::::new()) + } + + fn dep(name: &str) -> Dependency { + Dependency::new(name) + } + + fn registry(pkgs: Vec) -> MemRegistry { + MemRegistry::new(&pkgs) + } + + #[test] + pub fn test_resolving_empty_dependency_list() { + let res = resolve(&vec!(), ®istry(vec!())).unwrap(); + + assert_that(&res, equal_to(&Vec::::new())); + } + + #[test] + pub fn test_resolving_only_package() { + let reg = registry(vec!(pkg("foo"))); + let res = resolve(&vec!(dep("foo")), ®); + + assert_that(&res.unwrap(), equal_to(&vec!(pkg("foo")))); + } + + #[test] + pub fn test_resolving_one_dep() { + let reg = registry(vec!(pkg("foo"), pkg("bar"))); + let res = resolve(&vec!(dep("foo")), ®); + + assert_that(&res.unwrap(), equal_to(&vec!(pkg("foo")))); + } + + #[test] + pub fn test_resolving_multiple_deps() { + let reg = registry(vec!(pkg("foo"), pkg("bar"), pkg("baz"))); + let res = resolve(&vec!(dep("foo"), dep("baz")), ®).unwrap(); + + assert_that(&res, of_len(2)); + assert_that(&res, contains(vec!(pkg("foo"), pkg("baz"))).exactly()); + } + + #[test] + pub fn test_resolving_transitive_deps() { + } }