limpieza de codigo

src/main/java/es/ujaen/dae/carpoolingdae/entidades/Usuario.java

@@ -26,9 +26,6 @@ import jakarta.persistence.CascadeType;
 import jakarta.persistence.Entity;
 import jakarta.persistence.Entity;
 import jakarta.persistence.FetchType;
 import jakarta.persistence.FetchType;
 import jakarta.persistence.Id;
 import jakarta.persistence.Id;
-import jakarta.persistence.JoinColumn;
-import jakarta.persistence.ManyToMany;
-import jakarta.persistence.ManyToOne;
 import jakarta.persistence.OneToMany;
 import jakarta.persistence.OneToMany;
 
 
 /**
 /**

src/main/java/es/ujaen/dae/carpoolingdae/entidades/Viaje.java

@@ -13,8 +13,7 @@ import jakarta.persistence.GenerationType;
 import jakarta.persistence.Id;
 import jakarta.persistence.Id;
 import jakarta.persistence.JoinColumn;
 import jakarta.persistence.JoinColumn;
 import jakarta.persistence.ManyToMany;
 import jakarta.persistence.ManyToMany;
-import jakarta.persistence.ManyToOne;
-import jakarta.persistence.OneToMany;
+
 import jakarta.validation.constraints.Past;
 import jakarta.validation.constraints.Past;
 import lombok.Data;
 import lombok.Data;
 import lombok.NoArgsConstructor;
 import lombok.NoArgsConstructor;

src/main/java/es/ujaen/dae/carpoolingdae/repositorios/RepositorioViajes.java

 package es.ujaen.dae.carpoolingdae.repositorios;
 package es.ujaen.dae.carpoolingdae.repositorios;
 
 
-import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.LocalDateTime;
 import java.util.ArrayList;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.List;
@@ -17,8 +16,6 @@ import jakarta.persistence.EntityManager;
 import jakarta.persistence.PersistenceContext;
 import jakarta.persistence.PersistenceContext;
 import jakarta.persistence.Query;
 import jakarta.persistence.Query;
 
 
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 
 
 
 @Repository
 @Repository
@@ -26,7 +23,7 @@ import java.util.logging.Logger;
 public class RepositorioViajes {
 public class RepositorioViajes {
 
 
 
 
-    private final Logger logger = Logger.getLogger(Viaje.class.getName());
+    //private final Logger logger = Logger.getLogger(Viaje.class.getName());
     @PersistenceContext
     @PersistenceContext
     EntityManager em;
     EntityManager em;
 
 

src/main/java/es/ujaen/dae/carpoolingdae/servicios/SistemaCarpooling.java

@@ -5,23 +5,19 @@ import jakarta.validation.Valid;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.NotNull;
 
 
-import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.LocalDateTime;
-import java.util.ArrayList;
 import java.util.Optional;
 import java.util.Optional;
 
 
 import java.util.List;
 import java.util.List;
 
 
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.stereotype.Service;
-import org.springframework.transaction.annotation.Transactional;
 import org.springframework.validation.annotation.Validated;
 import org.springframework.validation.annotation.Validated;
 
 
 import es.ujaen.dae.carpoolingdae.entidades.Usuario;
 import es.ujaen.dae.carpoolingdae.entidades.Usuario;
 import es.ujaen.dae.carpoolingdae.entidades.Viaje;
 import es.ujaen.dae.carpoolingdae.entidades.Viaje;
 import es.ujaen.dae.carpoolingdae.excepciones.UsuarioNoRegistrado;
 import es.ujaen.dae.carpoolingdae.excepciones.UsuarioNoRegistrado;
 import es.ujaen.dae.carpoolingdae.excepciones.UsuarioYaRegistrado;
 import es.ujaen.dae.carpoolingdae.excepciones.UsuarioYaRegistrado;
-import es.ujaen.dae.carpoolingdae.excepciones.ViajeNoExistente;
 import es.ujaen.dae.carpoolingdae.repositorios.RepositorioUsuarios;
 import es.ujaen.dae.carpoolingdae.repositorios.RepositorioUsuarios;
 import es.ujaen.dae.carpoolingdae.repositorios.RepositorioViajes;
 import es.ujaen.dae.carpoolingdae.repositorios.RepositorioViajes;
 
 
@@ -96,16 +92,9 @@ public class SistemaCarpooling {
      */
      */
 
 
     public List<Viaje> buscarViajes(@NotBlank String origen, @NotBlank String destino, @NotNull LocalDateTime fecha) {
     public List<Viaje> buscarViajes(@NotBlank String origen, @NotBlank String destino, @NotNull LocalDateTime fecha) {
-        List<Viaje> viajes = new ArrayList<>();
+        
         List<Viaje> listaViajes = repositorioViajes.buscarViajes(origen,destino,fecha);
         List<Viaje> listaViajes = repositorioViajes.buscarViajes(origen,destino,fecha);
-     /*    for (Viaje viaje : listaViajes) { // Iterar sobre la colección de viajes
-            if (viaje.getOrigen().equals(origen) && viaje.getDestino().equals(destino)
-                    && viaje.getfechaHora().toLocalDate().equals(fecha)) { 
-                viajes.add(viaje);
-            } else {
-                throw new ViajeNoExistente();
-            }
-        }  */
+    
         return listaViajes;
         return listaViajes;
 
 
     }
     }

