-extern crate compiletest_rs as compiletest;
+#![feature(test)] // compiletest_rs requires this attribute
-use std::path::PathBuf;
-use std::env::{set_var, var};
+use compiletest_rs as compiletest;
+use compiletest_rs::common::Mode as TestMode;
+
+use std::env::{self, set_var};
+use std::ffi::OsStr;
+use std::fs;
+use std::io;
+use std::path::{Path, PathBuf};
+
+mod cargo;
+
+fn host_lib() -> PathBuf {
+ if let Some(path) = option_env!("HOST_LIBS") {
+ PathBuf::from(path)
+ } else {
+ cargo::CARGO_TARGET_DIR.join(env!("PROFILE"))
+ }
+}
fn clippy_driver_path() -> PathBuf {
if let Some(path) = option_env!("CLIPPY_DRIVER_PATH") {
PathBuf::from(path)
} else {
- PathBuf::from(concat!("target/", env!("PROFILE"), "/clippy-driver"))
+ cargo::TARGET_LIB.join("clippy-driver")
+ }
+}
+
+// When we'll want to use `extern crate ..` for a dependency that is used
+// both by the crate and the compiler itself, we can't simply pass -L flags
+// as we'll get a duplicate matching versions. Instead, disambiguate with
+// `--extern dep=path`.
+// See https://github.com/rust-lang/rust-clippy/issues/4015.
+//
+// FIXME: We cannot use `cargo build --message-format=json` to resolve to dependency files.
+// Because it would force-rebuild if the options passed to `build` command is not the same
+// as what we manually pass to `cargo` invocation
+fn third_party_crates() -> String {
+ use std::collections::HashMap;
+ static CRATES: &[&str] = &["serde", "serde_derive", "regex", "clippy_lints"];
+ let dep_dir = cargo::TARGET_LIB.join("deps");
+ let mut crates: HashMap<&str, PathBuf> = HashMap::with_capacity(CRATES.len());
+ for entry in fs::read_dir(dep_dir).unwrap() {
+ let path = match entry {
+ Ok(entry) => entry.path(),
+ _ => continue,
+ };
+ if let Some(name) = path.file_name().and_then(OsStr::to_str) {
+ for dep in CRATES {
+ if name.starts_with(&format!("lib{}-", dep)) && name.ends_with(".rlib") {
+ crates.entry(dep).or_insert(path);
+ break;
+ }
+ }
+ }
}
+
+ let v: Vec<_> = crates
+ .into_iter()
+ .map(|(dep, path)| format!("--extern {}={}", dep, path.display()))
+ .collect();
+ v.join(" ")
}
-fn run_mode(dir: &'static str, mode: &'static str) {
+fn default_config() -> compiletest::Config {
let mut config = compiletest::Config::default();
- let cfg_mode = mode.parse().expect("Invalid mode");
- config.target_rustcflags = Some("-L target/debug/ -L target/debug/deps -Dwarnings".to_owned());
- if let Ok(name) = var::<&str>("TESTNAME") {
- let s: String = name.to_owned();
- config.filter = Some(s)
+ if let Ok(name) = env::var("TESTNAME") {
+ config.filter = Some(name);
+ }
+
+ if let Some(path) = option_env!("RUSTC_LIB_PATH") {
+ let path = PathBuf::from(path);
+ config.run_lib_path = path.clone();
+ config.compile_lib_path = path;
}
- config.mode = cfg_mode;
- config.build_base = PathBuf::from("target/debug/test_build_base")
- .canonicalize()
- .unwrap();
- config.src_base = PathBuf::from(format!("tests/{}", dir));
+ config.target_rustcflags = Some(format!(
+ "-L {0} -L {1} -Dwarnings -Zui-testing {2}",
+ host_lib().join("deps").display(),
+ cargo::TARGET_LIB.join("deps").display(),
+ third_party_crates(),
+ ));
+
+ config.build_base = if cargo::is_rustc_test_suite() {
+ // This make the stderr files go to clippy OUT_DIR on rustc repo build dir
+ let mut path = PathBuf::from(env!("OUT_DIR"));
+ path.push("test_build_base");
+ path
+ } else {
+ host_lib().join("test_build_base")
+ };
config.rustc_path = clippy_driver_path();
+ config
+}
+
+fn run_mode(cfg: &mut compiletest::Config) {
+ cfg.mode = TestMode::Ui;
+ cfg.src_base = Path::new("tests").join("ui");
+ compiletest::run_tests(&cfg);
+}
- compiletest::run_tests(&config);
+#[allow(clippy::identity_conversion)]
+fn run_ui_toml_tests(config: &compiletest::Config, mut tests: Vec<tester::TestDescAndFn>) -> Result<bool, io::Error> {
+ let mut result = true;
+ let opts = compiletest::test_opts(config);
+ for dir in fs::read_dir(&config.src_base)? {
+ let dir = dir?;
+ if !dir.file_type()?.is_dir() {
+ continue;
+ }
+ let dir_path = dir.path();
+ set_var("CARGO_MANIFEST_DIR", &dir_path);
+ for file in fs::read_dir(&dir_path)? {
+ let file = file?;
+ let file_path = file.path();
+ if file.file_type()?.is_dir() {
+ continue;
+ }
+ if file_path.extension() != Some(OsStr::new("rs")) {
+ continue;
+ }
+ let paths = compiletest::common::TestPaths {
+ file: file_path,
+ base: config.src_base.clone(),
+ relative_dir: dir_path.file_name().unwrap().into(),
+ };
+ let test_name = compiletest::make_test_name(&config, &paths);
+ let index = tests
+ .iter()
+ .position(|test| test.desc.name == test_name)
+ .expect("The test should be in there");
+ result &= tester::run_tests_console(&opts, vec![tests.swap_remove(index)])?;
+ }
+ }
+ Ok(result)
+}
+
+fn run_ui_toml(config: &mut compiletest::Config) {
+ config.mode = TestMode::Ui;
+ config.src_base = Path::new("tests").join("ui-toml").canonicalize().unwrap();
+
+ let tests = compiletest::make_tests(&config);
+
+ let res = run_ui_toml_tests(&config, tests);
+ match res {
+ Ok(true) => {},
+ Ok(false) => panic!("Some tests failed"),
+ Err(e) => {
+ println!("I/O failure during tests: {:?}", e);
+ },
+ }
}
fn prepare_env() {
set_var("CLIPPY_DISABLE_DOCS_LINKS", "true");
set_var("CLIPPY_TESTS", "true");
- set_var("RUST_BACKTRACE", "0");
+ //set_var("RUST_BACKTRACE", "0");
}
#[test]
fn compile_test() {
prepare_env();
- run_mode("run-pass", "run-pass");
- run_mode("ui", "ui");
+ let mut config = default_config();
+ run_mode(&mut config);
+ run_ui_toml(&mut config);
}
projects / rust.git / blobdiff