style: apply ruff formatting to tests/ and add format check to bin/test.sh

Format 25 test files that were missed in the initial formatting pass.
Also add `ruff format --check` alongside `ruff check` in bin/test.sh
so CI catches both lint and formatting violations.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: tmgbedu

bin/test.sh

@@ -7,6 +7,7 @@ ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
 echo "============================================================"
 echo " Running: ruff lint checks"
 echo "============================================================"
+(cd "$ROOT/fastapi_startkit" && uv run ruff format --check src/ tests/)
 (cd "$ROOT/fastapi_startkit" && uv run ruff check src/ tests/)
 
 # ── Start MySQL via Docker Compose ────────────────────────────────────────────

fastapi_startkit/tests/conftest.py

@@ -4,4 +4,4 @@
 
 @pytest.fixture(scope="session", autouse=True)
 def init_app():
-    Application(env="testing")
+    Application(env="testing")

fastapi_startkit/tests/inertia/test_inertia.py

@@ -1,6 +1,7 @@
 import unittest
 from fastapi_startkit.inertia.inertia import Inertia, ResponseFactory, InertiaResponse
 
+
 class TestInertia(unittest.TestCase):
     def setUp(self):
         # Reset the singleton instance before each test
@@ -31,7 +32,7 @@ def test_factory_render_returns_response(self):
         factory = ResponseFactory()
         factory.share("auth", {"user": None})
         response = factory.render("Dashboard", {"count": 10})
-        
+
         self.assertIsInstance(response, InertiaResponse)
         self.assertEqual(response.component, "Dashboard")
         self.assertEqual(response.props, {"count": 10})
@@ -45,9 +46,9 @@ def test_facade_singleton(self):
     def test_facade_proxies_to_instance(self):
         Inertia.share("foo", "bar")
         self.assertEqual(Inertia.instance().shared_props["foo"], "bar")
-        
+
         Inertia.version("v1")
         self.assertEqual(Inertia.get_version(), "v1")
-        
+
         Inertia.set_root_view("app.html")
         self.assertEqual(Inertia.instance().root_view, "app.html")

fastapi_startkit/tests/inertia/test_inertia_response.py

@@ -5,6 +5,7 @@
 from fastapi_startkit.inertia.inertia import InertiaResponse, OptionalProp
 from fastapi_startkit.inertia.constant import Header
 
+
 class TestInertiaResponse(unittest.IsolatedAsyncioTestCase):
     def setUp(self):
         self.mock_request = MagicMock(spec=Request)
@@ -13,19 +14,16 @@ def setUp(self):
 
     async def test_inertia_response_to_json_on_inertia_request(self):
         self.mock_request.headers = {Header.INERTIA: "true"}
-        
+
         response = InertiaResponse(
-            component="User/Index",
-            shared_props={"app": "Test"},
-            props={"users": []},
-            version="v1"
+            component="User/Index", shared_props={"app": "Test"}, props={"users": []}, version="v1"
         )
-        
+
         actual_response = await response.to_response(self.mock_request)
-        
+
         self.assertEqual(actual_response.status_code, 200)
         self.assertEqual(actual_response.headers[Header.INERTIA], "true")
-        
+
         content = json.loads(actual_response.body)
         self.assertEqual(content["component"], "User/Index")
         self.assertEqual(content["props"], {"app": "Test", "users": []})
@@ -36,18 +34,18 @@ async def test_inertia_response_partial_reload(self):
         self.mock_request.headers = {
             Header.INERTIA: "true",
             Header.INERTIA_PARTIAL_COMPONENT: "User/Index",
-            "X-Inertia-Partial-Data": "users"
+            "X-Inertia-Partial-Data": "users",
         }
-        
+
         response = InertiaResponse(
             component="User/Index",
             shared_props={"app": "Test"},
             props={"users": ["user1"], "stats": {"likes": 10}},
         )
-        
+
         actual_response = await response.to_response(self.mock_request)
         data = json.loads(actual_response.body)
-        
+
         # Should only include "users", exclude "app" and "stats"
         self.assertIn("users", data["props"])
         self.assertNotIn("app", data["props"])
@@ -56,8 +54,9 @@ async def test_inertia_response_partial_reload(self):
     async def test_inertia_response_optional_props(self):
         # 1. Normal request - optional prop should be excluded
         self.mock_request.headers = {Header.INERTIA: "true"}
-        
+
         lazy_called = False
+
         def get_lazy():
             nonlocal lazy_called
             lazy_called = True
@@ -68,7 +67,7 @@ def get_lazy():
             shared_props={},
             props={"regular": "data", "lazy": OptionalProp(get_lazy)},
         )
-        
+
         actual_response = await response.to_response(self.mock_request)
         data = json.loads(actual_response.body)
         self.assertEqual(data["props"], {"regular": "data"})
@@ -78,17 +77,17 @@ def get_lazy():
         self.mock_request.headers = {
             Header.INERTIA: "true",
             Header.INERTIA_PARTIAL_COMPONENT: "User/Index",
-            "X-Inertia-Partial-Data": "lazy"
+            "X-Inertia-Partial-Data": "lazy",
         }
