From 9d23023cd9438b7c5a22cb3c5225b3239b822b4c Mon Sep 17 00:00:00 2001 From: Joyee Cheung Date: Mon, 12 Sep 2022 05:41:14 +0800 Subject: [PATCH] tools: increase timeout of running WPT MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit We run all WPT from one subset in the same process using workers. As the number of the tests grow, it can take longer to run some of the subsets, but it's still overall faster than running them in different processes. This patch increases the timeout for WPT to prevent the test from failing because it takes longer to run (even though it would still complete at some point). PR-URL: https://github.com/nodejs/node/pull/44574 Refs: https://github.com/nodejs/reliability/issues/371 Reviewed-By: Tobias Nießen Reviewed-By: Filip Skokan Reviewed-By: Michaël Zasso Reviewed-By: Luigi Pinca --- tools/test.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tools/test.py b/tools/test.py index 5ae7a461cc4c6a..788d68734fe250 100755 --- a/tools/test.py +++ b/tools/test.py @@ -957,8 +957,14 @@ def GetVm(self, arch, mode): def GetTimeout(self, mode, section=''): timeout = self.timeout * TIMEOUT_SCALEFACTOR[ARCH_GUESS or 'ia32'][mode] - if section == 'pummel' or section == 'benchmark' or section == 'wpt': + if section == 'pummel' or section == 'benchmark': timeout = timeout * 6 + # We run all WPT from one subset in the same process using workers. + # As the number of the tests grow, it can take longer to run some of the + # subsets, but it's still overall faster than running them in different + # processes. + elif section == 'wpt': + timeout = timeout * 12 return timeout def RunTestCases(cases_to_run, progress, tasks, flaky_tests_mode, measure_flakiness):