Skip to content

Commit 8b3fd98

Browse files
author
Gilad Naaman
committed
libtest: rustfmt run
libtest: Whoops
1 parent 94bd121 commit 8b3fd98

File tree

3 files changed

+747
-576
lines changed

3 files changed

+747
-576
lines changed

src/libtest/formatters.rs

Lines changed: 101 additions & 77 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,12 @@ pub(crate) trait OutputFormatter {
1414
fn write_run_start(&mut self, test_count: usize) -> io::Result<()>;
1515
fn write_test_start(&mut self, desc: &TestDesc) -> io::Result<()>;
1616
fn write_timeout(&mut self, desc: &TestDesc) -> io::Result<()>;
17-
fn write_result(&mut self,
18-
desc: &TestDesc,
19-
result: &TestResult,
20-
stdout: &[u8]) -> io::Result<()>;
17+
fn write_result(
18+
&mut self,
19+
desc: &TestDesc,
20+
result: &TestResult,
21+
stdout: &[u8],
22+
) -> io::Result<()>;
2123
fn write_run_finish(&mut self, state: &ConsoleTestState) -> io::Result<bool>;
2224
}
2325

@@ -34,11 +36,13 @@ pub(crate) struct HumanFormatter<T> {
3436
}
3537

3638
impl<T: Write> HumanFormatter<T> {
37-
pub fn new(out: OutputLocation<T>,
38-
use_color: bool,
39-
terse: bool,
40-
max_name_len: usize,
41-
is_multithreaded: bool) -> Self {
39+
pub fn new(
40+
out: OutputLocation<T>,
41+
use_color: bool,
42+
terse: bool,
43+
max_name_len: usize,
44+
is_multithreaded: bool,
45+
) -> Self {
4246
HumanFormatter {
4347
out,
4448
terse,
@@ -74,8 +78,12 @@ impl<T: Write> HumanFormatter<T> {
7478
self.write_pretty("bench", term::color::CYAN)
7579
}
7680

77-
pub fn write_short_result(&mut self, verbose: &str, quiet: &str, color: term::color::Color)
78-
-> io::Result<()> {
81+
pub fn write_short_result(
82+
&mut self,
83+
verbose: &str,
84+
quiet: &str,
85+
color: term::color::Color,
86+
) -> io::Result<()> {
7987
if self.terse {
8088
self.write_pretty(quiet, color)?;
8189
if self.test_count % QUIET_MODE_MAX_COLUMN == QUIET_MODE_MAX_COLUMN - 1 {
@@ -182,11 +190,7 @@ impl<T: Write> HumanFormatter<T> {
182190

183191
impl<T: Write> OutputFormatter for HumanFormatter<T> {
184192
fn write_run_start(&mut self, test_count: usize) -> io::Result<()> {
185-
let noun = if test_count != 1 {
186-
"tests"
187-
} else {
188-
"test"
189-
};
193+
let noun = if test_count != 1 { "tests" } else { "test" };
190194
self.write_plain(&format!("\nrunning {} {}\n", test_count, noun))
191195
}
192196

@@ -224,9 +228,11 @@ impl<T: Write> OutputFormatter for HumanFormatter<T> {
224228
self.write_test_name(desc)?;
225229
}
226230

227-
self.write_plain(&format!("test {} has been running for over {} seconds\n",
228-
desc.name,
229-
TEST_WARN_TIMEOUT_S))
231+
self.write_plain(&format!(
232+
"test {} has been running for over {} seconds\n",
233+
desc.name,
234+
TEST_WARN_TIMEOUT_S
235+
))
230236
}
231237

232238
fn write_run_finish(&mut self, state: &ConsoleTestState) -> io::Result<bool> {
@@ -255,15 +261,17 @@ impl<T: Write> OutputFormatter for HumanFormatter<T> {
255261
state.allowed_fail,
256262
state.ignored,
257263
state.measured,
258-
state.filtered_out)
264+
state.filtered_out
265+
)
259266
} else {
260267
format!(
261268
". {} passed; {} failed; {} ignored; {} measured; {} filtered out\n\n",
262269
state.passed,
263270
state.failed,
264271
state.ignored,
265272
state.measured,
266-
state.filtered_out)
273+
state.filtered_out
274+
)
267275
};
268276

269277
self.write_plain(&s)?;
@@ -273,7 +281,7 @@ impl<T: Write> OutputFormatter for HumanFormatter<T> {
273281
}
274282

275283
pub(crate) struct JsonFormatter<T> {
276-
out: OutputLocation<T>
284+
out: OutputLocation<T>,
277285
}
278286

279287
impl<T: Write> JsonFormatter<T> {
@@ -288,108 +296,121 @@ impl<T: Write> JsonFormatter<T> {
288296
self.out.write_all(b"\n")
289297
}
290298

291-
fn write_event(&mut self,
292-
ty: &str,
293-
name: &str,
294-
evt: &str,
295-
extra: Option<String>) -> io::Result<()> {
299+
fn write_event(
300+
&mut self,
301+
ty: &str,
302+
name: &str,
303+
evt: &str,
304+
extra: Option<String>,
305+
) -> io::Result<()> {
296306
if let Some(extras) = extra {
297-
self.write_message(&*format!(r#"{{ "type": "{}", "name": "{}", "event": "{}", {} }}"#,
298-
ty,
299-
name,
300-
evt,
301-
extras))
302-
}
303-
else {
304-
self.write_message(&*format!(r#"{{ "type": "{}", "name": "{}", "event": "{}" }}"#,
305-
ty,
306-
name,
307-
evt))
307+
self.write_message(&*format!(
308+
r#"{{ "type": "{}", "name": "{}", "event": "{}", {} }}"#,
309+
ty,
310+
name,
311+
evt,
312+
extras
313+
))
314+
} else {
315+
self.write_message(&*format!(
316+
r#"{{ "type": "{}", "name": "{}", "event": "{}" }}"#,
317+
ty,
318+
name,
319+
evt
320+
))
308321
}
309322
}
310323
}
311324

312325
impl<T: Write> OutputFormatter for JsonFormatter<T> {
313326
fn write_run_start(&mut self, test_count: usize) -> io::Result<()> {
314-
self.write_message(
315-
&*format!(r#"{{ "type": "suite", "event": "started", "test_count": "{}" }}"#,
316-
test_count))
327+
self.write_message(&*format!(
328+
r#"{{ "type": "suite", "event": "started", "test_count": "{}" }}"#,
329+
test_count
330+
))
317331
}
318332

319333
fn write_test_start(&mut self, desc: &TestDesc) -> io::Result<()> {
320-
self.write_message(&*format!(r#"{{ "type": "test", "event": "started", "name": "{}" }}"#,
321-
desc.name))
334+
self.write_message(&*format!(
335+
r#"{{ "type": "test", "event": "started", "name": "{}" }}"#,
336+
desc.name
337+
))
322338
}
323339

324-
fn write_result(&mut self,
325-
desc: &TestDesc,
326-
result: &TestResult,
327-
stdout: &[u8]) -> io::Result<()> {
340+
fn write_result(
341+
&mut self,
342+
desc: &TestDesc,
343+
result: &TestResult,
344+
stdout: &[u8],
345+
) -> io::Result<()> {
328346
match *result {
329-
TrOk => {
330-
self.write_event("test", desc.name.as_slice(), "ok", None)
331-
},
347+
TrOk => self.write_event("test", desc.name.as_slice(), "ok", None),
332348

333349
TrFailed => {
334350
let extra_data = if stdout.len() > 0 {
335-
Some(format!(r#""stdout": "{}""#,
336-
EscapedString(String::from_utf8_lossy(stdout))))
337-
}
338-
else {
351+
Some(format!(
352+
r#""stdout": "{}""#,
353+
EscapedString(String::from_utf8_lossy(stdout))
354+
))
355+
} else {
339356
None
340357
};
341358

342359
self.write_event("test", desc.name.as_slice(), "failed", extra_data)
343-
},
360+
}
344361

345362
TrFailedMsg(ref m) => {
346-
self.write_event("test",
347-
desc.name.as_slice(),
348-
"failed",
349-
Some(format!(r#""message": "{}""#, EscapedString(m))))
350-
},
363+
self.write_event(
364+
"test",
365+
desc.name.as_slice(),
366+
"failed",
367+
Some(format!(r#""message": "{}""#, EscapedString(m))),
368+
)
369+
}
351370

352-
TrIgnored => {
353-
self.write_event("test", desc.name.as_slice(), "ignored", None)
354-
},
371+
TrIgnored => self.write_event("test", desc.name.as_slice(), "ignored", None),
355372

356373
TrAllowedFail => {
357374
self.write_event("test", desc.name.as_slice(), "allowed_failure", None)
358-
},
375+
}
359376

360377
TrBench(ref bs) => {
361378
let median = bs.ns_iter_summ.median as usize;
362379
let deviation = (bs.ns_iter_summ.max - bs.ns_iter_summ.min) as usize;
363380

364381
let mbps = if bs.mb_s == 0 {
365382
"".into()
366-
}
367-
else {
383+
} else {
368384
format!(r#", "mib_per_second": {}"#, bs.mb_s)
369385
};
370386

371-
let line = format!("{{ \"type\": \"bench\", \
387+
let line = format!(
388+
"{{ \"type\": \"bench\", \
372389
\"name\": \"{}\", \
373390
\"median\": {}, \
374391
\"deviation\": {}{} }}",
375-
desc.name,
376-
median,
377-
deviation,
378-
mbps);
392+
desc.name,
393+
median,
394+
deviation,
395+
mbps
396+
);
379397

380398
self.write_message(&*line)
381-
},
399+
}
382400
}
383401
}
384402

385403
fn write_timeout(&mut self, desc: &TestDesc) -> io::Result<()> {
386-
self.write_message(&*format!(r#"{{ "type": "test", "event": "timeout", "name": "{}" }}"#,
387-
desc.name))
404+
self.write_message(&*format!(
405+
r#"{{ "type": "test", "event": "timeout", "name": "{}" }}"#,
406+
desc.name
407+
))
388408
}
389409

390410
fn write_run_finish(&mut self, state: &ConsoleTestState) -> io::Result<bool> {
391411

392-
self.write_message(&*format!("{{ \"type\": \"suite\", \
412+
self.write_message(&*format!(
413+
"{{ \"type\": \"suite\", \
393414
\"event\": \"{}\", \
394415
\"passed\": {}, \
395416
\"failed\": {}, \
@@ -403,7 +424,8 @@ impl<T: Write> OutputFormatter for JsonFormatter<T> {
403424
state.allowed_fail,
404425
state.ignored,
405426
state.measured,
406-
state.filtered_out))?;
427+
state.filtered_out
428+
))?;
407429

408430
Ok(state.failed == 0)
409431
}
@@ -454,7 +476,9 @@ impl<S: AsRef<str>> ::std::fmt::Display for EscapedString<S> {
454476
b'\x1e' => "\\u001e",
455477
b'\x1f' => "\\u001f",
456478
b'\x7f' => "\\u007f",
457-
_ => { continue; }
479+
_ => {
480+
continue;
481+
}
458482
};
459483

460484
if start < i {

0 commit comments

Comments
 (0)