Browse Source

Fix verbose test display (#2731)

Small display bug introduced by #2720
pull/2734/head
José Julián Espina 2 years ago
parent
commit
d448e8e7bb
  1. 38
      boa_tester/src/edition.rs
  2. 10
      boa_tester/src/exec/mod.rs
  3. 4
      boa_tester/src/main.rs

38
boa_tester/src/edition.rs

@ -276,27 +276,45 @@ static FEATURE_EDITION: phf::Map<&'static str, SpecEdition> = phf::phf_map! {
)] )]
#[serde(untagged)] #[serde(untagged)]
pub(crate) enum SpecEdition { pub(crate) enum SpecEdition {
/// [ECMAScript 5.1 Edition](https://262.ecma-international.org/5.1) /// ECMAScript 5.1 Edition
///
/// <https://262.ecma-international.org/5.1>
ES5 = 5, ES5 = 5,
/// [ECMAScript 6th Edition](https://262.ecma-international.org/6.0) /// ECMAScript 6th Edition
///
/// <https://262.ecma-international.org/6.0>
ES6, ES6,
/// [ECMAScript 7th Edition](https://262.ecma-international.org/7.0) /// ECMAScript 7th Edition
///
/// <https://262.ecma-international.org/7.0>
ES7, ES7,
/// [ECMAScript 8th Edition](https://262.ecma-international.org/8.0) /// ECMAScript 8th Edition
///
/// <https://262.ecma-international.org/8.0>
ES8, ES8,
/// [ECMAScript 9th Edition](https://262.ecma-international.org/9.0) /// ECMAScript 9th Edition
///
/// <https://262.ecma-international.org/9.0>
ES9, ES9,
/// [ECMAScript 10th Edition](https://262.ecma-international.org/10.0) /// ECMAScript 10th Edition
///
/// <https://262.ecma-international.org/10.0>
ES10, ES10,
/// [ECMAScript 11th Edition](https://262.ecma-international.org/11.0) /// ECMAScript 11th Edition
///
/// <https://262.ecma-international.org/11.0>
ES11, ES11,
/// [ECMAScript 12th Edition](https://262.ecma-international.org/12.0) /// ECMAScript 12th Edition
///
/// <https://262.ecma-international.org/12.0>
ES12, ES12,
/// [ECMAScript 13th Edition](https://262.ecma-international.org/13.0) /// ECMAScript 13th Edition
///
/// <https://262.ecma-international.org/13.0>
ES13, ES13,
/// The edition being worked on right now. /// The edition being worked on right now.
/// ///
/// A draft is currently available in <https://tc39.es/ecma262>. /// A draft is currently available [here](https://tc39.es/ecma262).
#[default] #[default]
ESNext, ESNext,
} }

10
boa_tester/src/exec/mod.rs

@ -105,21 +105,19 @@ impl TestSuite {
if verbose != 0 { if verbose != 0 {
println!( println!(
"Suite {} results: total: {}, passed: {}, ignored: {}, failed: {} (panics: \ "Suite {} results: total: {}, passed: {}, ignored: {}, failed: {} {}, conformance: {:.2}%",
{}{}), conformance: {:.2}%",
es_next.total,
self.path.display(), self.path.display(),
es_next.total,
es_next.passed.to_string().green(), es_next.passed.to_string().green(),
es_next.ignored.to_string().yellow(), es_next.ignored.to_string().yellow(),
(es_next.total - es_next.passed - es_next.ignored) (es_next.total - es_next.passed - es_next.ignored)
.to_string() .to_string()
.red(), .red(),
if es_next.panic == 0 { if es_next.panic == 0 {
"0".normal() String::new()
} else { } else {
es_next.panic.to_string().red() format!("({})", format!("{} panics", es_next.panic).red())
}, },
if es_next.panic == 0 { "" } else { " ⚠" }.red(),
(es_next.passed as f64 / es_next.total as f64) * 100.0 (es_next.passed as f64 / es_next.total as f64) * 100.0
); );
} }

4
boa_tester/src/main.rs

@ -343,9 +343,9 @@ fn run_test_suite(
println!("Passed tests: {}", passed.to_string().green()); println!("Passed tests: {}", passed.to_string().green());
println!("Ignored tests: {}", ignored.to_string().yellow()); println!("Ignored tests: {}", ignored.to_string().yellow());
println!( println!(
"Failed tests: {} (panics: {})", "Failed tests: {} ({})",
(total - passed - ignored).to_string().red(), (total - passed - ignored).to_string().red(),
panic.to_string().red() format!("{panic} panics").red()
); );
println!( println!(
"Conformance: {:.2}%", "Conformance: {:.2}%",

Loading…
Cancel
Save