Refactor: Actualizar tests de chat y añadir tests de dashboard

Este commit introduce varias refactorizaciones y adiciones en las aplicaciones de chat, dashboard y viajes.

**Aplicación Chat:**

*   Refactorizados los tests en `accounts/tests/test_views.py`, `chat/tests/test_consumers.py`, `chat/tests/test_models.py`, `chat/tests/test_utils.py`, `chat/tests/test_views.py` y `rides/tests/test_integration.py`.
*   Actualizada la lógica de los tests para reflejar el cambio donde los objetos `Message` ahora se asocian directamente con un objeto `Chat` en lugar de un `Ride`. Los tests ahora obtienen y usan correctamente el objeto `chat` vinculado a un `ride` para la creación de mensajes y las comprobaciones de acceso.
    *   Corregido `KeyError: 'chat_id'` en tests de consumers usando `chat.id`.
    *   Corregido `IntegrityError` en tests de modelos asegurando la existencia del `Chat`.
    *   Corregido `SynchronousOnlyOperation` en tests de consumers usando `database_sync_to_async`.
    *   Corregido `AssertionError` en la conexión del pasajero asegurando que esté en los participantes del chat.
    *   Corregido `KeyError: 'content'` en tests de consumers accediendo a la estructura anidada del mensaje.
*   Ajustadas las rutas de los mocks en `accounts/tests/test_views.py` para coincidir con la ubicación refactorizada de `create_stripe_onboarding_link`.

**Aplicación Dashboard:**

*   Añadida una suite de tests completa para la aplicación `dashboard` en `dashboard/tests/`.
    *   Creado `__init__.py`.
    *   Añadido `test_constants.py`.
    *   Añadido `test_utils.py` con `DashboardUtilsTests` para probar funciones de utilidad y serialización de datos.
    *   Añadido `test_views.py` con `DashboardViewsTests` para probar vistas, control de acceso, uso de plantillas, datos de contexto y endpoints JSON API.
*   Refactorizado `dashboard/urls.py`:
    *   Renombrados varios patrones de URL de API por consistencia (ej., `api_trips` a `get_trip_data_json`).
    *   Añadidos patrones de URL que faltaban y eran requeridos por las vistas: `dashboard_data_api`, `get_chat_messages`, `report_detail` y `mark_report`.
*   Añadida la implementación de la función de vista `mark_report` que faltaba en `dashboard/views.py`, que maneja el marcado de reportes como leídos/no leídos.

**Aplicación Rides:**

*   Ajuste menor de formato en la vista `book_ride` (`rides/views.py`).

Estos cambios mejoran la robustez de la funcionalidad de chat alineando los tests con el modelo de datos actual, aumentan significativamente la cobertura de tests para la funcionalidad del dashboard y aseguran que las URLs y vistas del dashboard sean consistentes y completas.

Author: alejandroGM0

codigo/accounts/tests/test_views.py

@@ -1,6 +1,7 @@
 # ==========================================
 # Autor: Alejandro Gasca Mediel
 # ==========================================
+
 from django.test import TestCase, Client
 from django.contrib.auth.models import User
 from django.urls import reverse
@@ -286,21 +287,21 @@ def test_onboarding_redirects_if_no_account(self):
         messages = list(get_messages(response.wsgi_request))
         self.assertTrue(any("configurar tu cuenta de pagos" in str(m) for m in messages))
 
-    @patch('accounts._utils.create_stripe_onboarding_link')
+    @patch('accounts.views.create_stripe_onboarding_link')
     def test_onboarding_redirects_to_stripe(self, mock_link):
         """
         Prueba que la vista de onboarding redirige a Stripe con el enlace generado.
         """
-        # Configurar el mock para que devuelva una URL específica
+        
         mock_url = 'https://onboarding.stripe.test/account/123'
         mock_link.return_value = mock_url
 
+        self.user.profile.stripe_account_id = 'acct_test_123'
+        self.user.profile.save()
         response = self.client.get(reverse('accounts:complete_stripe_onboarding'))
 
-        # Verificar que la función mock fue llamada con el usuario correcto
         mock_link.assert_called_once_with(self.user)
 
