Ensure env-vars are handled as strings

This avoids suprocess.run to fail ambiguously due to non-string values inherited from configuration
(like postgresql' port number). Refs #83.

septentrion/runner.py

@@ -38,7 +38,7 @@ def _env(self):
             "PGUSER": self.settings.USERNAME,
             "PGPASSWORD": self.settings.PASSWORD,
         }
-        return {key: value for key, value in environment.items() if value}
+        return {key: str(value) for key, value in environment.items() if value}
 
     def _run_simple(self):
 

tests/integration/test_runner.py

@@ -58,6 +58,18 @@ def test_run_psql_not_found(run_script, env):
     )
 
 
+def test_run_integer_in_settings(db, settings_factory, env, tmp_path):
+    settings = settings_factory(**db)
+    # settings are noew stringified to prevent subprocess.run
+    # crashing while building the appropriate command line.
+    settings.PORT = 5432
+    path = tmp_path / "script.sql"
+    path.write_text("SELECT 1;")
+    with io.open(path, "r", encoding="utf8") as f:
+        script = Script(settings, f, path)
+        script.run()
+
+
 def test_run_with_meta_loop(db, settings_factory, run_script):
     settings = settings_factory(**db)