-        
+
         actual_response = await response.to_response(self.mock_request)
         data = json.loads(actual_response.body)
         self.assertEqual(data["props"], {"lazy": "lazy data"})
         self.assertTrue(lazy_called)
 
     async def test_inertia_response_resolves_callable_props(self):
         self.mock_request.headers = {Header.INERTIA: "true"}
-        
+
         async def get_async_data():
             return "async result"
 
@@ -97,7 +96,7 @@ async def get_async_data():
             shared_props={"sync": lambda: "sync result"},
             props={"async": get_async_data},
         )
-        
+
         actual_response = await response.to_response(self.mock_request)
         data = json.loads(actual_response.body)
         self.assertEqual(data["props"]["sync"], "sync result")
@@ -106,9 +105,9 @@ async def get_async_data():
     async def test_inertia_response_initial_render_raises_if_no_templates(self):
         # Standard request (no X-Inertia header)
         self.mock_request.headers = {}
-        
+
         response = InertiaResponse(component="Test", shared_props={}, props={})
-        
+
         # This should fail because we haven't mocked the application container
         with self.assertRaisesRegex(RuntimeError, "Inertia requires 'templates' to be bound"):
             await response.to_response(self.mock_request)

fastapi_startkit/tests/inertia/test_middleware.py

@@ -6,28 +6,32 @@
 from fastapi_startkit.inertia.constant import Header
 from fastapi_startkit.inertia.inertia import Inertia
 
+
 class TestInertiaMiddleware(unittest.IsolatedAsyncioTestCase):
     def setUp(self):
         self.app = FastAPI()
         self.app.add_middleware(InertiaMiddleware)
-        
+
         @self.app.get("/test")
         async def test_route():
             return {"message": "ok"}
-        
+
         @self.app.post("/redirect")
         async def test_redirect():
             from fastapi.responses import RedirectResponse
+
             return RedirectResponse(url="/test", status_code=302)
 
         @self.app.put("/redirect-put")
         async def test_redirect_put():
             from fastapi.responses import RedirectResponse
+
             return RedirectResponse(url="/test", status_code=302)
 
         @self.app.get("/fragment-redirect")
         async def test_fragment_redirect():
             from fastapi.responses import RedirectResponse
+
             return RedirectResponse(url="/test#section", status_code=302)
 
         self.client = TestClient(self.app)
@@ -43,34 +47,31 @@ def test_middleware_version_conflict(self, mock_app_getter):
         # Setup mock container
         mock_container = MagicMock()
         mock_app_getter.return_value = mock_container
-        
+
         # Mock Vite version
         mock_vite = MagicMock()
         mock_vite.manifest_hash.return_value = "v2"
         mock_container.has.side_effect = lambda k: k == "vite"
         mock_container.make.side_effect = lambda k: mock_vite if k == "vite" else None
-        
+
         # Request with old version
-        response = self.client.get("/test", headers={
-            Header.INERTIA: "true",
-            Header.INERTIA_VERSION: "v1"
-        })
-        
+        response = self.client.get("/test", headers={Header.INERTIA: "true", Header.INERTIA_VERSION: "v1"})
+
         self.assertEqual(response.status_code, 409)
         self.assertEqual(response.headers[Header.INERTIA_LOCATION], "http://testserver/test")
 
     def test_middleware_changes_302_to_303_on_put_patch_delete(self):
         # POST stays 302
         response = self.client.post("/redirect", follow_redirects=False)
         self.assertEqual(response.status_code, 302)
-        
+
         # PUT changes to 303
         response = self.client.put("/redirect-put", follow_redirects=False, headers={Header.INERTIA: "true"})
         self.assertEqual(response.status_code, 303)
 
     def test_middleware_redirect_with_fragment(self):
         response = self.client.get("/fragment-redirect", headers={Header.INERTIA: "true"})
-        
+
         self.assertEqual(response.status_code, 409)
         self.assertEqual(response.headers[Header.INERTIA_REDIRECT], "/test#section")
 
