From caad4748cffa50d6c585667efbe3ba4668f06d25 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):