src/test/java/es/ujaen/dae/carpoolingdae/entidades/UsuarioTest.java

@@ -6,7 +6,6 @@ import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.LocalDateTime;
 
 
 
 
-import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.Test;
 
 
 import jakarta.validation.ConstraintViolation;
 import jakarta.validation.ConstraintViolation;

src/test/java/es/ujaen/dae/carpoolingdae/entidades/ViajeTest.java

 package es.ujaen.dae.carpoolingdae.entidades;
 package es.ujaen.dae.carpoolingdae.entidades;
 
 
-import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assertions.assertTrue;

src/test/java/es/ujaen/dae/carpoolingdae/servicios/SistemaCarpoolingTest.java

 package es.ujaen.dae.carpoolingdae.servicios;
 package es.ujaen.dae.carpoolingdae.servicios;
-import es.ujaen.dae.carpoolingdae.app.CarpoolingDaeApplication;
 import es.ujaen.dae.carpoolingdae.entidades.Usuario;
 import es.ujaen.dae.carpoolingdae.entidades.Usuario;
 import es.ujaen.dae.carpoolingdae.entidades.Viaje;
 import es.ujaen.dae.carpoolingdae.entidades.Viaje;
 import es.ujaen.dae.carpoolingdae.excepciones.UsuarioNoRegistrado;
 import es.ujaen.dae.carpoolingdae.excepciones.UsuarioNoRegistrado;
@@ -13,7 +12,6 @@ import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.LocalDateTime;
 import java.util.List;
 import java.util.List;
 
 
-import java.util.Optional;
 import jakarta.validation.ConstraintViolationException;
 import jakarta.validation.ConstraintViolationException;
 
 
 
 
@@ -22,7 +20,6 @@ import org.assertj.core.api.Assertions;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.context.annotation.ComponentScan;
 import org.springframework.test.annotation.DirtiesContext;
 import org.springframework.test.annotation.DirtiesContext;
 import org.springframework.test.annotation.DirtiesContext.MethodMode;
 import org.springframework.test.annotation.DirtiesContext.MethodMode;
 import org.springframework.test.context.ActiveProfiles;
 import org.springframework.test.context.ActiveProfiles;
@@ -126,6 +123,11 @@ public class SistemaCarpoolingTest {
         List<Viaje> viajesEncontrados = sistemaCarpooling.buscarViajes(origenBusqueda, destinoBusqueda, fechaBusqueda);
         List<Viaje> viajesEncontrados = sistemaCarpooling.buscarViajes(origenBusqueda, destinoBusqueda, fechaBusqueda);
         Viaje viajeEnc=viajesEncontrados.get(0);
         Viaje viajeEnc=viajesEncontrados.get(0);
         Assertions.assertThat(viajeEnc.getDestino()).isEqualTo(viaje.getDestino());
         Assertions.assertThat(viajeEnc.getDestino()).isEqualTo(viaje.getDestino());
+        Assertions.assertThat(viajeEnc.getOrigen()).isEqualTo(viaje.getOrigen());
+        Assertions.assertThat(viajeEnc.getfechaHora()).isEqualTo(viaje.getfechaHora());
+        Assertions.assertThat(viajeEnc.getPlazasDisponibles()).isEqualTo(viaje.getPlazasDisponibles());
+        Assertions.assertThat(viajeEnc.getPrecioAsiento()).isEqualTo(viaje.getPrecioAsiento());
+       
 
 
         // Verificar que se encuentren los viajes esperados
         // Verificar que se encuentren los viajes esperados
        // Assertions.assertThat(viajesEncontrados).contains(viaje);
        // Assertions.assertThat(viajesEncontrados).contains(viaje);