Fix config generation

Author: coolreader18

smoketests/__init__.py

@@ -237,7 +237,6 @@ def log_records(self, n):
         return list(map(json.loads, logs.splitlines()))
 
     def publish_module(self, domain=None, *, clear=True, capture_stderr=True, num_replicas=None, break_clients=False):
-        print("publishing module", self.publish_module)
         publish_output = self.spacetime(
             "publish",
             *[domain] if domain is not None else [],
@@ -256,6 +255,8 @@ def publish_module(self, domain=None, *, clear=True, capture_stderr=True, num_re
 
     @classmethod
     def reset_config(cls):
+        if not STDB_CONFIG:
+            raise Exception("config toml has not been initialized yet")
         cls.config_path.write_text(STDB_CONFIG)
 
     def fingerprint(self):

smoketests/__main__.py

@@ -13,6 +13,8 @@
 import logging
 import itertools
 import tempfile
+from pathlib import Path
+import shutil
 
 def check_docker():
     docker_ps = smoketests.run_cmd("docker", "ps", "--format=json")
@@ -94,13 +96,10 @@ def main():
         try:
             os.symlink(update_bin_name, SPACETIME_BIN)
         except OSError:
-            import shutil
             shutil.copyfile(SPACETIME_BIN.with_name(update_bin_name), SPACETIME_BIN)
 
     os.environ["SPACETIME_SKIP_CLIPPY"] = "1"
 
-    build_template_target()
-
     if args.docker:
         # have docker logs print concurrently with the test output
         if args.compose_file:
@@ -113,9 +112,9 @@ def main():
                 subprocess.Popen(["docker", "logs", "-f", docker_container])
         smoketests.HAVE_DOCKER = True
 
-    with tempfile.NamedTemporaryFile(mode="w+b", suffix=".toml", buffering=0) as config_file:
-        config_file.write(BASE_STDB_CONFIG_PATH.read_bytes())
-        config_file.flush()
+    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)
 
         if args.remote_server is not None:
             smoketests.spacetime("--config-path", config_file.name, "server", "edit", "localhost", "--url", args.remote_server, "--yes")
@@ -128,8 +127,9 @@ def main():
         else:
             smoketests.new_identity(config_file.name)
 
-        config_file.seek(0)
-        smoketests.STDB_CONFIG = config_file.read().decode()
+        smoketests.STDB_CONFIG = Path(config_file.name).read_text()
+
+    build_template_target()
 
     if not args.skip_dotnet:
         smoketests.HAVE_DOTNET = check_dotnet()