fixed tests

Author: rasputyashka

src/fastapi_cli/cli.py

@@ -63,7 +63,9 @@ def _run(
     is_factory: bool = False,
 ) -> None:
     try:
-        use_uvicorn_app = get_import_string(path=path, app_name=app, is_factory=is_factory)
+        use_uvicorn_app = get_import_string(
+            path=path, app_name=app, is_factory=is_factory
+        )
     except FastAPICLIException as e:
         logger.error(str(e))
         raise typer.Exit(code=1) from None
@@ -85,6 +87,7 @@ def _run(
             padding=(1, 2),
             style="green",
         )
+    print(Padding(panel, 1))
     if not uvicorn:
         raise FastAPICLIException(
             "Could not import Uvicorn, try running 'pip install uvicorn'"
@@ -148,7 +151,9 @@ def dev(
     ] = True,
     factory: Annotated[
         bool,
-        typer.Option(help="Treat [bold]path[bold] as an application factory, i.e. a () -> <ASGI app> callable.")
+        typer.Option(
+            help="Treat [bold]path[/bold] as an application factory, i.e. a () -> <ASGI app> callable."
+        ),
     ] = False,
 ) -> Any:
     """
@@ -242,7 +247,9 @@ def run(
     ] = True,
     factory: Annotated[
         bool,
-        typer.Option(help="Treat [bold]path[bold] as an application factory, i.e. a () -> <ASGI app> callable.")
+        typer.Option(
+            help="Treat [bold]path[/bold] as an application factory, i.e. a () -> <ASGI app> callable."
+        ),
     ] = False,
 ) -> Any:
     """
@@ -280,7 +287,7 @@ def run(
         app=app,
         command="run",
         proxy_headers=proxy_headers,
-        is_factory=factory
+        is_factory=factory,
     )
 
 

tests/test_cli.py

@@ -29,6 +29,7 @@ def test_dev() -> None:
                 "workers": None,
                 "root_path": "",
                 "proxy_headers": True,
+                "factory": False,
             }
         assert "Using import string single_file_app:app" in result.output
         assert (
@@ -71,6 +72,7 @@ def test_dev_args() -> None:
                 "workers": None,
                 "root_path": "/api",
                 "proxy_headers": False,
+                "factory": False,
             }
         assert "Using import string single_file_app:api" in result.output
         assert (
@@ -97,6 +99,7 @@ def test_run() -> None:
                 "workers": None,
                 "root_path": "",
                 "proxy_headers": True,
+                "factory": False,
             }
         assert "Using import string single_file_app:app" in result.output
         assert (
@@ -141,6 +144,7 @@ def test_run_args() -> None:
                 "workers": 2,
                 "root_path": "/api",
                 "proxy_headers": False,
+                "factory": False,
             }
         assert "Using import string single_file_app:api" in result.output
         assert (

tests/test_utils_package.py

@@ -432,7 +432,8 @@ def test_package_dir_no_app() -> None:
         with pytest.raises(FastAPICLIException) as e:
             get_import_string(path=Path("package/core/utils.py"))
         assert (
-            "Could not find FastAPI app in module, try using --app" in e.value.args[0]
+            "Could not find FastAPI app or app factory in module, try using --app"
+            in e.value.args[0]
         )