-        # Verificar la redirección a la URL de Stripe
         self.assertEqual(response.status_code, 302)
         self.assertEqual(response.url, mock_url)
 

codigo/chat/_utils.py

@@ -1,6 +1,8 @@
 # ==========================================
 # Autor: Alejandro Gasca Mediel
 # ==========================================
+
+
 """
 Utilidades para la aplicación de chat.
 """
@@ -34,9 +36,9 @@ def get_messages_for_chat(chat):
     messages_data = []
 
     for message in messages:
-        # Format date as day/month/year
+        
         date_str = message.created_at.strftime('%d/%m/%Y')
-        # Format time as hour:minute
+        
         time_str = message.created_at.strftime('%H:%M')
 
         messages_data.append({
@@ -112,9 +114,9 @@ def synchronize_chat_participants(ride):
     if not hasattr(ride, 'chat') or not ride.chat:
         return
 
-    # Añadir el conductor
+    
     ride.chat.participants.add(ride.driver)
 
-    # Añadir todos los pasajeros
+    
     for passenger in ride.passengers.all():
         ride.chat.participants.add(passenger)

codigo/chat/tests/test_consumers.py

@@ -18,58 +18,65 @@
 import pytest
 
 class ChatConsumerTests(TestCase):
+    @database_sync_to_async
+    def get_ride_and_chat(self, ride_id):
+        from rides.models import Ride
+        ride = Ride.objects.get(id=ride_id)
+        chat = ride.chat
+        return ride, chat
+
     async def test_chat_consumer_connection(self):
         """Prueba la conexión del consumidor de chat"""
         driver = await self.create_user(DRIVER_USERNAME, DRIVER_PASSWORD)
         passenger = await self.create_user(PASSENGER_USERNAME, PASSENGER_PASSWORD)
         ride = await self.create_ride(driver, passenger)
-        
+        ride, chat = await self.get_ride_and_chat(ride.id)
         communicator = WebsocketCommunicator(
             ChatConsumer.as_asgi(),
-            WS_CHAT_PATH.format(ride.id)
+            WS_CHAT_PATH.format(chat.id)
         )
         communicator.scope["user"] = driver
-        communicator.scope["url_route"] = {"kwargs": {"ride_id": str(ride.id)}}
-        
+        communicator.scope["url_route"] = {"kwargs": {"chat_id": str(chat.id)}}
         connected, _ = await communicator.connect()
         self.assertTrue(connected)
-        
         await communicator.disconnect()
-    
+
+    @database_sync_to_async
+    def ensure_passenger_in_chat(self, ride, passenger):
+        chat = ride.chat
+        chat.participants.add(passenger)
+        chat.save()
+
     async def test_chat_consumer_receive_message(self):
         """Prueba el envío y recepción de mensajes en el consumidor"""
         driver = await self.create_user(DRIVER_USERNAME, DRIVER_PASSWORD)
         passenger = await self.create_user(PASSENGER_USERNAME, PASSENGER_PASSWORD)
         ride = await self.create_ride(driver, passenger)
-        
+        ride, chat = await self.get_ride_and_chat(ride.id)
         driver_comm = WebsocketCommunicator(
             ChatConsumer.as_asgi(),
-            WS_CHAT_PATH.format(ride.id)
+            WS_CHAT_PATH.format(chat.id)
         )
         driver_comm.scope["user"] = driver
-        driver_comm.scope["url_route"] = {"kwargs": {"ride_id": str(ride.id)}}
-        
+        driver_comm.scope["url_route"] = {"kwargs": {"chat_id": str(chat.id)}}
         passenger_comm = WebsocketCommunicator(
             ChatConsumer.as_asgi(),
-            WS_CHAT_PATH.format(ride.id)
+            WS_CHAT_PATH.format(chat.id)
         )
         passenger_comm.scope["user"] = passenger
-        passenger_comm.scope["url_route"] = {"kwargs": {"ride_id": str(ride.id)}}
-        
+        passenger_comm.scope["url_route"] = {"kwargs": {"chat_id": str(chat.id)}}
+        await self.ensure_passenger_in_chat(ride, passenger)
         await driver_comm.connect()
         await passenger_comm.connect()
-        
         await driver_comm.send_json_to({
             'message': DRIVER_MESSAGE
         })
-        
         response = await passenger_comm.receive_json_from()
-        self.assertEqual(response['content'], DRIVER_MESSAGE)
-        self.assertEqual(response['sender'], DRIVER_USERNAME)
-        
+        self.assertEqual(response['message']['content'], DRIVER_MESSAGE)
+        self.assertEqual(response['message']['sender'], DRIVER_USERNAME)
         await driver_comm.disconnect()
         await passenger_comm.disconnect()
-    
+
     @database_sync_to_async
     def create_user(self, username, password):
         """Utilidad para crear usuarios de forma asíncrona"""
@@ -78,7 +85,7 @@ def create_user(self, username, password):
             email=f'{username}@example.com',
             password=password
         )
-    
+
     @database_sync_to_async
     def create_ride(self, driver, passenger):
         """Utilidad para crear viajes de forma asíncrona"""

codigo/chat/tests/test_models.py

@@ -35,32 +35,38 @@ def setUp(self):
             total_seats=RIDE_TOTAL_SEATS,
             price=RIDE_PRICE
         )
