Skip to content

Commit

Permalink
Test fixes and review feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
alexcrichton committed Apr 8, 2015
1 parent 2625276 commit 3df822f
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 18 deletions.
30 changes: 15 additions & 15 deletions src/compiletest/runtest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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 }) => {
Expand Down Expand Up @@ -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);
}
Expand Down
2 changes: 1 addition & 1 deletion src/librustdoc/test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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(),
};

Expand Down
2 changes: 0 additions & 2 deletions src/test/run-make/issue-22131/foo.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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);
/// ```
Expand Down

0 comments on commit 3df822f

Please sign in to comment.