feat: Búsqueda de Películas por Atributos, ej: buscar por nombre y duración; por nombre y género; por nombre, género y duración; etc.

Author: luiscacuango2

src/main/java/com/luigi/javatest/movies/model/Movie.java

@@ -5,15 +5,15 @@
 public class Movie {
     private Integer id;
     private String name;
-    private int minutes;
+    private Integer minutes;
     private Gender gender;
     private String director; // Nuevo atributo
 
     public Movie(String name, int minutes, Gender gender, String director) {
         this(null, name, minutes, gender, director);
     }
 
-    public Movie(Integer id, String name, int minutes, Gender gender, String director) {
+    public Movie(Integer id, String name, Integer minutes, Gender gender, String director) {
         this.id = id;
         this.name = name;
         this.minutes = minutes;
@@ -29,7 +29,7 @@ public String getName() {
         return name;
     }
 
-    public int getMinutes() {
+    public Integer getMinutes() {
         return minutes;
     }
 

src/main/java/com/luigi/javatest/movies/service/MovieService.java

@@ -36,4 +36,25 @@ public Collection<Movie> findMoviesByDirector(String director) {
                 .filter(movie -> movie.getDirector().toLowerCase().contains(director.toLowerCase()))
                 .collect(Collectors.toList());
     }
+
+    public Collection<Movie> findMoviesByTemplate(Movie template) {
+        // Regla: Si hay ID, ignoramos lo demás
+        if (template.getId() != null) {
+            return movieRepository.findAll().stream()
+                    .filter(m -> m.getId().equals(template.getId()))
+                    .collect(Collectors.toList());
+        }
+
+        // Regla: Minutos negativos lanzan excepción
+        if (template.getMinutes() != null && template.getMinutes() < 0) {
+            throw new IllegalArgumentException("Los minutos no pueden ser negativos");
+        }
+
+        return movieRepository.findAll().stream()
+                .filter(m -> template.getName() == null || m.getName().toLowerCase().contains(template.getName().toLowerCase().trim()))
+                .filter(m -> template.getMinutes() == null || m.getMinutes() <= template.getMinutes())
+                .filter(m -> template.getGender() == null || m.getGender().equals(template.getGender()))
+                .filter(m -> template.getDirector() == null || m.getDirector().toLowerCase().contains(template.getDirector().toLowerCase().trim()))
+                .collect(Collectors.toList());
+    }
 }

src/test/java/com/luigi/javatest/movies/service/MovieServiceShould.java

@@ -30,11 +30,11 @@ void setUp() {
                 Arrays.asList(
                         new Movie(1, "Dark Knight", 152, Gender.ACTION, "Christopher Nolan"),
                         new Movie(2, "Memento", 113, Gender.THRILLER, "Christopher Nolan"),
-                        new Movie(3, "Super 8", 112, Gender.THRILLER, "J.J. Abrams"),
-                        new Movie(4, "Superman", 103, Gender.ACTION, "Richard Donner"),
+                        new Movie(3, "Super 8", 112, Gender.DRAMA, "J.J. Abrams"),
+                        new Movie(4, "Superman", 173, Gender.ACTION, "Richard Donner"),
                         new Movie(5, "Home Alone", 103, Gender.COMEDY, "Chris Columbus"),
                         new Movie(6, "Home Alone", 120, Gender.COMEDY, "director1"),
-                        new Movie(7, "Matrix", 136, Gender.ACTION, "director2")
+                        new Movie(7, "Matrix", 166, Gender.ACTION, "director2")
                 )
         );
     }
@@ -54,7 +54,7 @@ public void return_movies_by_length() {
         // WHEN
         Collection<Movie> movies = movieService.findMoviesByLength(120);
         // THEN
-        assertEquals(Arrays.asList(2, 3, 4, 5, 6), getMoviesIds(movies));
+        assertEquals(Arrays.asList(2, 3, 5, 6), getMoviesIds(movies));
     }
 
     public static List<Integer> getMoviesIds(Collection<Movie> movies) {
@@ -79,4 +79,41 @@ void return_movies_by_director() {
         List<Integer> ids = movies.stream().map(Movie::getId).collect(Collectors.toList());
         assertEquals(Arrays.asList(1, 2), ids); // Dark Knight y Memento
     }
+    @Test
+    void find_movies_by_genre_and_max_duration() {
+        // Buscar peliculas de ACCION de menos de 160 min (Debería salir Superman)
+        Movie template = new Movie(null, null, 160, Gender.ACTION, null);
+        Collection<Movie> movies = movieService.findMoviesByTemplate(template);
+
+        assertEquals(1, movies.size());
+        assertTrue(movies.stream().allMatch(m -> m.getName().equals("Dark Knight")));
+    }
+
+    @Test
+    void find_movies_by_name_and_director() {
+        // Buscar "Super" del director "Abrams"
+        Movie template = new Movie(null, "Super", null, null, "Abrams");
+        Collection<Movie> movies = movieService.findMoviesByTemplate(template);
+
+        assertEquals(1, movies.size());
+    }
+
+    @Test
+    void throw_exception_when_minutes_are_negative() {
+        Movie template = new Movie(null, null, -10, null, null);
+
+        assertThrows(IllegalArgumentException.class, () -> {
+            movieService.findMoviesByTemplate(template);
+        });
+    }
+
+    @Test
+    void ignore_other_filters_if_id_is_present() {
+        // Aunque pida comedia, si el ID 1 es Acción, debe devolver el 1 (Acción)
+        Movie template = new Movie(1, "Inexistente", 10, Gender.ACTION, "Nadie");
+        Collection<Movie> movies = movieService.findMoviesByTemplate(template);
+
+        assertEquals(1, movies.size());
+        assertEquals("Dark Knight", movies.iterator().next().getName());
+    }
 }