✨ feat: 전체 랭킹 조회 + DBRider 테스트 적용 (#73)

dlsrks1021 · web-flow · commit 3c89ab62df42 · 2025-07-18T12:45:49.000+09:00
diff --git a/backend/build.gradle b/backend/build.gradle
@@ -42,6 +42,7 @@ dependencies {
 	testAnnotationProcessor 'org.projectlombok:lombok'
 	testRuntimeOnly 'com.h2database:h2'
 	testImplementation 'org.springframework.security:spring-security-test'
+	testImplementation 'com.github.database-rider:rider-spring:1.44.0'
 
 	/* ETC */
 	implementation 'org.apache.commons:commons-lang3:3.12.0'
diff --git a/backend/src/main/java/io/f1/backend/domain/stat/api/StatController.java b/backend/src/main/java/io/f1/backend/domain/stat/api/StatController.java
@@ -0,0 +1,32 @@
+package io.f1.backend.domain.stat.api;
+
+import io.f1.backend.domain.stat.app.StatService;
+import io.f1.backend.domain.stat.dto.StatPageResponse;
+import io.f1.backend.global.validation.LimitPageSize;
+
+import lombok.RequiredArgsConstructor;
+
+import org.springframework.data.domain.Pageable;
+import org.springframework.data.domain.Sort.Direction;
+import org.springframework.data.web.PageableDefault;
+import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
+
+@RestController
+@RequiredArgsConstructor
+@RequestMapping("/stats")
+public class StatController {
+
+    private final StatService statService;
+
+    @LimitPageSize
+    @GetMapping("/rankings")
+    public ResponseEntity<StatPageResponse> getRankings(
+            @PageableDefault(sort = "score", direction = Direction.DESC) Pageable pageable) {
+        StatPageResponse response = statService.getRanks(pageable);
+
+        return ResponseEntity.ok().body(response);
+    }
+}
diff --git a/backend/src/main/java/io/f1/backend/domain/stat/app/StatService.java b/backend/src/main/java/io/f1/backend/domain/stat/app/StatService.java
@@ -0,0 +1,25 @@
+package io.f1.backend.domain.stat.app;
+
+import static io.f1.backend.domain.stat.mapper.StatMapper.toStatListPageResponse;
+
+import io.f1.backend.domain.stat.dao.StatRepository;
+import io.f1.backend.domain.stat.dto.StatPageResponse;
+import io.f1.backend.domain.stat.dto.StatWithNickname;
+
+import lombok.RequiredArgsConstructor;
+
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.Pageable;
+import org.springframework.stereotype.Service;
+
+@Service
+@RequiredArgsConstructor
+public class StatService {
+
+    private final StatRepository statRepository;
+
+    public StatPageResponse getRanks(Pageable pageable) {
+        Page<StatWithNickname> stats = statRepository.findWithUser(pageable);
+        return toStatListPageResponse(stats);
+    }
+}
diff --git a/backend/src/main/java/io/f1/backend/domain/stat/dao/StatRepository.java b/backend/src/main/java/io/f1/backend/domain/stat/dao/StatRepository.java
@@ -0,0 +1,22 @@
+package io.f1.backend.domain.stat.dao;
+
+import io.f1.backend.domain.stat.dto.StatWithNickname;
+import io.f1.backend.domain.stat.entity.Stat;
+
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.Pageable;
+import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Query;
+
+public interface StatRepository extends JpaRepository<Stat, Long> {
+
+    @Query(
+            """
+            SELECT
+            		new io.f1.backend.domain.stat.dto.StatWithNickname
+            				(u.nickname, s.totalGames, s.winningGames, s.score)
+            FROM
+            		Stat s JOIN s.user u
+            """)
+    Page<StatWithNickname> findWithUser(Pageable pageable);
+}
diff --git a/backend/src/main/java/io/f1/backend/domain/stat/dto/StatPageResponse.java b/backend/src/main/java/io/f1/backend/domain/stat/dto/StatPageResponse.java
@@ -0,0 +1,6 @@
+package io.f1.backend.domain.stat.dto;
+
+import java.util.List;
+
+public record StatPageResponse(
+        int totalPages, int currentPage, int totalElements, List<StatResponse> ranks) {}
diff --git a/backend/src/main/java/io/f1/backend/domain/stat/dto/StatResponse.java b/backend/src/main/java/io/f1/backend/domain/stat/dto/StatResponse.java
@@ -0,0 +1,4 @@
+package io.f1.backend.domain.stat.dto;
+
+public record StatResponse(
+        long rank, String nickname, long totalGames, long winningGames, long score) {}
diff --git a/backend/src/main/java/io/f1/backend/domain/stat/dto/StatWithNickname.java b/backend/src/main/java/io/f1/backend/domain/stat/dto/StatWithNickname.java
@@ -0,0 +1,3 @@
+package io.f1.backend.domain.stat.dto;
+
+public record StatWithNickname(String nickname, long totalGames, long winningGames, long score) {}
diff --git a/backend/src/main/java/io/f1/backend/domain/stat/mapper/StatMapper.java b/backend/src/main/java/io/f1/backend/domain/stat/mapper/StatMapper.java
@@ -0,0 +1,40 @@
+package io.f1.backend.domain.stat.mapper;
+
+import io.f1.backend.domain.stat.dto.StatPageResponse;
+import io.f1.backend.domain.stat.dto.StatResponse;
+import io.f1.backend.domain.stat.dto.StatWithNickname;
+
+import org.springframework.data.domain.Page;
+
+public class StatMapper {
+
+    public static StatPageResponse toStatListPageResponse(Page<StatWithNickname> statPage) {
+        int curPage = statPage.getNumber() + 1;
+
+        return new StatPageResponse(
+                statPage.getTotalPages(),
+                curPage,
+                statPage.getNumberOfElements(),
+                statPage.stream()
+                        .map(
+                                stat -> {
+                                    long rank =
+                                            getRankFromPage(
+                                                    curPage,
+                                                    statPage.getSize(),
+                                                    statPage.getContent().indexOf(stat));
+                                    return toStatResponse(stat, rank);
+                                })
+                        .toList());
+    }
+
+    private static StatResponse toStatResponse(StatWithNickname stat, long rank) {
+        return new StatResponse(
+                rank, stat.nickname(), stat.totalGames(), stat.winningGames(), stat.score());
+    }
+
+    private static long getRankFromPage(int curPage, int pageSize, int index) {
+        int startRank = (curPage - 1) * pageSize + 1;
+        return startRank + index;
+    }
+}
diff --git a/backend/src/main/java/io/f1/backend/global/config/CustomPageableHandlerArgumentResolver.java b/backend/src/main/java/io/f1/backend/global/config/CustomPageableHandlerArgumentResolver.java
@@ -0,0 +1,45 @@
+package io.f1.backend.global.config;
+
+import io.f1.backend.global.exception.CustomException;
+import io.f1.backend.global.exception.errorcode.CommonErrorCode;
+import io.f1.backend.global.validation.LimitPageSize;
+
+import org.springframework.core.MethodParameter;
+import org.springframework.data.domain.PageRequest;
+import org.springframework.data.domain.Pageable;
+import org.springframework.data.web.PageableHandlerMethodArgumentResolver;
+import org.springframework.web.bind.support.WebDataBinderFactory;
+import org.springframework.web.context.request.NativeWebRequest;
+import org.springframework.web.method.support.ModelAndViewContainer;
+
+public class CustomPageableHandlerArgumentResolver extends PageableHandlerMethodArgumentResolver {
+
+    @Override
+    public Pageable resolveArgument(
+            MethodParameter methodParameter,
+            ModelAndViewContainer mavContainer,
+            NativeWebRequest webRequest,
+            WebDataBinderFactory binderFactory) {
+
+        Pageable pageable =
+                super.resolveArgument(methodParameter, mavContainer, webRequest, binderFactory);
+
+        if (methodParameter.hasMethodAnnotation(LimitPageSize.class)) {
+            LimitPageSize limitPageSize = methodParameter.getMethodAnnotation(LimitPageSize.class);
+            validatePageable(limitPageSize, pageable);
+        }
+
+        return PageRequest.of(
+                oneIndexedPageNumber(pageable.getPageNumber()), pageable.getPageSize());
+    }
+
+    private int oneIndexedPageNumber(int pageNumber) {
+        return pageNumber <= 0 ? 0 : pageNumber - 1;
+    }
+
+    private void validatePageable(LimitPageSize limitPageSize, Pageable pageable) {
+        if (pageable.getPageSize() > limitPageSize.max()) {
+            throw new CustomException(CommonErrorCode.INVALID_PAGINATION);
+        }
+    }
+}
diff --git a/backend/src/main/java/io/f1/backend/global/config/WebConfig.java b/backend/src/main/java/io/f1/backend/global/config/WebConfig.java
@@ -3,9 +3,12 @@
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.web.filter.HiddenHttpMethodFilter;
+import org.springframework.web.method.support.HandlerMethodArgumentResolver;
 import org.springframework.web.servlet.config.annotation.ResourceHandlerRegistry;
 import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;
 
+import java.util.List;
+
 @Configuration
 public class WebConfig implements WebMvcConfigurer {
 
@@ -17,6 +20,11 @@ public void addResourceHandlers(ResourceHandlerRegistry registry) {
                 .addResourceLocations("file:images/thumbnail/");
     }
 
+    @Override
+    public void addArgumentResolvers(List<HandlerMethodArgumentResolver> resolvers) {
+        resolvers.add(new CustomPageableHandlerArgumentResolver());
+    }
+
     @Bean
     public HiddenHttpMethodFilter hiddenHttpMethodFilter() {
         return new HiddenHttpMethodFilter();
diff --git a/backend/src/main/java/io/f1/backend/global/validation/LimitPageSize.java b/backend/src/main/java/io/f1/backend/global/validation/LimitPageSize.java
@@ -0,0 +1,14 @@
+package io.f1.backend.global.validation;
+
+import java.lang.annotation.Documented;
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
+
+@Documented
+@Target(ElementType.METHOD)
+@Retention(RetentionPolicy.RUNTIME)
+public @interface LimitPageSize {
+    int max() default 100;
+}
diff --git a/backend/src/test/java/io/f1/backend/domain/stat/StatBrowserTest.java b/backend/src/test/java/io/f1/backend/domain/stat/StatBrowserTest.java
@@ -0,0 +1,89 @@
+package io.f1.backend.domain.stat;
+
+import static io.f1.backend.global.exception.errorcode.CommonErrorCode.INVALID_PAGINATION;
+
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+import com.github.database.rider.core.api.dataset.DataSet;
+
+import io.f1.backend.global.template.BrowserTestTemplate;
+
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+import org.springframework.security.test.context.support.WithMockUser;
+import org.springframework.test.web.servlet.ResultActions;
+
+@WithMockUser
+public class StatBrowserTest extends BrowserTestTemplate {
+    @Test
+    @DataSet("datasets/stat/one-user-stat.yml")
+    @DisplayName("총 유저 수가 1명이면 첫 페이지에서 1개의 결과를 반환한다")
+    void totalRankingForSingleUser() throws Exception {
+        // when
+        ResultActions result = mockMvc.perform(get("/stats/rankings"));
+
+        // then
+        result.andExpectAll(
+                status().isOk(),
+                jsonPath("$.totalPages").value(1),
+                jsonPath("$.currentPage").value(1),
+                jsonPath("$.totalElements").value(1),
+                jsonPath("$.ranks.length()").value(1));
+    }
+
+    @Test
+    @DisplayName("100을 넘는 페이지 크기 요청이 오면 예외를 발생시킨다")
+    void totalRankingForSingleUserWithInvalidPageSize() throws Exception {
+        // when
+        ResultActions result = mockMvc.perform(get("/stats/rankings").param("size", "101"));
+
+        // then
+        result.andExpectAll(
+                status().isBadRequest(), jsonPath("$.code").value(INVALID_PAGINATION.getCode()));
+    }
+
+    @Test
+    @DataSet("datasets/stat/three-user-stat.yml")
+    @DisplayName("총 유저 수가 3명이면 첫 페이지에서 3개의 결과를 반환한다")
+    void totalRankingForThreeUser() throws Exception {
+        // when
+        ResultActions result = mockMvc.perform(get("/stats/rankings"));
+
+        // then
+        result.andExpectAll(
+                status().isOk(),
+                jsonPath("$.totalPages").value(1),
+                jsonPath("$.currentPage").value(1),
+                jsonPath("$.totalElements").value(3),
+                jsonPath("$.ranks.length()").value(3));
+    }
+
+    @Test
+    @DataSet("datasets/stat/three-user-stat.yml")
+    @DisplayName("총 유저 수가 3명일 때 페이지 크기가 2이면, 첫 페이지에서 2개, 두 번째 페이지에 1개의 결과를 반환한다")
+    void totalRankingForThreeUserWithPageSize2() throws Exception {
+        // when
+        ResultActions resultPage1 =
+                mockMvc.perform(get("/stats/rankings").param("page", "1").param("size", "2"));
+
+        ResultActions resultPage2 =
+                mockMvc.perform(get("/stats/rankings").param("page", "2").param("size", "2"));
+
+        // then
+        resultPage1.andExpectAll(
+                status().isOk(),
+                jsonPath("$.totalPages").value(2),
+                jsonPath("$.currentPage").value(1),
+                jsonPath("$.totalElements").value(2),
+                jsonPath("$.ranks.length()").value(2));
+
+        resultPage2.andExpectAll(
+                status().isOk(),
+                jsonPath("$.totalPages").value(2),
+                jsonPath("$.currentPage").value(2),
+                jsonPath("$.totalElements").value(1),
+                jsonPath("$.ranks.length()").value(1));
+    }
+}
diff --git a/backend/src/test/java/io/f1/backend/global/config/TestPhysicalNamingStrategy.java b/backend/src/test/java/io/f1/backend/global/config/TestPhysicalNamingStrategy.java
@@ -0,0 +1,20 @@
+package io.f1.backend.global.config;
+
+import org.hibernate.boot.model.naming.CamelCaseToUnderscoresNamingStrategy;
+import org.hibernate.boot.model.naming.Identifier;
+import org.hibernate.engine.jdbc.env.spi.JdbcEnvironment;
+
+public class TestPhysicalNamingStrategy extends CamelCaseToUnderscoresNamingStrategy {
+
+    @Override
+    public Identifier toPhysicalTableName(Identifier name, JdbcEnvironment context) {
+        if (name == null) return null;
+
+        String tableName = name.getText();
+        if ("user".equalsIgnoreCase(tableName)) {
+            return Identifier.toIdentifier("user_test");
+        }
+
+        return super.toPhysicalTableName(name, context);
+    }
+}
diff --git a/backend/src/test/java/io/f1/backend/global/template/BrowserTestTemplate.java b/backend/src/test/java/io/f1/backend/global/template/BrowserTestTemplate.java
@@ -0,0 +1,15 @@
+package io.f1.backend.global.template;
+
+import com.github.database.rider.spring.api.DBRider;
+
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.test.web.servlet.MockMvc;
+
+@DBRider
+@SpringBootTest
+@AutoConfigureMockMvc
+public abstract class BrowserTestTemplate {
+    @Autowired protected MockMvc mockMvc;
+}
diff --git a/backend/src/test/resources/application.yml b/backend/src/test/resources/application.yml
@@ -1,9 +1,15 @@
 spring:
   datasource:
-    url: jdbc:h2:mem:testdb;MODE=MYSQL
+    url: jdbc:h2:mem:testdb;MODE=MYSQL;DATABASE_TO_LOWER=TRUE
     username: sa
     password:
 
+  jpa:
+    hibernate:
+      ddl-auto: create
+      naming:
+        physical-strategy: io.f1.backend.global.config.TestPhysicalNamingStrategy
+
   sql:
     init:
       mode: never
diff --git a/backend/src/test/resources/datasets/stat.yml b/backend/src/test/resources/datasets/stat.yml
@@ -0,0 +1,18 @@
+stat:
+  - id: 1
+    user_id: 1
+    total_games: 100
+    winning_games: 100
+    score: 1000
+
+  - id: 2
+    user_id: 2
+    total_games: 20
+    winning_games: 20
+    score: 200
+
+  - id: 3
+    user_id: 3
+    total_games: 30
+    winning_games: 30
+    score: 300
diff --git a/backend/src/test/resources/datasets/stat/one-user-stat.yml b/backend/src/test/resources/datasets/stat/one-user-stat.yml
diff --git a/backend/src/test/resources/datasets/stat/three-user-stat.yml b/backend/src/test/resources/datasets/stat/three-user-stat.yml
diff --git a/backend/src/test/resources/datasets/user.yml b/backend/src/test/resources/datasets/user.yml
diff --git a/backend/src/test/resources/dbunit.yml b/backend/src/test/resources/dbunit.yml

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+package io.f1.backend.domain.stat.dto;`
	`2`	`+`
	`3`	`+public record StatWithNickname(String nickname, long totalGames, long winningGames, long score) {}`