summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLuca Casonato <lucacasonato@yahoo.com>2021-01-13 02:14:17 +0100
committerGitHub <noreply@github.com>2021-01-12 17:14:17 -0800
commit1728b3ba1972faf532f261d2a7cfd6130d0f548f (patch)
treec167d5906b4f5e9df599679ec4076453afe524f8
parentef0bbbbeaf1d5d701da7850deaa1d02b7dcfb6aa (diff)
tests: disable logging of 'ok' web platform tests (#9087)
-rw-r--r--cli/tests/integration_tests.rs1
-rw-r--r--cli/tests/wpt_testharnessconsolereporter.js19
2 files changed, 12 insertions, 8 deletions
diff --git a/cli/tests/integration_tests.rs b/cli/tests/integration_tests.rs
index 35e770c47..5067b87eb 100644
--- a/cli/tests/integration_tests.rs
+++ b/cli/tests/integration_tests.rs
@@ -5329,6 +5329,7 @@ fn web_platform_tests() {
.arg("-A")
.arg(file.path())
.arg(deno_core::serde_json::to_string(&expect_fail).unwrap())
+ .arg("--quiet")
.stdin(std::process::Stdio::piped())
.spawn()
.unwrap();
diff --git a/cli/tests/wpt_testharnessconsolereporter.js b/cli/tests/wpt_testharnessconsolereporter.js
index 5c217f29e..2e0e06c02 100644
--- a/cli/tests/wpt_testharnessconsolereporter.js
+++ b/cli/tests/wpt_testharnessconsolereporter.js
@@ -39,6 +39,13 @@ function shouldExpectFail(name) {
window.add_result_callback(({ message, name, stack, status }) => {
const expectFail = shouldExpectFail(name);
+ testResults.push({
+ name,
+ passed: status === 0,
+ expectFail,
+ message,
+ stack,
+ });
let simpleMessage = `test ${name} ... `;
switch (status) {
case 0:
@@ -46,6 +53,10 @@ window.add_result_callback(({ message, name, stack, status }) => {
simpleMessage += red("ok (expected fail)");
} else {
simpleMessage += green("ok");
+ if (Deno.args[1] == "--quiet") {
+ // don't print `ok` tests if --quiet is enabled
+ return;
+ }
}
break;
case 1:
@@ -72,14 +83,6 @@ window.add_result_callback(({ message, name, stack, status }) => {
}
console.log(simpleMessage);
-
- testResults.push({
- name,
- passed: status === 0,
- expectFail,
- message,
- stack,
- });
});
window.add_completion_callback((tests, harnessStatus) => {