refactor test: Cleaner combine_logs.py logic

hodlinator · Dec 5, 2024 · 213e59d · 213e59d
1 parent 11f68cc
commit 213e59d
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/test/functional/combine_logs.py b/test/functional/combine_logs.py
@@ -81,13 +81,14 @@ def read_logs(tmp_dir):
 
     # Find out what the folder is called that holds node 0's debug.log file
     debug_logs = list(pathlib.Path(tmp_dir).glob('node0/**/debug.log'))
-    if len(debug_logs) > 0:
-        assert len(debug_logs) < 2, 'Max one debug.log is supported, ' \
-            'found several:\n\t' + '\n\t'.join([str(f) for f in debug_logs])
-        path = debug_logs[0]
-        chain = re.search(r'node0/(.+?)/debug\.log$', path.as_posix()).group(1)  # extract the chain name
-    else:
-        chain = 'regtest'  # fallback to regtest (should only happen when none exists)
+    match len(debug_logs):
+        case 0:
+            chain = 'regtest' # fallback to regtest
+        case 1:
+            chain = re.search(r'node0/(.+?)/debug\.log$', debug_logs[0].as_posix()).group(1)
+        case _:
+            raise RuntimeError('Max one debug.log is supported, found several:\n\t' +
+                               '\n\t'.join([str(f) for f in debug_logs]))
 
     files = [("test", "%s/test_framework.log" % tmp_dir)]
     for i in itertools.count():