Skip to content

Commit 2a7e80e

Browse files
committed
tests/run-tests.py: Use TEST_TIMEOUT as timeout for bare-metal tests.
This parameter is already used for PC-based tests (eg unix and webassembly ports), and it makes sense for it to be used for bare-metal ports as well. That way the timeout is configurable for all targets. Because this increases the default timeout from 10s to 30s, this fixes some long-running tests that would previously fail due to a timeout such as `thread/stress_aes.py` on ESP32. Signed-off-by: Damien George <damien@micropython.org>
1 parent 0af9908 commit 2a7e80e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/run-tests.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
import threading
1616
import tempfile
1717

18-
# Maximum time to run a PC-based test, in seconds.
18+
# Maximum time to run a single test, in seconds.
1919
TEST_TIMEOUT = float(os.environ.get('MICROPY_TEST_TIMEOUT', 30))
2020

2121
# See stackoverflow.com/questions/2632199: __file__ nor sys.argv[0]
@@ -328,7 +328,7 @@ def run_script_on_remote_target(pyb, args, test_file, is_special):
328328
try:
329329
had_crash = False
330330
pyb.enter_raw_repl()
331-
output_mupy = pyb.exec_(script)
331+
output_mupy = pyb.exec_(script, timeout=TEST_TIMEOUT)
332332
except pyboard.PyboardError as e:
333333
had_crash = True
334334
if not is_special and e.args[0] == "exception":

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy