diff options
| author | bors <bors@rust-lang.org> | 2024-09-21 07:22:44 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2024-09-21 07:22:44 +0000 |
| commit | 0e1ded0f8d9cdfb5e15c2e1bb4f0b78c48a8aded (patch) | |
| tree | c537ea03cf8304b16c27e1dea2ff94f8681a5fd9 | |
| parent | 9be28b143993f48cd0756778b28244763cd72e48 (diff) | |
| parent | 2775dcdd24c2fa6a3596a071450f8863d0746d78 (diff) | |
| download | rust-0e1ded0f8d9cdfb5e15c2e1bb4f0b78c48a8aded.tar.gz rust-0e1ded0f8d9cdfb5e15c2e1bb4f0b78c48a8aded.zip | |
Auto merge of #13384 - Alexendoo:remove-old-collect-metadata, r=llogiq
Remove unused `collect_metadata` function Leftover from #13221 changelog: none
| -rw-r--r-- | tests/dogfood.rs | 49 |
1 files changed, 1 insertions, 48 deletions
diff --git a/tests/dogfood.rs b/tests/dogfood.rs index 8d10d5e7161..858be389a9e 100644 --- a/tests/dogfood.rs +++ b/tests/dogfood.rs @@ -6,11 +6,9 @@ #![warn(rust_2018_idioms, unused_lifetimes)] use itertools::Itertools; -use std::fs::File; use std::io::{self, IsTerminal}; use std::path::PathBuf; use std::process::Command; -use std::time::SystemTime; use test_utils::IS_RUSTC_TEST_SUITE; use ui_test::Args; @@ -28,11 +26,7 @@ fn main() { println!("dogfood: test"); } } else if !args.skip.iter().any(|arg| arg == "dogfood") { - if args.filters.iter().any(|arg| arg == "collect_metadata") { - collect_metadata(); - } else { - dogfood(); - } + dogfood(); } } @@ -61,47 +55,6 @@ fn dogfood() { ); } -fn collect_metadata() { - assert!(cfg!(feature = "internal")); - - // Setup for validation - let metadata_output_path = PathBuf::from(env!("CARGO_MANIFEST_DIR")).join("util/gh-pages/lints.json"); - let start_time = SystemTime::now(); - - // Run collection as is - std::env::set_var("ENABLE_METADATA_COLLECTION", "1"); - assert!(run_clippy_for_package( - "clippy_lints", - &["-A", "unfulfilled_lint_expectations"] - )); - - // Check if cargo caching got in the way - if let Ok(file) = File::open(metadata_output_path) { - if let Ok(metadata) = file.metadata() { - if let Ok(last_modification) = metadata.modified() { - if last_modification > start_time { - // The output file has been modified. Most likely by a hungry - // metadata collection monster. So We'll return. - return; - } - } - } - } - - // Force cargo to invalidate the caches - filetime::set_file_mtime( - PathBuf::from(env!("CARGO_MANIFEST_DIR")).join("clippy_lints/src/lib.rs"), - filetime::FileTime::now(), - ) - .unwrap(); - - // Running the collection again - assert!(run_clippy_for_package( - "clippy_lints", - &["-A", "unfulfilled_lint_expectations"] - )); -} - #[must_use] fn run_clippy_for_package(project: &str, args: &[&str]) -> bool { let root_dir = PathBuf::from(env!("CARGO_MANIFEST_DIR")); |
