style(tests): apply consistent formatting and spacing across test files

Author: marcuxyz

tests/conftest.py

@@ -1,6 +1,7 @@
 """
 Comprehensive test configuration with improved fixtures and utilities.
 """
+
 import pytest
 import tempfile
 import os
@@ -14,15 +15,17 @@
 def app():
     """Create application for the tests with session scope."""
     app = create_app()
-    app.config.update({
-        "TESTING": True,
-        "SQLALCHEMY_DATABASE_URI": "sqlite:///:memory:",
-        "WTF_CSRF_ENABLED": False,
-        "SECRET_KEY": "test-secret-key",
-        "SERVER_NAME": "localhost.localdomain",
-        "APPLICATION_ROOT": "/",
-        "PREFERRED_URL_SCHEME": "http"
-    })
+    app.config.update(
+        {
+            "TESTING": True,
+            "SQLALCHEMY_DATABASE_URI": "sqlite:///:memory:",
+            "WTF_CSRF_ENABLED": False,
+            "SECRET_KEY": "test-secret-key",
+            "SERVER_NAME": "localhost.localdomain",
+            "APPLICATION_ROOT": "/",
+            "PREFERRED_URL_SCHEME": "http",
+        }
+    )
 
     return app
 
@@ -116,32 +119,37 @@ def sample_messages(app):
 def db_helper():
     """Database helper for tests."""
     from tests.test_utils import DatabaseHelper
+
     return DatabaseHelper()
 
 
 @pytest.fixture
 def response_helper():
     """Response helper for tests."""
     from tests.test_utils import ResponseHelper
+
     return ResponseHelper()
 
 
 @pytest.fixture
 def route_helper():
     """Route helper for tests."""
     from tests.test_utils import RouteHelper
+
     return RouteHelper()
 
 
 @pytest.fixture
 def validation_helper():
     """Validation helper for tests."""
     from tests.test_utils import ValidationHelper
+
     return ValidationHelper()
 
 
 @pytest.fixture
 def benchmark_helper():
     """Benchmark helper for tests."""
     from tests.test_utils import BenchmarkHelper
+
     return BenchmarkHelper()

tests/test_callbacks_functional.py

@@ -1,6 +1,7 @@
 """
 Comprehensive tests for MVC Flask callback middleware system.
 """
+
 import pytest
 from flask import url_for, request, Response
 
@@ -10,6 +11,7 @@
 
 # Callback Middleware Tests
 
+
 def test_before_request_execution(client):
     """Test that before_request callback is executed."""
     response = client.get(url_for("callbacks.index"))
@@ -80,10 +82,12 @@ def test_hook_execution_with_matching_endpoint(app):
     with app.test_request_context("/callbacks"):
         # Mock the request to simulate endpoint matching
         from unittest.mock import patch
-        with patch('flask.request') as mock_request:
+
+        with patch("flask.request") as mock_request:
             mock_request.endpoint = "callbacks.index"
 
             executed = []
+
             def mock_hook():
                 executed.append("called")
 
@@ -99,10 +103,12 @@ def test_hook_execution_without_matching_endpoint(app):
     with app.test_request_context("/other"):
         # Mock the request to simulate endpoint not matching
         from unittest.mock import patch
-        with patch('flask.request') as mock_request:
+
+        with patch("flask.request") as mock_request:
             mock_request.endpoint = "other.index"
 
             executed = []
+
             def mock_hook():
                 executed.append("called")
 
@@ -112,22 +118,24 @@ def mock_hook():
 
 # Callback Configuration Tests
 
+
 def test_multiple_actions_before_request(app):
     """Test before_request callback with multiple actions."""
+
     class MultiActionController:
         before_request = dict(callback="setup", actions="index show edit")
 
         def setup(self):
             self.data = "initialized"
 
         def index(self):
-            return getattr(self, 'data', 'not initialized')
+            return getattr(self, "data", "not initialized")
 
         def show(self, id):
-            return getattr(self, 'data', 'not initialized')
+            return getattr(self, "data", "not initialized")
 
         def edit(self, id):
-            return getattr(self, 'data', 'not initialized')
+            return getattr(self, "data", "not initialized")
 
     controller = MultiActionController()
     middleware = CallbackMiddleware(app, "multi", controller)
@@ -138,6 +146,7 @@ def edit(self, id):
 
 def test_after_request_with_response_modification(app):
     """Test after_request callback modifying response."""
+
     class ResponseModifierController:
         after_request = dict(callback="modify_response", actions="index")
 
@@ -154,7 +163,8 @@ def modify_response(self, response):
     with app.test_request_context("/modifier"):
         # Mock the request to simulate endpoint matching
         from unittest.mock import patch
-        with patch('flask.request') as mock_request:
+
+        with patch("flask.request") as mock_request:
             mock_request.endpoint = "modifier.index"
 
             response = Response("test")
@@ -167,8 +177,10 @@ def modify_response(self, response):
 
 # Edge Cases Tests
 
+
 def test_callback_without_actions(app):
     """Test callback configuration without actions."""
+
     class NoActionsController:
         before_request = dict(callback="setup", actions="")
 
@@ -184,6 +196,7 @@ def setup(self):
 
 def test_callback_with_invalid_method(app):
     """Test callback configuration with invalid method."""
+
     class InvalidMethodController:
         before_request = dict(callback="nonexistent_method", actions="index")
 
@@ -202,6 +215,7 @@ def setup(self):
 
 def test_empty_controller(app):
     """Test middleware with controller that has no callbacks."""
+
     class EmptyController:
         def index(self):
             return "empty"
