about summary refs log tree commit diff
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2022-01-19 08:55:40 +0000
committerbors <bors@rust-lang.org>2022-01-19 08:55:40 +0000
commit0b143e347c61ab2fc62e9b533d48009aede2f3fc (patch)
treeff449e0b4112ff2c99312e1a598711c34891e716
parent16032c53c9b023ba55c102bc4e94691303399487 (diff)
parent3afbae33ab55f7206460ad34196c816aa2ac8909 (diff)
downloadrust-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.rs1
-rw-r--r--src/driver.rs3
-rw-r--r--tests/compile-test.rs8
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);