about summary refs log tree commit diff
path: root/library/test/src/formatters
diff options
context:
space:
mode:
authorAliƩnore Bouttefeux <alienore.bouttefeux@gmail.com>2021-05-18 18:17:36 +0200
committerAliƩnore Bouttefeux <alienore.bouttefeux@gmail.com>2021-05-18 18:18:28 +0200
commit6de13c3ffc969ceac87f2a8466c5cd850288721c (patch)
treed235f1dded3907fddfb911cbcfb63660a2cc124b /library/test/src/formatters
parent3d8180635289437ac96b20ec94419394194f7c83 (diff)
change based on review
Diffstat (limited to 'library/test/src/formatters')
-rw-r--r--library/test/src/formatters/pretty.rs7
-rw-r--r--library/test/src/formatters/terse.rs7
2 files changed, 6 insertions, 8 deletions
diff --git a/library/test/src/formatters/pretty.rs b/library/test/src/formatters/pretty.rs
index b3efb2c4437..e17fc08a9ae 100644
--- a/library/test/src/formatters/pretty.rs
+++ b/library/test/src/formatters/pretty.rs
@@ -169,11 +169,10 @@ impl<T: Write> PrettyFormatter<T> {
 
     fn write_test_name(&mut self, desc: &TestDesc) -> io::Result<()> {
         let name = desc.padded_name(self.max_name_len, desc.name.padding());
-        let test_mode = desc.test_mode();
-        if test_mode == "" {
-            self.write_plain(&format!("test {} ... ", name))?;
-        } else {
+        if let Some(test_mode) = desc.test_mode() {
             self.write_plain(&format!("test {} - {} ... ", name, test_mode))?;
+        } else {
+            self.write_plain(&format!("test {} ... ", name))?;
         }
 
         Ok(())
diff --git a/library/test/src/formatters/terse.rs b/library/test/src/formatters/terse.rs
index ce73f8d3bfb..a2c223c494c 100644
--- a/library/test/src/formatters/terse.rs
+++ b/library/test/src/formatters/terse.rs
@@ -158,11 +158,10 @@ impl<T: Write> TerseFormatter<T> {
 
     fn write_test_name(&mut self, desc: &TestDesc) -> io::Result<()> {
         let name = desc.padded_name(self.max_name_len, desc.name.padding());
-        let test_mode = desc.test_mode();
-        if test_mode == "" {
-            self.write_plain(&format!("test {} ... ", name))?;
-        } else {
+        if let Some(test_mode) = desc.test_mode() {
             self.write_plain(&format!("test {} - {} ... ", name, test_mode))?;
+        } else {
+            self.write_plain(&format!("test {} ... ", name))?;
         }
 
         Ok(())