@@ -79,16 +80,16 @@ def test_middleware_resolves_validation_errors_from_session(self, mock_app_gette
         # We need a fresh app and session-enabled middleware
         from fastapi import Request
         from starlette.middleware.sessions import SessionMiddleware
-        
+
         app = FastAPI()
         app.add_middleware(InertiaMiddleware)
         app.add_middleware(SessionMiddleware, secret_key="secret")
-        
+
         @app.get("/set-errors")
         def set_errors(request: Request):
             request.session["errors"] = {"email": "Required"}
             return "ok"
-        
+
         @app.get("/check-errors")
         def check_errors(request: Request):
             # Middleware should have shared the errors from the session
@@ -99,15 +100,15 @@ def check_errors(request: Request):
         mock_container = MagicMock()
         mock_app_getter.return_value = mock_container
         mock_container.has.return_value = False
-        
+
         client = TestClient(app)
-        
+
         # Reset Inertia singleton for this specific test
         Inertia._instance = None
 
         # 1. First request sets the errors in session
         client.get("/set-errors")
-        
+
         # 2. Second request should have errors shared by middleware
         response = client.get("/check-errors")
         self.assertEqual(response.json(), {"email": "Required"})

fastapi_startkit/tests/masoniteorm/collection/test_collection.py

@@ -240,9 +240,7 @@ def test_count(self):
         collection = Collection([1, 1, 2, 4])
         self.assertEqual(collection.count(), 4)
 
-        collection = Collection(
-            [{"name": "Corentin All", "age": 1}, {"name": "Corentin All", "age": 2}]
-        )
+        collection = Collection([{"name": "Corentin All", "age": 1}, {"name": "Corentin All", "age": 2}])
         self.assertEqual(collection.count(), 2)
 
     def test_chunk(self):
@@ -364,9 +362,7 @@ def test_reject(self):
         collection.reject(lambda x: x if x["age"] > 2 else None)
 
         self.assertEqual(
-            Collection(
-                [{"name": "Corentin All", "age": 3}, {"name": "Corentin All", "age": 4}]
-            ),
+            Collection([{"name": "Corentin All", "age": 3}, {"name": "Corentin All", "age": 4}]),
             collection.all(),
         )
 
@@ -507,9 +503,7 @@ def test_implode(self):
         result = collection.implode("-")
         self.assertEqual(result, "1-2-3-4")
 
-        collection = Collection(
-            [{"name": "Corentin"}, {"name": "Joe"}, {"name": "Marlysson"}]
-        )
+        collection = Collection([{"name": "Corentin"}, {"name": "Joe"}, {"name": "Marlysson"}])
         result = collection.implode(key="name")
         self.assertEqual(result, "Corentin,Joe,Marlysson")
 
@@ -524,9 +518,7 @@ def __eq__(self, other):
                 return self.code == other.code
 
         currencies = collection.map_into(Currency)
-        self.assertEqual(
-            currencies.all(), [Currency("USD"), Currency("EUR"), Currency("GBP")]
-        )
+        self.assertEqual(currencies.all(), [Currency("USD"), Currency("EUR"), Currency("GBP")])
 
     def test_map(self):
         collection = Collection([1, 2, 3, 4])

fastapi_startkit/tests/masoniteorm/commands/fixtures/app.py

@@ -12,15 +12,18 @@ def create_app() -> Application:
     return Application(
         base_path=BASE_DIR,
         providers=[
-            (DatabaseProvider, {
-                "default": "sqlite",
-                "connections": {
-                    "sqlite": SQLiteConfig(
-                        driver="sqlite",
-                        url=f"sqlite+aiosqlite:///{DB_PATH}",
-                        options=None,
-                    ),
-                }
-            }),
+            (
+                DatabaseProvider,
+                {
+                    "default": "sqlite",
+                    "connections": {
+                        "sqlite": SQLiteConfig(
+                            driver="sqlite",
+                            url=f"sqlite+aiosqlite:///{DB_PATH}",
+                            options=None,
+                        ),
+                    },
+                },
+            ),
         ],
     )

fastapi_startkit/tests/masoniteorm/commands/fixtures/databases/migrations/2026_01_01_000001_add_body_to_posts_table.py

@@ -8,4 +8,4 @@ async def up(self):
 
     async def down(self):
         async with await self.schema.table("posts") as table:
-            table.drop_column("body")
+            table.drop_column("body")

fastapi_startkit/tests/masoniteorm/commands/test_shell.py

@@ -63,10 +63,7 @@ def test_for_mssql(self):
             "full_details": {"driver": "mssql"},
         }
         command, _ = self.command.get_command(config)
-        assert (
-            command
-            == "sqlcmd -d orm -U root -P secretpostgres -S tcp:db.masonite.com,1234"
-        )
+        assert command == "sqlcmd -d orm -U root -P secretpostgres -S tcp:db.masonite.com,1234"
 
     @skip("ShellCommand.handle() uses legacy load_config() not available in new framework")
     def test_running_command_with_sqlite(self):
@@ -86,6 +83,4 @@ def test_hiding_sensitive_options(self):
         }
         command, _ = self.command.get_command(config)
         cleaned_command = self.command.hide_sensitive_options(config, command)
-        assert (
-            cleaned_command == "mysql orm --host localhost --user root --password ***"
-        )
+        assert cleaned_command == "mysql orm --host localhost --user root --password ***"

fastapi_startkit/tests/masoniteorm/configurations/test_config_merge.py

@@ -34,9 +34,7 @@ def test_merge_with_file_path(self):
         config.set("testkey", {"existing": "orig"})
 
         # Mock Loader to return params from file
-        with patch(
-            "fastapi_startkit.configuration.Configuration.Loader"
-        ) as MockLoaderClass:
+        with patch("fastapi_startkit.configuration.Configuration.Loader") as MockLoaderClass:
             mock_loader = MockLoaderClass.return_value
             mock_loader.get_parameters.return_value = {
                 "New": "from_file",