Make argv default to an empty tuple in parse_config()

Makes it easier to use parse_config() from tests.

diff --git a/mkosi/__main__.py b/mkosi/__main__.py
index 9b3e1f44690e7dd4555bec10b439933aa481d650..8788e6b69810b3937e2d69ddf7772f3b82a10a73 100644 (file)
--- a/mkosi/__main__.py
+++ b/mkosi/__main__.py
@@ -42,7 +42,7 @@ def propagate_failed_return() -> Iterator[None]:
 @propagate_failed_return()
 def main() -> None:
     log_setup()
-    args, presets = parse_config()
+    args, presets = parse_config(sys.argv[1:])
 
     try:
         run_verb(args, presets)

diff --git a/mkosi/config.py b/mkosi/config.py
index 9b45dc0d9107ae4098ed02d674c79f50741d7cd2..4694b3be5928f1daeaa46073208a277bc41d3c03 100644 (file)
--- a/mkosi/config.py
+++ b/mkosi/config.py
@@ -16,7 +16,6 @@ import platform
 import shlex
 import shutil
 import subprocess
-import sys
 import textwrap
 import uuid
 from collections.abc import Iterable, Sequence
@@ -1842,7 +1841,7 @@ def resolve_deps(args: MkosiArgs, presets: Sequence[MkosiConfig]) -> list[MkosiC
     return sorted(presets, key=lambda p: order.index(p.preset))
 
 
-def parse_config(argv: Optional[Sequence[str]] = None) -> tuple[MkosiArgs, tuple[MkosiConfig, ...]]:
+def parse_config(argv: Sequence[str] = ()) -> tuple[MkosiArgs, tuple[MkosiConfig, ...]]:
     settings_lookup_by_name = {s.name: s for s in SETTINGS}
     settings_lookup_by_dest = {s.dest: s for s in SETTINGS}
     match_lookup = {m.name: m for m in MATCHES}
@@ -1964,8 +1963,6 @@ def parse_config(argv: Optional[Sequence[str]] = None) -> tuple[MkosiArgs, tuple
     namespace = argparse.Namespace()
     defaults = argparse.Namespace()
 
-    if argv is None:
-        argv = sys.argv[1:]
     argv = list(argv)
 
     # Make sure the verb command gets explicitly passed. Insert a -- before the positional verb argument