-        
+        self.ride.refresh_from_db()
+        if not hasattr(self.ride, 'chat') or self.ride.chat is None:
+            from chat.models import Chat
+            chat = Chat.objects.create()
+            self.ride.chat = chat
+            self.ride.save()
+        self.ride.refresh_from_db()
         self.ride.passengers.add(self.passenger)
 
     def test_message_creation(self):
         """Prueba la creación de mensajes"""
+        chat = self.ride.chat
         message = Message.objects.create(
-            ride=self.ride,
+            chat=chat,
             sender=self.driver,
             content=DRIVER_MESSAGE,
             is_read=False
         )
-        
-        self.assertEqual(message.ride, self.ride)
+        self.assertEqual(message.chat, chat)
         self.assertEqual(message.sender, self.driver)
         self.assertEqual(message.content, DRIVER_MESSAGE)
         self.assertFalse(message.is_read)
         self.assertIsNotNone(message.created_at)
 
     def test_message_string_representation(self):
         """Prueba la representación en cadena del mensaje"""
+        chat = self.ride.chat
         message = Message.objects.create(
-            ride=self.ride,
+            chat=chat,
             sender=self.driver,
             content=DRIVER_MESSAGE,
             is_read=False
         )
-        
         expected = DRIVER_MESSAGE
         self.assertEqual(str(message), expected)

codigo/chat/tests/test_utils.py

@@ -10,7 +10,7 @@
 from chat.models import Message
 from rides.models import Ride
 from chat.public import user_has_chat_access, get_user_chats
-from chat._utils import format_message_for_api, get_messages_for_ride, can_send_message
+from chat._utils import format_message_for_api, can_send_message
 from .test_constants import *
 
 class ChatUtilsTests(TestCase):
@@ -45,28 +45,28 @@ def setUp(self):
         )
 
         self.ride.passengers.add(self.passenger)
-        
+        self.ride.refresh_from_db()
+        chat = self.ride.chat
+        chat.participants.add(self.passenger)  # Asegura que el pasajero está en el chat
         Message.objects.create(
-            ride=self.ride,
+            chat=chat,
             sender=self.driver,
             content=DRIVER_MESSAGE,
             is_read=True
         )
-        
         Message.objects.create(
-            ride=self.ride,
+            chat=chat,
             sender=self.passenger,
             content=PASSENGER_MESSAGE,
             is_read=False
         )
+        self.chat = chat
 
     def test_user_has_chat_access(self):
         """Prueba que verifica quién tiene acceso al chat"""
-        self.assertTrue(user_has_chat_access(self.driver, self.ride))
-        
-        self.assertTrue(user_has_chat_access(self.passenger, self.ride))
-        
-        self.assertFalse(user_has_chat_access(self.other_user, self.ride))
+        self.assertTrue(user_has_chat_access(self.driver, self.chat))
+        self.assertTrue(user_has_chat_access(self.passenger, self.chat))
+        self.assertFalse(user_has_chat_access(self.other_user, self.chat))
 
     def test_get_user_chats_for_driver(self):
         """Prueba obtener los chats del conductor"""

