functional_tests: rewrite mining code, clearer timeout info

tests/functional_tests/mining.py

@@ -95,20 +95,19 @@ class MiningTest():
         assert res_status.block_reward >= 600000000000
 
         # wait till we mined a few of them
+        target_height = prev_height + 5
+        height = prev_height
         timeout = 60 # randomx is slow to init
-        timeout_height = prev_height
+        while height < target_height:
-        while True:
+            seen_height = height
-            time.sleep(1)
+            for _ in range(timeout):
-            res_info = daemon.get_info()
+                time.sleep(1)
-            height = res_info.height
+                height = daemon.get_info().height
-            if height >= prev_height + 5:
+                if height > seen_height:
-                break
+                    break
-            if height > timeout_height:
-              timeout = 5
-              timeout_height = height
             else:
-              timeout -= 1
+                assert False, 'Failed to mine successor to block %d (initial block = %d)' % (seen_height, prev_height)
-            assert timeout >= 0
+            timeout = 5
 
         if via_daemon:
             res = daemon.stop_mining()