about summary refs log tree commit diff
diff options
context:
space:
mode:
authorTomasz Miąsko <tomasz.miasko@gmail.com>2021-02-21 00:00:00 +0000
committerTomasz Miąsko <tomasz.miasko@gmail.com>2021-02-21 00:00:00 +0000
commit88753cead814a0830acdb527f513957fb9fe620a (patch)
tree9dd2b9d41fe8b025f76b780d7df71f12c931d3c2
parent83b30a639d5abd1270ade35d9bd92271f5a5ba18 (diff)
downloadrust-88753cead814a0830acdb527f513957fb9fe620a.tar.gz
rust-88753cead814a0830acdb527f513957fb9fe620a.zip
test: Print test name only once on timeout
Pretty formatter when using multiple test threads displays test name twice on
timeout event. This implicitly suggest that those are two different events,
while in fact they are always printed together.

Print test name only once.

Before:

```
running 3 tests
test src/lib.rs - c (line 16) ... ok
test src/lib.rs - a (line 3) ... ok
test src/lib.rs - b (line 9) ... test src/lib.rs - b (line 9) has been running for over 60 seconds
test src/lib.rs - b (line 9) ... ok
```

After:

```
running 3 tests
test src/lib.rs - c (line 16) ... ok
test src/lib.rs - a (line 3) ... ok
test src/lib.rs - b (line 9) has been running for over 60 seconds
test src/lib.rs - b (line 9) ... ok
```
-rw-r--r--library/test/src/formatters/pretty.rs4
1 files changed, 0 insertions, 4 deletions
diff --git a/library/test/src/formatters/pretty.rs b/library/test/src/formatters/pretty.rs
index 6fa36929841..5e41d6d9692 100644
--- a/library/test/src/formatters/pretty.rs
+++ b/library/test/src/formatters/pretty.rs
@@ -222,10 +222,6 @@ impl<T: Write> OutputFormatter for PrettyFormatter<T> {
     }
 
     fn write_timeout(&mut self, desc: &TestDesc) -> io::Result<()> {
-        if self.is_multithreaded {
-            self.write_test_name(desc)?;
-        }
-
         self.write_plain(&format!(
             "test {} has been running for over {} seconds\n",
             desc.name,