diff options
| author | bors <bors@rust-lang.org> | 2022-01-19 08:55:40 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2022-01-19 08:55:40 +0000 |
| commit | 0b143e347c61ab2fc62e9b533d48009aede2f3fc (patch) | |
| tree | ff449e0b4112ff2c99312e1a598711c34891e716 | |
| parent | 16032c53c9b023ba55c102bc4e94691303399487 (diff) | |
| parent | 3afbae33ab55f7206460ad34196c816aa2ac8909 (diff) | |
| download | rust-0b143e347c61ab2fc62e9b533d48009aede2f3fc.tar.gz rust-0b143e347c61ab2fc62e9b533d48009aede2f3fc.zip | |
Auto merge of #8304 - camsteffen:test-tweaks, r=flip1995
Remove __CLIPPY_INTERNAL_TESTS env var changelog: none It doesn't have any effect AFAICT.
| -rw-r--r-- | clippy_dev/src/lint.rs | 1 | ||||
| -rw-r--r-- | src/driver.rs | 3 | ||||
| -rw-r--r-- | tests/compile-test.rs | 8 |
3 files changed, 2 insertions, 10 deletions
diff --git a/clippy_dev/src/lint.rs b/clippy_dev/src/lint.rs index dfd16f71054..b8287980a4b 100644 --- a/clippy_dev/src/lint.rs +++ b/clippy_dev/src/lint.rs @@ -7,7 +7,6 @@ pub fn run(filename: &str) { .args(["-Z", "no-codegen"]) .args(["--edition", "2021"]) .arg(filename) - .env("__CLIPPY_INTERNAL_TESTS", "true") .status() .expect("failed to run cargo") .code(); diff --git a/src/driver.rs b/src/driver.rs index a8aa3a76abc..8f8f1140a3d 100644 --- a/src/driver.rs +++ b/src/driver.rs @@ -331,11 +331,10 @@ pub fn main() { // - IF Clippy is run on the main crate, not on deps (`!cap_lints_allow`) THEN // - IF `--no-deps` is not set (`!no_deps`) OR // - IF `--no-deps` is set and Clippy is run on the specified primary package - let clippy_tests_set = env::var("__CLIPPY_INTERNAL_TESTS").map_or(false, |val| val == "true"); let cap_lints_allow = arg_value(&orig_args, "--cap-lints", |val| val == "allow").is_some(); let in_primary_package = env::var("CARGO_PRIMARY_PACKAGE").is_ok(); - let clippy_enabled = clippy_tests_set || (!cap_lints_allow && (!no_deps || in_primary_package)); + let clippy_enabled = !cap_lints_allow && (!no_deps || in_primary_package); if clippy_enabled { args.extend(clippy_args); } diff --git a/tests/compile-test.rs b/tests/compile-test.rs index 531890c863f..6505028db9f 100644 --- a/tests/compile-test.rs +++ b/tests/compile-test.rs @@ -328,15 +328,9 @@ fn run_ui_cargo(config: &mut compiletest::Config) { } } -fn prepare_env() { - set_var("CLIPPY_DISABLE_DOCS_LINKS", "true"); - set_var("__CLIPPY_INTERNAL_TESTS", "true"); - //set_var("RUST_BACKTRACE", "0"); -} - #[test] fn compile_test() { - prepare_env(); + set_var("CLIPPY_DISABLE_DOCS_LINKS", "true"); let mut config = default_config(); run_ui(&mut config); run_ui_test(&mut config); |
