Improve coverage

Author: mattmess1221

tests/assets/projects/bad_configured_app/app.py

@@ -0,0 +1,8 @@
+from fastapi import FastAPI
+
+app = FastAPI()
+
+
+@app.get("/")
+def app_root():
+    return {"message": "badly configured app"}

tests/assets/projects/bad_configured_app/pyproject.toml

@@ -0,0 +1,2 @@
+[tool.fastapi.cli.run]
+port = "http"

tests/test_cli.py

@@ -210,6 +210,16 @@ def test_run_error() -> None:
         assert "Path does not exist non_existing_file.py" in result.output
 
 
+def test_project_config_error() -> None:
+    with changing_dir(assets_path / "projects/bad_configured_app"):
+        result = runner.invoke(app, ["run"])
+        assert result.exit_code == 2, result.output
+        assert (
+            "Error parsing pyproject.toml: key 'tool.fastapi.cli.run.port'"
+            in result.output
+        )
+
+
 def test_dev_help() -> None:
     result = runner.invoke(app, ["dev", "--help"])
     assert result.exit_code == 0, result.output

tests/utils.py

@@ -19,9 +19,9 @@ def changing_dir(directory: Union[str, Path]) -> Generator[None, None, None]:
 
 
 @contextmanager
-def importing(names: Union[str, list[str]]) -> Generator[None, None, None]:
+def importing(names: list[str]) -> Generator[None, None, None]:
     for name in names:
-        if name in sys.modules:
+        if name in sys.modules:  # pragma: no cover
             warnings.warn(
                 f"{name} is already imported",
                 category=UserWarning,
@@ -30,8 +30,6 @@ def importing(names: Union[str, list[str]]) -> Generator[None, None, None]:
     try:
         yield
     finally:
-        if isinstance(names, str):
-            names = [names]
         for name in names:
             if name in sys.modules:
                 del sys.modules[name]