Merge pull request #15 from vvbakhanovich/add-remove-endpoint

feat: Удаление фильмов и пользователей

Author: ayzatmr

pom.xml

@@ -5,7 +5,7 @@
 	<parent>
 		<groupId>org.springframework.boot</groupId>
 		<artifactId>spring-boot-starter-parent</artifactId>
-		<version>2.7.14</version>
+		<version>2.7.17</version>
 		<relativePath/> <!-- lookup parent from repository -->
 	</parent>
 	<groupId>ru.yandex.practicum</groupId>
@@ -54,6 +54,7 @@
 		<dependency>
 			<groupId>com.h2database</groupId>
 			<artifactId>h2</artifactId>
+			<version>2.2.220</version>
 			<scope>runtime</scope>
 		</dependency>
 

src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java

@@ -52,5 +52,10 @@ public FilmDto removeLike(@PathVariable long id, @PathVariable long userId) {
     public Collection<FilmDto> getMostPopularFilms(@RequestParam(required = false, defaultValue = "10") int count) {
         return filmService.getMostPopularFilms(count);
     }
+
+    @DeleteMapping("/{id}")
+    public void removeFilm(@PathVariable long id) {
+        filmService.removeFilm(id);
+    }
 }
 

src/main/java/ru/yandex/practicum/filmorate/controller/UserController.java

@@ -59,6 +59,11 @@ public void removeFriend(@PathVariable long id, @PathVariable long friendId) {
         userService.removeFriend(id, friendId);
     }
 
