Avoid hanging in tests if timeout doesn't work.

pekkaklarck · pekkaklarck · commit 091cf8fe7cfb · 2020-01-03T12:15:36.000+02:00
diff --git a/atest/testdata/running/timeouts_with_logging.py b/atest/testdata/running/timeouts_with_logging.py
@@ -8,12 +8,21 @@
 
 
 def rf_logger():
-    while True:
-        logger.info(MSG)
-        time.sleep(0)    # give time for other threads
+    _log_a_lot(logger.info)
 
 
 def python_logging():
-    while True:
-        logging.info(MSG)
-        time.sleep(0)    # give time for other threads
+    _log_a_lot(logging.info)
+
+
+def _log_a_lot(info):
+    # Assigning local variables is performance optimization to give as much
+    # time as as possible for actual logging.
+    msg = MSG
+    sleep = time.sleep
+    current = time.time
+    end = current() + 1
+    while current() < end:
+        info(msg)
+        sleep(0)    # give time for other threads
+    raise AssertionError('Execution should have been stopped by timeout.')
