Acabada primera versión de modelo del sistema carpooling

.mvn/wrapper/maven-wrapper.properties

-distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.4/apache-maven-3.9.4-bin.zip
-wrapperUrl=https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.2.0/maven-wrapper-3.2.0.jar

nbproject/project.properties

+debug=true

pom.xml

@@ -24,22 +24,21 @@
             <artifactId>spring-boot-starter</artifactId>
         </dependency>
 
-        <dependency>
+          <!-- <dependency>
             <groupId>org.springframework.boot</groupId>
             <artifactId>spring-boot-starter-data-jpa</artifactId>
-        </dependency>
+        </dependency>   -->
 
         <dependency>
+    <groupId>org.hibernate</groupId>
+    <artifactId>hibernate-core</artifactId>
+    <version>5.0.7.Final</version>
+</dependency>
+         <dependency>
             <groupId>mysql</groupId>
             <artifactId>mysql-connector-java</artifactId>
             <scope>runtime</scope>
-        </dependency>
-
-        <dependency>
-            <groupId>com.h2database</groupId>
-            <artifactId>h2</artifactId>
-            <scope>test</scope>
-        </dependency>
+        </dependency> 
 
 
         <dependency>
@@ -58,11 +57,11 @@
             <artifactId>junit-jupiter</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
+         <dependency>
             <groupId>org.projectlombok</groupId>
             <artifactId>lombok</artifactId>
             <optional>true</optional>
-        </dependency>
+        </dependency> 
 
     </dependencies>
     <build>

src/main/java/es/ujaen/dae/carpoolingdae/CarpoolingDaeApplication.java → src/main/java/es/ujaen/dae/carpoolingdae/app/CarpoolingDaeApplication.java

-package es.ujaen.dae.carpoolingdae;
+package es.ujaen.dae.carpoolingdae.app;
 
 import org.springframework.boot.SpringApplication;
 import org.springframework.boot.autoconfigure.SpringBootApplication;
 import org.springframework.context.ApplicationContext;
 
