refactor: movieRepository 메서드 변경 findByDeletedAt() -> findValidMovies()

server/src/main/java/com/ticketing/server/movie/domain/repository/MovieRepository.java

@@ -5,6 +5,7 @@ import java.time.LocalDateTime;
 import java.util.List;
 import java.util.Optional;
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Query;
 import org.springframework.stereotype.Repository;
 
 @Repository
@@ -12,6 +13,9 @@ public interface MovieRepository extends JpaRepository<Movie, Long> {
 
     Optional<Movie> findByTitle(String title);
 
-    List<Movie> findByDeletedAt(LocalDateTime localDateTime);
+    @Query(value = "SELECT * "
+        + "FROM movie "
+        + "WHERE deleted_at IS NULL", nativeQuery = true)
+    List<Movie> findValidMovies();
 
 }

server/src/test/java/com/ticketing/server/movie/service/MovieServiceImplTest.java

@@ -1,12 +1,13 @@
 package com.ticketing.server.movie.service;
 
-import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.Mockito.when;
 
 import com.ticketing.server.movie.domain.Movie;
 import com.ticketing.server.movie.domain.repository.MovieRepository;
 import com.ticketing.server.movie.service.dto.MovieDto;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
@@ -30,26 +31,36 @@ public class MovieServiceImplTest {
     @InjectMocks
     MovieServiceImpl movieService;
 
-    @BeforeEach
-    void init() {
-        movie = new Movie("범죄도시2", 106);
-        movieDto = movie.toDto();
+    @Test
+    @DisplayName("Movie Service Test - get movies when there is no movie")
+    void shouldGetEmptyList() {
+        // given
+        when(movieRepository.findByDeletedAt(null)).thenReturn(Collections.emptyList());
+
+        // when
+        List<MovieDto> movieDtoList = movieService.getMovies();
+
+        // then
+        assertTrue(movieDtoList.isEmpty());
 
-        movies.add(movie);
-        movieDtos.add(movieDto);
     }
 
     @Test
     @DisplayName("Movie Service Test - get movies")
     void shouldAbleToGetMovies() {
         // given
+        movie = new Movie("범죄도시2", 106);
+        movieDto = movie.toDto();
+        movies.add(movie);
+        movieDtos.add(movieDto);
+
         when(movieRepository.findByDeletedAt(null)).thenReturn(movies);
 
         // when
         List<MovieDto> movieDtoList = movieService.getMovies();
 
         // then
-        assertFalse(movieDtoList.isEmpty());
+        assertTrue(!movieDtoList.isEmpty());
     }
 
 }