back to using log-crate for testing,

Author: syphar

src/docbuilder/rustwide_builder.rs

@@ -763,13 +763,15 @@ mod tests {
     #[ignore]
     fn test_build_crate() {
         wrapper(|env| {
+            let crate_ = "log";
+            let version = "0.4.14";
             let default_target = "x86_64-unknown-linux-gnu";
 
             assert_eq!(env.config().include_default_targets, true);
 
             let mut builder = RustwideBuilder::init(env).unwrap();
             builder
-                .build_package(DUMMY_CRATE_NAME, DUMMY_CRATE_VERSION, PackageKind::CratesIo)
+                .build_package(crate_, version, PackageKind::CratesIo)
                 .map(|_| ())?;
 
             // check release record in the db (default and other targets)
@@ -786,7 +788,7 @@ mod tests {
                     WHERE 
                         c.name = $1 AND 
                         r.version = $2",
-                    &[&DUMMY_CRATE_NAME, &DUMMY_CRATE_VERSION],
+                    &[&crate_, &version],
                 )
                 .unwrap();
             let row = rows.get(0).unwrap();
@@ -798,8 +800,8 @@ mod tests {
                 .get::<_, Value>("doc_targets")
                 .as_array()
                 .unwrap()
-                .into_iter()
-                .map(|v| v.as_str().unwrap().clone().to_owned())
+                .iter()
+                .map(|v| v.as_str().unwrap().to_owned())
                 .collect();
             targets.sort();
             assert_eq!(
@@ -814,17 +816,15 @@ mod tests {
             );
 
             let storage = env.storage();
-            let base = format!("rustdoc/{}/{}", DUMMY_CRATE_NAME, DUMMY_CRATE_VERSION);
+            let base = format!("rustdoc/{}/{}", crate_, version);
 
             // default target was built
-            assert!(storage.exists(&format!("{}/{}/index.html", base, DUMMY_CRATE_NAME))?);
+            assert!(storage.exists(&format!("{}/{}/index.html", base, crate_))?);
 
             // other targets too
             for target in DEFAULT_TARGETS {
-                let target_docs_present = storage.exists(&format!(
-                    "{}/{}/{}/index.html",
-                    base, DUMMY_CRATE_NAME, target
-                ))?;
+                let target_docs_present =
+                    storage.exists(&format!("{}/{}/{}/index.html", base, crate_, target))?;
                 if target == &default_target {
                     assert!(!target_docs_present);
                 } else {