diff options
author | Benjamin Coenen <[email protected]> | 2020-04-17 09:32:12 +0100 |
---|---|---|
committer | Benjamin Coenen <[email protected]> | 2020-04-17 09:32:12 +0100 |
commit | 828f69ce549c9e07dec528247c97507749b1aec7 (patch) | |
tree | 201aaf205ee0eb7750785f2e518c28dae29a0186 | |
parent | 8d296be1090b21b60e509c831864ae85feec2490 (diff) |
tests: add more info about what failed in tidy tests
Signed-off-by: Benjamin Coenen <[email protected]>
-rw-r--r-- | xtask/tests/tidy-tests/main.rs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xtask/tests/tidy-tests/main.rs b/xtask/tests/tidy-tests/main.rs index 101ae19bd..ead642acc 100644 --- a/xtask/tests/tidy-tests/main.rs +++ b/xtask/tests/tidy-tests/main.rs | |||
@@ -35,7 +35,7 @@ fn check_todo(path: &Path, text: &str) { | |||
35 | } | 35 | } |
36 | if text.contains("TODO") || text.contains("TOOD") || text.contains("todo!") { | 36 | if text.contains("TODO") || text.contains("TOOD") || text.contains("todo!") { |
37 | panic!( | 37 | panic!( |
38 | "\nTODO markers should not be committed to the master branch,\n\ | 38 | "\nTODO markers or todo! macros should not be committed to the master branch,\n\ |
39 | use FIXME instead\n\ | 39 | use FIXME instead\n\ |
40 | {}\n", | 40 | {}\n", |
41 | path.display(), | 41 | path.display(), |
@@ -47,9 +47,9 @@ fn check_trailing_ws(path: &Path, text: &str) { | |||
47 | if is_exclude_dir(path, &["test_data"]) { | 47 | if is_exclude_dir(path, &["test_data"]) { |
48 | return; | 48 | return; |
49 | } | 49 | } |
50 | for line in text.lines() { | 50 | for (line_number, line) in text.lines().enumerate() { |
51 | if line.chars().last().map(char::is_whitespace) == Some(true) { | 51 | if line.chars().last().map(char::is_whitespace) == Some(true) { |
52 | panic!("Trailing whitespace in {}", path.display()) | 52 | panic!("Trailing whitespace in {} at line {}", path.display(), line_number) |
53 | } | 53 | } |
54 | } | 54 | } |
55 | } | 55 | } |