+    @DeleteMapping("/{id}")
+    public void removeUser(@PathVariable long id) {
+        userService.removeUser(id);
+    }
+
     @GetMapping("/{id}/recommendations")
     public Collection<FilmDto> showRecommendations(@PathVariable long id) {
         return userService.showRecommendations(id);

src/main/java/ru/yandex/practicum/filmorate/dao/impl/FilmDbStorage.java

@@ -56,7 +56,10 @@ public Film add(final Film film) {
     @Override
     public void remove(final long id) {
         final String sql = "DELETE FROM film WHERE id = ?";
-        jdbcTemplate.update(sql, id);
+        int amount = jdbcTemplate.update(sql, id);
+        if (amount != 1) {
+            throw new NotFoundException("Фильм с id '" + id + "' не найден.");
+        }
     }
 
     @Override
@@ -69,7 +72,6 @@ public void update(final Film film) {
         }
 
         filmGenreStorage.deleteAllById(film.getId());
-
         filmGenreStorage.batchUpdate(film.getId(), film.getGenres());
     }
 

src/main/java/ru/yandex/practicum/filmorate/service/FilmService.java

@@ -17,5 +17,7 @@ public interface FilmService {
 
     FilmDto removeLike(long filmId, long userId);
 
+    void removeFilm(long filmId);
+
     Collection<FilmDto> getMostPopularFilms(final int count);
 }

src/main/java/ru/yandex/practicum/filmorate/service/UserService.java

@@ -23,5 +23,7 @@ public interface UserService {
 
     void removeFriend(long userId, long friendId);
 
+    void removeUser(long userId);
+
     Collection<FilmDto> showRecommendations(long id);
 }

src/main/java/ru/yandex/practicum/filmorate/service/impl/FilmServiceImpl.java

@@ -114,6 +114,16 @@ public FilmDto removeLike(final long filmId, final long userId) {
         return toDto(filmStorage.findById(filmId));
     }
 
+    /**
+     * Удаление фильма.
+     *
+     * @param filmId идентификатор фильма, который будет удален
+     */
+    @Override
+    public void removeFilm(long filmId) {
+        filmStorage.remove(filmId);
+    }
+
     /**
      * Получение списка самых популярных фильмов. Под популярностью понимается количество лайков у фильма. Чем больше
      * лайков, тем популярнее фильм.

src/main/java/ru/yandex/practicum/filmorate/service/impl/UserServiceImpl.java

@@ -156,6 +156,16 @@ public void removeFriend(final long userId, final long friendId) {
         log.info("Пользователи с id {} и {} перестали быть друзьями", userId, friendId);
     }
 
+    /**
+     * Удаление пользователя.
+     *
+     * @param userId идентификатор пользователя, который будет удален
+     */
+    @Override
+    public void removeUser(long userId) {
+        userStorage.remove(userId);
+    }
+
     /**
      * Создание рекомендаций пользователю фильмов, которые ему могут понравиться. Для подготовки рекомендаций
      * выгружаем данные из таблицы film_like, находим пользователей с максимальным количеством одинаковых с нашим

src/main/resources/schema.sql

@@ -25,7 +25,7 @@ CREATE TABLE IF NOT EXISTS FILM_GENRE(
     ID INTEGER GENERATED BY DEFAULT AS IDENTITY(START WITH 1) NOT NULL PRIMARY KEY,
     FILM_ID INTEGER NOT NULL,
     GENRE_ID INTEGER NOT NULL,
-	FOREIGN KEY (FILM_ID) REFERENCES FILM(ID),
+	FOREIGN KEY (FILM_ID) REFERENCES FILM(ID) ON DELETE CASCADE,
 	FOREIGN KEY (GENRE_ID) REFERENCES GENRE(ID)
 );
 
@@ -59,7 +59,7 @@ CREATE TABLE IF NOT EXISTS FILM_LIKE(
     FILM_ID INTEGER NOT NULL,
     USER_ID INTEGER NOT NULL,
 	CONSTRAINT pk_film_like PRIMARY KEY (FILM_ID, USER_ID),
-	FOREIGN KEY (FILM_ID) REFERENCES FILM(ID),
+	FOREIGN KEY (FILM_ID) REFERENCES FILM(ID) ON DELETE CASCADE,
 	FOREIGN KEY (USER_ID) REFERENCES FILMORATE_USER(ID)
 );
 

src/test/java/ru/yandex/practicum/filmorate/storage/FilmDbStorageTest.java

@@ -287,6 +287,26 @@ void testFindByIdWithAllFields() {
                 .isEqualTo(film);
     }
 
+    @Test
+    @DisplayName("Тест удаление фильма")
+    void testDeleteFilm() {
+        Film newFilm = filmDbStorage.add(film);
+        filmDbStorage.remove(newFilm.getId());
+
+        String formattedResponse = String.format("Фильм с id '%s' не найден.", newFilm.getId());
+        NotFoundException e = assertThrows(NotFoundException.class, () -> filmDbStorage.findById(newFilm.getId()));
+        assertEquals(formattedResponse, e.getMessage());
+    }
+
+    @Test
+    @DisplayName("Тест удаление несуществующего фильма")
+    void testDeleteNotExistingUser() {
+        int filmId = 999;
+        String formattedResponse = String.format("Фильм с id '%s' не найден.", filmId);
+        NotFoundException e = assertThrows(NotFoundException.class, () -> filmDbStorage.remove(filmId));
+        assertEquals(formattedResponse, e.getMessage());
+    }
+
     @Test
     @DisplayName("Тест на получение самых популярных фильмов с несколькими жанрами")
     void testMostPopularFilmsWithSeveralGenres() {

src/test/java/ru/yandex/practicum/filmorate/storage/UserDbStorageTest.java

@@ -345,6 +345,26 @@ void testGetEmptyFriendsList() {
                 .isEmpty();
     }
 
+    @Test
+    @DisplayName("Тест удаление пользователя")
+    void testDeleteUser() {
+        User newUser = userStorage.add(user);
+        userStorage.remove(newUser.getId());
+
+        String formattedResponse = String.format("Пользователь с id '%s' не найден.", newUser.getId());
+        NotFoundException e = assertThrows(NotFoundException.class, () -> userStorage.findById(newUser.getId()));
+        assertEquals(formattedResponse, e.getMessage());
+    }
+
+    @Test
+    @DisplayName("Тест удаление несуществующего пользователя")
+    void testDeleteNotExistingUser() {
+        int userId = 999;
+        String formattedResponse = String.format("Пользователь с id '%s' не найден.", userId);
+        NotFoundException e = assertThrows(NotFoundException.class, () -> userStorage.remove(userId));
+        assertEquals(formattedResponse, e.getMessage());
+    }
+
     @Test
     @DisplayName("Тест получения мапы с ключами userId и сетом с списком айдишников залайканных фильмов.")
     void testGetRecommendationsList() {