[bfops/fix-list-tests]: Fix running smoketests --list

Author: bfops

smoketests/__main__.py

@@ -79,6 +79,39 @@ def main():
     parser.add_argument("--spacetime-login", action="store_true", help="Use `spacetime login` for these tests (and disable tests that don't work with that)")
     args = parser.parse_args()
 
+    if args.docker:
+        # have docker logs print concurrently with the test output
+        if args.compose_file:
+            smoketests.COMPOSE_FILE = args.compose_file
+        if not args.no_docker_logs:
+            if args.compose_file:
+                subprocess.Popen(["docker", "compose", "-f", args.compose_file, "logs", "-f"])
+            else:
+                docker_container = check_docker()
+                subprocess.Popen(["docker", "logs", "-f", docker_container])
+        smoketests.HAVE_DOCKER = True
+
+    if not args.skip_dotnet:
+        smoketests.HAVE_DOTNET = check_dotnet()
+        if not smoketests.HAVE_DOTNET:
+            print("no suitable dotnet installation found")
+            exit(1)
+
+    add_prefix = lambda testlist: [TESTPREFIX + test for test in testlist]
+    import fnmatch
+    excludelist = add_prefix(args.exclude)
+    testlist = add_prefix(args.test)
+
+    loader = ExclusionaryTestLoader(excludelist)
+    loader.testNamePatterns = args.testNamePatterns
+
+    tests = loader.loadTestsFromNames(testlist)
+    if args.list:
+        print("Selected tests:\n")
+        for test in itertools.chain(*itertools.chain(*tests)):
+            print(f"{test}")
+        exit(0)
+
     if not args.no_build_cli:
         logging.info("Compiling spacetime cli...")
         smoketests.run_cmd("cargo", "build", cwd=TEST_DIR.parent, capture_stderr=False)
@@ -100,18 +133,6 @@ def main():
 
     os.environ["SPACETIME_SKIP_CLIPPY"] = "1"
 
-    if args.docker:
-        # have docker logs print concurrently with the test output
-        if args.compose_file:
-            smoketests.COMPOSE_FILE = args.compose_file
-        if not args.no_docker_logs:
-            if args.compose_file:
-                subprocess.Popen(["docker", "compose", "-f", args.compose_file, "logs", "-f"])
-            else:
-                docker_container = check_docker()
-                subprocess.Popen(["docker", "logs", "-f", docker_container])
-        smoketests.HAVE_DOCKER = True
-
     with tempfile.NamedTemporaryFile(mode="w+b", suffix=".toml", buffering=0, delete_on_close=False) as config_file:
         with BASE_STDB_CONFIG_PATH.open("rb") as src, config_file.file as dst:
             shutil.copyfileobj(src, dst)
@@ -130,28 +151,6 @@ def main():
         smoketests.STDB_CONFIG = Path(config_file.name).read_text()
 
     build_template_target()
-
-    if not args.skip_dotnet:
-        smoketests.HAVE_DOTNET = check_dotnet()
-        if not smoketests.HAVE_DOTNET:
-            print("no suitable dotnet installation found")
-            exit(1)
-
-    add_prefix = lambda testlist: [TESTPREFIX + test for test in testlist]
-    import fnmatch
-    excludelist = add_prefix(args.exclude)
-    testlist = add_prefix(args.test)
-
-    loader = ExclusionaryTestLoader(excludelist)
-    loader.testNamePatterns = args.testNamePatterns
-
-    tests = loader.loadTestsFromNames(testlist)
-    if args.list:
-        print("Selected tests:\n")
-        for test in itertools.chain(*itertools.chain(*tests)):
-            print(f"{test}")
-        exit(0)
-
     buffer = not args.show_all_output
     verbosity = 2