codigo/chat/tests/test_views.py

@@ -47,16 +47,17 @@ def setUp(self):
         )
 
         self.ride.passengers.add(self.passenger)
-        
+        self.ride.refresh_from_db()
+        chat = self.ride.chat
         Message.objects.create(
-            ride=self.ride,
+            chat=chat,
             sender=self.driver,
             content=DRIVER_MESSAGE,
             is_read=True
         )
 
         Message.objects.create(
-            ride=self.ride,
+            chat=chat,
             sender=self.passenger,
             content=PASSENGER_MESSAGE,
             is_read=False

codigo/dashboard/tests/__init__.py

@@ -0,0 +1,6 @@
+"""
+Tests para el módulo de dashboard.
+"""
+from dashboard.tests.test_constants import *
+from dashboard.tests.test_utils import DashboardUtilsTests
+from dashboard.tests.test_views import DashboardViewsTests

codigo/dashboard/tests/test_constants.py

@@ -0,0 +1,63 @@
+# ==========================================
+# Autor: Alejandro Gasca Mediel
+# ==========================================
+"""
+Constantes para las pruebas del dashboard.
+"""
+from django.utils import timezone
+from datetime import timedelta
+
+# Usuarios
+ADMIN_USERNAME = 'admin_test'
+ADMIN_EMAIL = 'admin@test.com'
+ADMIN_PASSWORD = 'admin12345'
+
+REGULAR_USERNAME = 'user_test'
+REGULAR_EMAIL = 'user@test.com'
+REGULAR_PASSWORD = 'user12345'
+
+# Periodos
+TEST_PERIOD_TODAY = 'today'
+TEST_PERIOD_WEEK = 'week'
+TEST_PERIOD_MONTH = 'month'
+TEST_PERIOD_YEAR = 'year'
+TEST_PERIOD_ALL = 'all'
+
+# URLs y templates
+URL_DASHBOARD = 'dashboard:dashboard'
+URL_RIDE_MANAGEMENT = 'dashboard:ride_management'
+URL_USER_MANAGEMENT = 'dashboard:user_management'
+URL_CHAT_MANAGEMENT = 'dashboard:chat_management'
+URL_TRIP_STATS = 'dashboard:trip_stats'
+URL_MSG_STATS = 'dashboard:msg_stats'  # Corregido
+URL_REPORT_STATS = 'dashboard:report_stats'
+URL_USER_STATS = 'dashboard:user_stats'
+
+
+# URLs para APIs
+URL_API_TRIPS = 'dashboard:api_trips'
+URL_API_MESSAGES = 'dashboard:api_messages'
+URL_API_REPORTS = 'dashboard:api_reports'
+URL_API_USERS = 'dashboard:api_users'
+URL_API_PAYMENTS = 'dashboard:api_payments'
+URL_GET_CHAT_MESSAGES = 'dashboard:get_chat_messages'
+
+TEMPLATE_DASHBOARD = 'dashboard/dashboard.html'
+TEMPLATE_RIDE_MANAGEMENT = 'dashboard/ride_management.html'
+TEMPLATE_USER_MANAGEMENT = 'dashboard/user_management.html'
+TEMPLATE_CHAT_MANAGEMENT = 'dashboard/chat_management.html'
+TEMPLATE_TRIP_STATS = 'dashboard/trip_stats.html'
+TEMPLATE_MSG_STATS = 'dashboard/msg_stats.html'
+TEMPLATE_REPORT_STATS = 'dashboard/report_stats.html'
+TEMPLATE_USER_STATS = 'dashboard/user_stats.html'
+TEMPLATE_REPORT_DETAIL = 'dashboard/report_detail.html'
+
+# Fechas
+DAYS_FUTURE = 5
+DAYS_PAST = 5
+
+# Datos de prueba
+TEST_ORIGIN = 'Madrid'
+TEST_DESTINATION = 'Barcelona'
+TEST_SEATS = 3
+TEST_PRICE = 25.0