diff options
Diffstat (limited to 'xtask/tests')
-rw-r--r-- | xtask/tests/tidy.rs | 35 |
1 files changed, 25 insertions, 10 deletions
diff --git a/xtask/tests/tidy.rs b/xtask/tests/tidy.rs index 68a70da9e..ca9749ed4 100644 --- a/xtask/tests/tidy.rs +++ b/xtask/tests/tidy.rs | |||
@@ -44,11 +44,26 @@ fn rust_files_are_tidy() { | |||
44 | let text = fs2::read_to_string(&path).unwrap(); | 44 | let text = fs2::read_to_string(&path).unwrap(); |
45 | check_todo(&path, &text); | 45 | check_todo(&path, &text); |
46 | check_trailing_ws(&path, &text); | 46 | check_trailing_ws(&path, &text); |
47 | deny_clippy(&path, &text); | ||
47 | tidy_docs.visit(&path, &text); | 48 | tidy_docs.visit(&path, &text); |
48 | } | 49 | } |
49 | tidy_docs.finish(); | 50 | tidy_docs.finish(); |
50 | } | 51 | } |
51 | 52 | ||
53 | fn deny_clippy(path: &PathBuf, text: &String) { | ||
54 | if text.contains("[\u{61}llow(clippy") { | ||
55 | panic!( | ||
56 | "\n\nallowing lints is forbidden: {}. | ||
57 | rust-analyzer intentionally doesn't check clippy on CI. | ||
58 | You can allow lint globally via `xtask clippy`. | ||
59 | See https://github.com/rust-lang/rust-clippy/issues/5537 for discussion. | ||
60 | |||
61 | ", | ||
62 | path.display() | ||
63 | ) | ||
64 | } | ||
65 | } | ||
66 | |||
52 | #[test] | 67 | #[test] |
53 | fn check_licenses() { | 68 | fn check_licenses() { |
54 | let expected = " | 69 | let expected = " |
@@ -179,16 +194,16 @@ impl TidyDocs { | |||
179 | } | 194 | } |
180 | 195 | ||
181 | let poorly_documented = [ | 196 | let poorly_documented = [ |
182 | "ra_hir", | 197 | "hir", |
183 | "ra_hir_expand", | 198 | "hir_expand", |
184 | "ra_ide", | 199 | "ide", |
185 | "ra_mbe", | 200 | "mbe", |
186 | "ra_parser", | 201 | "parser", |
187 | "ra_prof", | 202 | "profile", |
188 | "ra_project_model", | 203 | "project_model", |
189 | "ra_syntax", | 204 | "syntax", |
190 | "ra_tt", | 205 | "tt", |
191 | "ra_hir_ty", | 206 | "hir_ty", |
192 | ]; | 207 | ]; |
193 | 208 | ||
194 | let mut has_fixmes = | 209 | let mut has_fixmes = |