-@SpringBootApplication(scanBasePackages="es.ujaen.dae.carpooling")
+@SpringBootApplication(scanBasePackages="es.ujaen.dae.carpooling.servicios")
 public class CarpoolingDaeApplication {
 
 	public static void main(String[] args) throws Exception {
+		// Creación de servidor
 		SpringApplication servidor = new SpringApplication(CarpoolingDaeApplication.class);
 		ApplicationContext context = servidor.run(args);
 		

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

 package es.ujaen.dae.carpoolingdae.entidades;
 
-import java.io.Serializable;
-import java.util.ArrayList;
-import java.util.List;
+
+/* import java.util.ArrayList;
+import java.util.List; */
 
 import javax.persistence.Entity;
 import javax.persistence.GeneratedValue;
 import javax.persistence.GenerationType;
 import javax.persistence.Id;
-import javax.validation.constraints.NotBlank;
 
-import lombok.Data;
-import lombok.NoArgsConstructor;
+
+
 
 @Entity
 public class Reserva {
 
     @Id
     @GeneratedValue(strategy = GenerationType.IDENTITY)
-    private Long id;
+    int id;
 
     //@OneToMany
-    private List<Usuario> pasajerosConfirmados = new ArrayList<>();
+   // private List<Usuario> pasajerosConfirmados = new ArrayList<>();
 
     // Constructor, getters y setters
 }

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

@@ -5,8 +5,6 @@ import java.util.ArrayList;
 
 import javax.persistence.Entity;
 import javax.persistence.Id;
-import javax.persistence.ManyToMany;
-import javax.persistence.OneToMany;
 import javax.validation.constraints.Email;
 import javax.validation.constraints.NotBlank;
 import javax.validation.constraints.Past;
@@ -23,6 +21,10 @@ import es.ujaen.dae.carpoolingdae.utils.ExprReg;
 import lombok.Data;
 import lombok.NoArgsConstructor;
 
+/**
+ * Usuario a la app de carpooling
+ * @author maria
+ */
 @Data
 @NoArgsConstructor
 @Entity
@@ -32,25 +34,25 @@ public class Usuario {
     @Id
     @Size(min = 9, max = 9)
     @Pattern(regexp = ExprReg.DNI)
-    private String dni;
+    String dni;
 
     @NotBlank
-    private String nombre;
+    String nombre;
 
     @NotBlank
-    private String apellidos;
+    String apellidos;
 
     @Past
-    private LocalDate fecha_nac;
+    LocalDate fecha_nac;
 
     @Pattern(regexp = ExprReg.TLF)
-    private String telef;
+    String telef;
 
     @Email
-    private String email;
+    String email;
 
     @NotBlank
-    private String clave;
+    String clave;
 
     public Usuario(String dni, String nombre, String apellidos, LocalDate fecha_nac, String telef, String email,
             String clave) {
@@ -88,45 +90,48 @@ public class Usuario {
         return email;
     }
 
-    // private List<Viaje> viajesActuales;
+    // Lista de viajes actuales disponibles
+     List<Viaje> viajesActuales;
     private int totalViajes = 0;
 
     public void crearViaje(Viaje viaje) {
-        viaje.setId(totalViajes);
-        /*
-         * viajesActuales.add(viaje);
-         * totalViajes++;
-         */
+          viajesActuales.add(viaje);
+          totalViajes++;
+         
     }
 
-    @OneToMany(mappedBy = "conductor") // Relación uno a muchos con Viaje (Usuario como conductor)
-    private List<Viaje> viajesComoConductor = new ArrayList<>();
-
-    @ManyToMany(mappedBy = "pasajeros") // Relación muchos a muchos con Viaje (Usuario como pasajero)
-    private List<Viaje> viajesComoPasajero = new ArrayList<>();
-
-  
-    // Otros campos y métodos
+    //Lista de viajes como pasajero
+     List<Viaje> viajesComoPasajero = new ArrayList<>();
 
+     /**
+     * Enviar solicitud para ser pasajero de un viaje
+     * @param viaje al que se envía la solicitud
+     */
     public void enviarSolicitudViaje(Viaje viaje) {
         if (viaje != null && !viajesComoPasajero.contains(viaje)) {
-            viajesComoPasajero.add(viaje);
-            viaje.getUsuariosSolicitantes().add(this); // Agregar este usuario a la lista de solicitantes en el Viaje
+            //viajesComoPasajero.add(viaje);
+            viaje.usuariosSolicitantes.add(this); // Agregar este usuario a la lista de solicitantes en el Viaje
         }
     }
+ /**
+     * Aceptar pasajero en un viaje
+     * @param viaje
+     * @param pasajero que es aceptado en el viaje
+     */
 
     public void aceptarPasajero(Viaje viaje, Usuario pasajero) {
-        if (viaje != null && viaje.getUsuariosSolicitantes().contains(pasajero)) {
-            viaje.getUsuariosSolicitantes().remove(pasajero);
-            viaje.getUsuariosConfirmados().add(pasajero);
+        if (viaje != null && viaje.usuariosSolicitantes.contains(pasajero)) {
+            viaje.usuariosSolicitantes.remove(pasajero);
+            viaje.usuariosConfirmados.add(pasajero);
             pasajero.getViajesComoPasajero().add(viaje);
         }
     }
-    
-    
-
-
 
+       /**
+     * Compara la clave con la del usuario, codificándola en Md5
+     * @param clave
+     * @return 
+     */
 
     public boolean claveValida(String clave) {
         return clave.equals(clave);

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

@@ -4,25 +4,20 @@ import java.time.LocalDateTime;
 import java.util.ArrayList;
 import java.util.List;
 
-import javax.persistence.Entity;
 import javax.persistence.GeneratedValue;
 import javax.persistence.GenerationType;
 import javax.persistence.Id;
-import javax.persistence.OneToMany;
 import javax.validation.constraints.NotBlank;
 import javax.validation.constraints.Past;
-import lombok.Data;
-import lombok.NoArgsConstructor;
 
-@Data
-@NoArgsConstructor
-@Entity
+
+
 public class Viaje  {
 
-    @OneToMany(mappedBy = "viajeSolicitado") // Relación uno a muchos con Usuario
-    private List<Usuario> usuariosSolicitantes = new ArrayList<>();
-   
-    private List<Usuario> usuariosConfirmados = new ArrayList<>();
+    //Lista de usuarios que solicitan unirse a un viaje
+     List<Usuario> usuariosSolicitantes = new ArrayList<>();
+   //Lista de usuarios que son confirmados en un viaje
+     List<Usuario> usuariosConfirmados = new ArrayList<>();
 
 
 @Id
@@ -30,21 +25,21 @@ public class Viaje  {
 int id;
 
 @NotBlank
-    private String destino;
+     String destino;
 
     @NotBlank
-    private String origen;
+     String origen;
 
     @Past
-    private LocalDateTime fecha_hora;
+     LocalDateTime fecha_hora;
 
     @NotBlank
-    private int plazasDisponibles;
+     int plazasDisponibles;
 
     @NotBlank
-    private Float precioAsiento;
+     Double precioAsiento;
 
-    public Viaje(int id, String destino, String origen, LocalDateTime fecha_hora, int plazasDisponibles, Float precioAsiento){
+    public Viaje(int id, String destino, String origen, LocalDateTime fecha_hora, int plazasDisponibles, Double precioAsiento){
         this.id = id;
         this.origen = origen;
         this.destino = destino;
@@ -69,7 +64,7 @@ int id;
      public int getPlazas(){
         return plazasDisponibles;
     }
-     public Float getPrecio(){
+     public Double getPrecio(){
         return precioAsiento;
     }
     

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

@@ -23,7 +23,9 @@ import es.ujaen.dae.carpoolingdae.excepciones.UsuarioYaRegistrado;
 @Validated
 public class SistemaCarpooling {
 
+     /** Mapa con la lista de Usuarios */
     Map<String, Usuario> usuarios;
+     /** Mapa con la lista de Viajes */
     Map<String, Viaje> viajes;
 
     public SistemaCarpooling(){
@@ -31,26 +33,39 @@ public class SistemaCarpooling {
         viajes = new TreeMap<>();
     }
 
+     /**
+     * Dar de alta usuario 
+     * @param usuario el usuario a dar de alta
+     */
+
      public void altaUsuario(@NotNull @Valid Usuario usuario) {
         if (usuarios.containsKey(usuario.getDni())) {
             throw new UsuarioYaRegistrado();
         }
         
-        // Registrar cliente
+        // Registrar usuario
         usuarios.put(usuario.getDni(), usuario);
 
     
 }
 
  /**
-     * Realiza un login de un cliente
-     * @param dni el DNI del cliente
+     * Realiza un login de un usuario
+     * @param email email del usuario
      * @param clave la clave de acceso
-     * @return el objeto de la clase Cliente asociado
+     * @return el objeto de la clase Viaje asociado
      */
-    public Optional<Usuario> loginCliente(@NotBlank String email, @NotBlank String clave) {
+    public Optional<Usuario> loginUsuario(@NotBlank String email, @NotBlank String clave) {
         return Optional.ofNullable(usuarios.get(email)).filter((cliente)->cliente.claveValida(clave));
     }
+
+     /**
+     * Buscar viaje
+     * @param origen donde empieza el viaje
+     * @param destino donde acaba el viaje
+     * @param fecha fecha del viaje
+     * @return viajes encontrados con esos datos
+     */
    
     public List<Viaje> buscarViajes(String origen, String destino, LocalDate fecha) {
         List<Viaje> viajesEncontrados = new ArrayList<>();
@@ -62,6 +77,11 @@ public class SistemaCarpooling {
         return viajesEncontrados;
     }
 
+       /**
+     * Agregar viaje
+     * @param nuevoViaje viaje creado
+     */
+
     public void agregarViaje(@NotNull @Valid Viaje nuevoViaje) {
         viajes.put(String.valueOf(nuevoViaje.getId()), nuevoViaje);
     }

src/main/resources/application.properties

-

src/test/java/com/example/demo/DemoApplicationTests.java

-package com.example.demo;
-
-import org.junit.jupiter.api.Test;
-import org.springframework.boot.test.context.SpringBootTest;
-
-@SpringBootTest
-class DemoApplicationTests {
-
-	@Test
-	void contextLoads() {
-	}
-
-}

src/test/java/es/ujaen/dae/carpooling/carpoolingdae/CarpoolingDaeApplicationTests.java

-package es.ujaen.dae.carpooling.carpoolingdae;
-
-import org.junit.jupiter.api.Test;
-import org.springframework.boot.test.context.SpringBootTest;
-
-@SpringBootTest
-class CarpoolingDaeApplicationTests {
-
-	@Test
-	void contextLoads() {
-	}
-
-}

src/test/java/es/ujaen/dae/carpoolingdae/CarpoolingDaeApplicationTests.java

-package es.ujaen.dae.carpoolingdae;
-
-import org.junit.jupiter.api.Test;
-import org.springframework.boot.test.context.SpringBootTest;
-
-@SpringBootTest
-class CarpoolingDaeApplicationTests {
-
-	@Test
-	void contextLoads() {
-	}
-
-}

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

+package es.ujaen.dae.carpoolingdae.entidades;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import java.time.LocalDate;
+import java.time.LocalDateTime;
+
+import javax.validation.ConstraintViolation;
+import javax.validation.Validation;
+import javax.validation.Validator;
+import org.junit.jupiter.api.Test;
+import org.assertj.core.api.Assertions;
+
+import java.util.Set;
+
+
+public class UsuarioTest {
+
+    public UsuarioTest() {}
+
+    @Test
+    void testValidacionUsuario() {
+        String clave  = "mariahm";
+        
+        Usuario usuario = new Usuario(
+            "22334455H",
+            "María",
+            "Hidalgo Morcillo",
+            LocalDate.of(1990, 11, 1),
+            "189273810",
+            "mhm@red.ujaen.es",
+            clave);
+        
+        Validator validator = Validation.buildDefaultValidatorFactory().getValidator();
+        Set<ConstraintViolation<Usuario>> violations = validator.validate(usuario);
+                
+        Assertions.assertThat(violations).isEmpty();
+    }
+
+        @Test
+    void testCrearViaje() {
+        String clave = "mariahm";
+        
+       Usuario usuario = new Usuario(
+            "22334455H",
+            "María",
+            "Hidalgo Morcillo",
+            LocalDate.of(1990, 11, 1),
+            "189273810",
+            "mhm@red.ujaen.es",
+            clave);
+        
+        Viaje viaje = new Viaje(1,  "Origen", "Destino", LocalDateTime.of(2023, 9, 30, 8, 0), 4, 50.0);
+        usuario.crearViaje(viaje);
+        
+    }  
+        
+
+    @Test
+    public void testEnviarSolicitudViaje() {
+        Viaje viaje = new Viaje(1,  "Origen", "Destino", LocalDateTime.of(2023, 9, 30, 8, 0), 4, 50.0);
+        Usuario usuario = new Usuario("987654321", "OtroNombre", "OtroApellido", LocalDate.of(1995, 2, 2), "987654321", "otrocorreo@example.com", "clave");
+
+        usuario.enviarSolicitudViaje(viaje);
+        assertTrue(viaje.usuariosSolicitantes.contains(usuario));
+
+        // Asegúrate de que otroUsuario no esté en la lista de solicitantes
+        assertFalse(viaje.usuariosSolicitantes.contains(usuario));
+    }
+
+    @Test
+    public void testAceptarPasajero() {
+        Viaje viaje = new Viaje(1,  "Origen", "Destino", LocalDateTime.of(2023, 9, 30, 8, 0), 4, 50.0);
+        Usuario pasajero = new Usuario("987654321", "OtroNombre", "OtroApellido", LocalDate.of(1995, 2, 2), "987654321", "otrocorreo@example.com", "clave");
+
+        viaje.usuariosSolicitantes.add(pasajero);
+        pasajero.aceptarPasajero(viaje, pasajero);
+
+        assertFalse(viaje.usuariosSolicitantes.contains(pasajero));
+        assertTrue(viaje.usuariosConfirmados.contains(pasajero));
+        assertTrue(pasajero.getViajesComoPasajero().contains(viaje));
+    }
+
+
+    @Test
+    void testComprobacionClave() {
+        String clave = "mariahm";
+        
+       Usuario usuario = new Usuario(
+            "22334455H",
+            "María",
+            "Hidalgo Morcillo",
+            LocalDate.of(1990, 11, 1),
+            "189273810",
+            "mhm@red.ujaen.es",
+            clave);
+        
+        Assertions.assertThat(usuario.claveValida(clave)).isTrue();
+    }
+    
+  
+}

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

+package es.ujaen.dae.carpoolingdae.servicios;
+
+import java.time.LocalDate;
+import java.time.LocalDateTime;
+import java.util.List;
+import java.util.Optional;
+
+import javax.validation.ConstraintViolationException;
+
+import org.assertj.core.api.Assertions;
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.test.annotation.DirtiesContext;
+import org.springframework.test.annotation.DirtiesContext.MethodMode;
+
+import es.ujaen.dae.carpoolingdae.entidades.Usuario;
+//import es.ujaen.dae.carpoolingdae.utils.CodificadorMd5;
+import es.ujaen.dae.carpoolingdae.entidades.Viaje;
+
+//@Disabled
+@SpringBootTest(classes = es.ujaen.dae.carpoolingdae.app.CarpoolingDaeApplication.class)
+public class SistemaCarpoolingTest {
+
+    @Autowired
+    SistemaCarpooling sistemaCarpooling;
+
+    @Test
+    public void testAccesoSistemaCarpooling() {
+        Assertions.assertThat(sistemaCarpooling).isNotNull();
+    }
+
+    @Test
+    @DirtiesContext(methodMode = MethodMode.AFTER_METHOD)
+    public void testAltaUsuarioInvalido() {
+        // Usuario con e-mail incorrecto!!!
+        // String clave = "mariahm";
+        Usuario usuario = new Usuario(
+                "22334455H",
+                "María",
+                "Hidalgo Morcillo",
+                LocalDate.of(1990, 11, 1),
+                "189273810",
+                "mhm@red..es",
+                "clave");
+
+        Assertions.assertThatThrownBy(() -> {
+            sistemaCarpooling.altaUsuario(usuario);
+        }).isInstanceOf(ConstraintViolationException.class);
+    }
+
+    @Test
+    @DirtiesContext(methodMode = MethodMode.AFTER_METHOD)
+    public void testAltaYLoginUsuarioCuenta() {
+        Usuario usuario = new Usuario(
+                "22334455H",
+                "María",
+                "Hidalgo Morcillo",
+                LocalDate.of(1990, 11, 1),
+                "189273810",
+                "mhm@red.ujaen.es",
+                "clave");
+
+        // Cuenta cuenta = servicioUjaCoin.altaCliente(cliente);
+        Optional<Usuario> usuarioLogin = sistemaCarpooling.loginUsuario(usuario.getEmail(), "clave");
+
+        Assertions.assertThat(usuarioLogin.isPresent()).isTrue();
+        Assertions.assertThat(usuarioLogin.get()).isEqualTo(usuario);
+    }
+
+    @Test
+    @DirtiesContext(methodMode = MethodMode.AFTER_METHOD)
+    public void testIngreso() {
+        // Registrar cliente y realizar login
+        Usuario usuario = new Usuario(
+                "22334455H",
+                "María",
+                "Hidalgo Morcillo",
+                LocalDate.of(1990, 11, 1),
+                "189273810",
+                "mhm@red.ujaen.es",
+                "clave");
+
+        // Cuenta cuenta = servicioUjaCoin.altaCliente(cliente);
+        Usuario usuarioLogin = sistemaCarpooling.loginUsuario(usuario.getEmail(), "clave").get();
+
+        // Añadir una tarjeta
+
+    }
+
+    @Test
+    @DirtiesContext(methodMode = MethodMode.AFTER_METHOD)
+    public void testBuscarViajes() {
+
+        Viaje viaje = new Viaje(
+        12345, // ID del viaje
+        "Origen",
+        "Destino",
+        LocalDateTime.of(2023, 9, 30, 8, 0),
+        2,
+        50.0);
+
+
+        // Crear algunos viajes de prueba
+
+        // Agregar los viajes al sistema
+        sistemaCarpooling.agregarViaje(viaje);
+
+        // Realizar la búsqueda de viajes
+        String origenBusqueda = "Origen";
+        String destinoBusqueda = "Destino";
+        LocalDate fechaBusqueda = LocalDate.of(2023, 9, 30);
+
+        List<Viaje> viajesEncontrados = sistemaCarpooling.buscarViajes(origenBusqueda, destinoBusqueda, fechaBusqueda);
+
+        // Verificar que se encuentren los viajes esperados
+        Assertions.assertThat(viajesEncontrados).containsExactly(viaje);
+
+        // Limpiar los viajes creados durante la prueba
+        // sistemaCarpooling.eliminarViaje(viaje);
+
+    }
+
+    @Test
+    @DirtiesContext(methodMode = MethodMode.AFTER_METHOD)
+    public void testAgregarViaje() {
+        // Crear un objeto Viaje válido
+        Viaje nuevoViaje = new Viaje(
+        12345, // ID del viaje
+        "Origen",
+        "Destino",
+        LocalDateTime.of(2023, 9, 30, 8, 0),
+        2,
+        50.0);
+
+         // Agregar el viaje al sistema
+         sistemaCarpooling.agregarViaje(nuevoViaje);
+
+      // Buscar el viaje paraver si ha sido añadido
+        String origenBusqueda = "Origen";
+        String destinoBusqueda = "Destino";
+        LocalDate fechaBusqueda = LocalDate.of(2023, 9, 30);
+
+        List<Viaje> viajesEncontrados = sistemaCarpooling.buscarViajes(origenBusqueda, destinoBusqueda, fechaBusqueda);
+
+        // Verificar que se encuentren los viajes esperados
+        Assertions.assertThat(viajesEncontrados).contains(nuevoViaje);
+
+    }
+}
+
+