Test fixes and review feedback

Author: alexcrichton

src/compiletest/runtest.rs

@@ -726,18 +726,18 @@ fn run_debuginfo_lldb_test(config: &Config, props: &TestProps, testfile: &Path)
                 -> ProcRes {
         // Prepare the lldb_batchmode which executes the debugger script
         let lldb_script_path = rust_src_root.join("src/etc/lldb_batchmode.py");
-        cmd2proces(config,
-                   test_executable,
-                   Command::new(&config.python)
-                           .arg(&lldb_script_path)
-                           .arg(test_executable)
-                           .arg(debugger_script)
-                           .env("PYTHONPATH",
-                                config.lldb_python_dir.as_ref().unwrap()))
+        cmd2procres(config,
+                    test_executable,
+                    Command::new(&config.python)
+                            .arg(&lldb_script_path)
+                            .arg(test_executable)
+                            .arg(debugger_script)
+                            .env("PYTHONPATH",
+                                 config.lldb_python_dir.as_ref().unwrap()))
     }
 }
 
-fn cmd2proces(config: &Config, test_executable: &Path, cmd: &mut Command)
+fn cmd2procres(config: &Config, test_executable: &Path, cmd: &mut Command)
               -> ProcRes {
     let (status, out, err) = match cmd.output() {
         Ok(Output { status, stdout, stderr }) => {
@@ -1828,12 +1828,12 @@ fn run_rustdoc_test(config: &Config, props: &TestProps, testfile: &Path) {
     }
     let root = find_rust_src_root(config).unwrap();
 
-    let res = cmd2proces(config,
-                         testfile,
-                         Command::new(&config.python)
-                                 .arg(root.join("src/etc/htmldocck.py"))
-                                 .arg(out_dir)
-                                 .arg(testfile));
+    let res = cmd2procres(config,
+                          testfile,
+                          Command::new(&config.python)
+                                  .arg(root.join("src/etc/htmldocck.py"))
+                                  .arg(out_dir)
+                                  .arg(testfile));
     if !res.status.success() {
         fatal_proc_rec("htmldocck failed!", &res);
     }

src/librustdoc/test.rs

@@ -124,7 +124,7 @@ fn scrape_test_config(krate: &::syntax::ast::Crate) -> TestOptions {
     use syntax::print::pprust;
 
     let mut opts = TestOptions {
-        no_crate_inject: true,
+        no_crate_inject: false,
         attrs: Vec::new(),
     };
 

src/test/run-make/issue-22131/foo.rs

@@ -8,8 +8,6 @@
 // option. This file may not be copied, modified, or distributed
 // except according to those terms.
 
-#![crate_name="foo"]
-
 /// ```rust
 /// assert_eq!(foo::foo(), 1);
 /// ```

src/test/rustdoc/default-impl.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:rustdoc-default-impl.rs
+// ignore-android
 
 extern crate rustdoc_default_impl as foo;
 

src/test/rustdoc/extern-default-method.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:rustdoc-extern-default-method.rs
+// ignore-android
 
 extern crate rustdoc_extern_default_method as ext;
 

src/test/rustdoc/extern-method.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:rustdoc-extern-method.rs
+// ignore-android
 
 #![feature(unboxed_closures)]
 

src/test/rustdoc/ffi.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:rustdoc-ffi.rs
+// ignore-android
 
 extern crate rustdoc_ffi as lib;
 

src/test/rustdoc/inline-default-methods.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:inline-default-methods.rs
+// ignore-android
 
 extern crate inline_default_methods;
 

src/test/rustdoc/issue-13698.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-13698.rs
+// ignore-android
 
 extern crate issue_13698;
 

src/test/rustdoc/issue-15318-2.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-15318.rs
+// ignore-android
 
 extern crate issue_15318;
 

src/test/rustdoc/issue-15318.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-15318.rs
+// ignore-android
 
 #![feature(no_std)]
 #![no_std]

src/test/rustdoc/issue-17476.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-17476.rs
+// ignore-android
 
 extern crate issue_17476;
 

src/test/rustdoc/issue-20646.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-20646.rs
+// ignore-android
 
 #![feature(associated_types)]
 

src/test/rustdoc/issue-20727-2.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-20727.rs
+// ignore-android
 
 extern crate issue_20727;
 

src/test/rustdoc/issue-20727-3.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-20727.rs
+// ignore-android
 
 extern crate issue_20727;
 

src/test/rustdoc/issue-20727-4.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-20727.rs
+// ignore-android
 
 extern crate issue_20727;
 

src/test/rustdoc/issue-20727.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-20727.rs
+// ignore-android
 
 extern crate issue_20727;
 

src/test/rustdoc/issue-21092.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-21092.rs
+// ignore-android
 
 extern crate issue_21092;
 

src/test/rustdoc/issue-21801.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-21801.rs
+// ignore-android
 
 extern crate issue_21801;
 

src/test/rustdoc/issue-22025.rs

@@ -9,6 +9,7 @@
 // except according to those terms.
 
 // aux-build:issue-22025.rs
+// ignore-android
 
 extern crate issue_22025;
 

src/test/rustdoc/issue-23207.rs

@@ -10,6 +10,7 @@
 
 // aux-build:issue-23207-1.rs
 // aux-build:issue-23207-2.rs
+// ignore-android
 
 extern crate issue_23207_2;