@@ -216,6 +230,7 @@ def index(self):
 
 def test_malformed_callback_config(app):
     """Test malformed callback configuration."""
+
     class MalformedController:
         before_request = "not a dict"
 
@@ -228,6 +243,7 @@ class MalformedController:
 
 # Integration Tests
 
+
 def test_full_request_cycle_with_callbacks(client):
     """Test complete request cycle with both before and after callbacks."""
     response = client.get(url_for("callbacks.index"))

tests/test_controllers_functional.py

@@ -1,6 +1,7 @@
 """
 Comprehensive tests for MVC Flask controllers functionality.
 """
+
 import pytest
 import json
 from flask import url_for
@@ -11,6 +12,7 @@
 
 # CRUD Operations Tests
 
+
 def test_index_with_messages(client, sample_messages):
     """Test index action displays messages correctly."""
     response = client.get(url_for("messages.index"))
@@ -55,9 +57,7 @@ def test_create_with_json(empty_client):
     headers = {"Content-Type": "application/json"}
 
     response = empty_client.post(
-        url_for("messages.create"),
-        data=json.dumps(data),
-        headers=headers
+        url_for("messages.create"), data=json.dumps(data), headers=headers
     )
 
     assert response.status_code == 201
@@ -73,9 +73,7 @@ def test_create_without_data(empty_client):
 
     with pytest.raises(KeyError):
         empty_client.post(
-            url_for("messages.create"),
-            data=json.dumps({}),
-            headers=headers
+            url_for("messages.create"), data=json.dumps({}), headers=headers
         )
 
 
@@ -96,9 +94,7 @@ def test_update_with_json(client):
     headers = {"Content-Type": "application/json"}
 
     response = client.put(
-        url_for("messages.update", id=message.id),
-        data=json.dumps(data),
-        headers=headers
+        url_for("messages.update", id=message.id), data=json.dumps(data), headers=headers
     )
 
     assert response.status_code == 200
@@ -116,7 +112,7 @@ def test_update_with_form_data(client):
     response = client.put(
         url_for("messages.update", id=message.id),
         data={"title": "Form Updated Title"},
-        headers={"Content-Type": "application/x-www-form-urlencoded"}
+        headers={"Content-Type": "application/x-www-form-urlencoded"},
     )
 
     assert response.status_code == 200
@@ -132,9 +128,7 @@ def test_update_nonexistent_message(client):
 
     try:
         response = client.put(
-            url_for("messages.update", id=99999),
-            data=json.dumps(data),
-            headers=headers
+            url_for("messages.update", id=99999), data=json.dumps(data), headers=headers
         )
         assert response.status_code in [200, 404, 500]
     except Exception:
@@ -166,6 +160,7 @@ def test_delete_nonexistent_message(empty_client):
 
 # Controller Helper Tests
 
+
 def test_multiple_messages_handling(empty_client):
     """Test controller behavior with multiple messages."""
     messages = []
@@ -185,14 +180,13 @@ def test_multiple_messages_handling(empty_client):
 
 # Error Handling Tests
 
+
 def test_malformed_json_in_create(empty_client):
     """Test create action with malformed JSON."""
     headers = {"Content-Type": "application/json"}
 
     response = empty_client.post(
-        url_for("messages.create"),
-        data="invalid json",
-        headers=headers
+        url_for("messages.create"), data="invalid json", headers=headers
     )
 
     assert response.status_code in [400, 500]
@@ -205,7 +199,7 @@ def test_missing_content_type_in_update(client):
     try:
         response = client.put(
             url_for("messages.update", id=message.id),
-            data=json.dumps({"title": "Updated"})
+            data=json.dumps({"title": "Updated"}),
         )
         assert response.status_code in [200, 400, 500]
     except KeyError:
@@ -218,22 +212,21 @@ def test_empty_title_in_create(empty_client):
     headers = {"Content-Type": "application/json"}
 
     response = empty_client.post(
-        url_for("messages.create"),
-        data=json.dumps(data),
-        headers=headers
+        url_for("messages.create"), data=json.dumps(data), headers=headers
     )
 
     assert response.status_code == 201
 
 
 # Integration Tests
 
+
 def test_full_crud_workflow(empty_client):
     """Test complete CRUD workflow."""
     create_response = empty_client.post(
         url_for("messages.create"),
         data=json.dumps({"title": "Workflow Test Message"}),
-        headers={"Content-Type": "application/json"}
+        headers={"Content-Type": "application/json"},
     )
     assert create_response.status_code == 201
 
@@ -249,7 +242,7 @@ def test_full_crud_workflow(empty_client):
     update_response = empty_client.put(
         url_for("messages.update", id=message.id),
         data=json.dumps({"title": "Updated Workflow Message"}),
-        headers={"Content-Type": "application/json"}
+        headers={"Content-Type": "application/json"},
     )
     assert update_response.status_code == 200
 
@@ -262,17 +255,14 @@ def test_full_crud_workflow(empty_client):
 
 def test_concurrent_operations(empty_client):
     """Test concurrent operations on messages."""
-    messages_data = [
-        {"title": f"Concurrent Message {i}"}
-        for i in range(5)
-    ]
+    messages_data = [{"title": f"Concurrent Message {i}"} for i in range(5)]
 
     created_messages = []
     for data in messages_data:
         response = empty_client.post(
             url_for("messages.create"),
             data=json.dumps(data),
-            headers={"Content-Type": "application/json"}
+            headers={"Content-Type": "application/json"},
         )
         assert response.status_code == 201
         created_messages.append(response.json)