Commit d607f091 by María Hidalgo

limpieza de codigo y test funcionando

parent 9b5c8e30
......@@ -21,8 +21,6 @@ import es.ujaen.dae.carpoolingdae.excepciones.ViajeRealizado;
import es.ujaen.dae.carpoolingdae.utils.CodificadorMd5;
import es.ujaen.dae.carpoolingdae.utils.ExprReg;
//import lombok.Data;
//import lombok.NoArgsConstructor;
import jakarta.persistence.CascadeType;
import jakarta.persistence.Entity;
import jakarta.persistence.FetchType;
......
......@@ -72,7 +72,7 @@ public class Viaje {
public Viaje(String destino, String origen, LocalDateTime fechaHora, int plazasDisponibles,
Double precioAsiento) {
//this.id = id;
this.origen = origen;
this.destino = destino;
this.fechaHora = fechaHora;
......
......@@ -23,7 +23,6 @@ import jakarta.persistence.Query;
public class RepositorioViajes {
//private final Logger logger = Logger.getLogger(Viaje.class.getName());
@PersistenceContext
EntityManager em;
......@@ -33,9 +32,8 @@ public class RepositorioViajes {
}
public void guardarViaje(Viaje viaje) {
//viaje=em.merge(viaje);
em.persist(viaje);
//em.flush();
}
public List<Viaje> buscarViajes(String origen, String destino, LocalDateTime fecha) {
......
package es.ujaen.dae.carpoolingdae.servicios;
import jakarta.transaction.Transactional;
import jakarta.validation.Valid;
import jakarta.validation.constraints.NotBlank;
import jakarta.validation.constraints.NotNull;
......@@ -142,7 +141,7 @@ public class SistemaCarpooling {
* @param viaje el viaje del que se obtendrán las solicitudes
* @return lista de solicitudes pendientes del viaje
*/
//@Transactional
public List<Usuario> obtenerSolicitudesPendientes(@NotNull Viaje viaje) {
// Verificar que el viaje esté en estado PENDIENTE
if (viaje.getEstado() != EstadoViaje.PENDIENTE) {
......@@ -159,7 +158,7 @@ public class SistemaCarpooling {
* @param viaje el viaje del que se obtendrán los pasajeros
* @return lista de pasajeros confirmados del viaje
*/
//@Transactional
public List<Usuario> obtenerPasajeros(@NotNull Viaje viaje) {
//Podemos ver los pasajeros antes y después del viaje
if (viaje.getEstado() != EstadoViaje.PENDIENTE && viaje.getEstado() != EstadoViaje.REALIZADO) {
......
......@@ -7,7 +7,6 @@ import es.ujaen.dae.carpoolingdae.excepciones.UsuarioNoRegistrado;
import static org.junit.jupiter.api.Assertions.assertEquals;
//import jakarta.validation.ConstraintViolationException;
import static org.junit.jupiter.api.Assertions.assertTrue;
......@@ -111,7 +110,7 @@ public class SistemaCarpoolingTest {
// Agregar el viaje al sistema
sistemaCarpooling.agregarViaje(usuario, viaje);
// el viaje paraver si ha sido añadido
//ver si ha sido añadido
String origenBusqueda = "Origen";
String destinoBusqueda = "Destino";
LocalDateTime fechaBusqueda = LocalDateTime.of(2023, 9, 30, 8, 0);
......@@ -125,8 +124,6 @@ public class SistemaCarpoolingTest {
Assertions.assertThat(viajeEnc.getPlazasDisponibles()).isEqualTo(viaje.getPlazasDisponibles());
Assertions.assertThat(viajeEnc.getPrecioAsiento()).isEqualTo(viaje.getPrecioAsiento());
// Verificar que se encuentren los viajes esperados
// Assertions.assertThat(viajesEncontrados).contains(viaje);
}
......@@ -169,6 +166,7 @@ public class SistemaCarpoolingTest {
// Agregar el viaje al sistema
sistemaCarpooling.agregarViaje(conductor, viaje);
viaje.setEstado(EstadoViaje.PENDIENTE);
// el viaje paraver si ha sido añadido
String origenBusqueda = "Origen";
......@@ -193,6 +191,7 @@ public class SistemaCarpoolingTest {
// Comprobar que el pasajero está en la lista de solicitudes pendientes
assertTrue(solicitudesPendientes.contains(pasajero));
assertEquals(EstadoViaje.PENDIENTE, viaje.getEstado());
}
//@Transactional
......@@ -223,6 +222,7 @@ public class SistemaCarpoolingTest {
// Login antes de agregar el viaje
sistemaCarpooling.loginUsuario(conductor.getDni(), "clave");
// Crear un viaje
Viaje viaje = new Viaje(
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment