feat: 전체 랭킹 조회 구현

dlsrks1021 · dlsrks1021 · commit 60e4d69ecf69 · 2025-07-18T09:43:41.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,23 @@
+package io.f1.backend.domain.stat.app;
+
+import static io.f1.backend.domain.stat.mapper.StatMapper.toStatListPageResponse;
+
+import io.f1.backend.domain.stat.dto.StatWithNickname;
+import io.f1.backend.domain.stat.dao.StatRepository;
+import io.f1.backend.domain.stat.dto.StatPageResponse;
+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,21 @@
+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,10 @@
+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,9 @@
+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,8 @@
+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/entity/Stat.java b/backend/src/main/java/io/f1/backend/domain/stat/entity/Stat.java
@@ -14,9 +14,11 @@
 
 import lombok.AccessLevel;
 import lombok.Builder;
+import lombok.Getter;
 import lombok.NoArgsConstructor;
 
 @Entity
+@Getter
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 public class Stat extends BaseEntity {
 
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,44 @@
+package io.f1.backend.domain.stat.mapper;
+
+import io.f1.backend.domain.stat.dto.StatWithNickname;
+import io.f1.backend.domain.stat.dto.StatPageResponse;
+import io.f1.backend.domain.stat.dto.StatResponse;
+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,40 @@
+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
@@ -1,8 +1,10 @@
 package io.f1.backend.global.config;
 
+import java.util.List;
 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;
 
@@ -17,7 +19,12 @@ public void addResourceHandlers(ResourceHandlerRegistry registry) {
                 .addResourceLocations("file:images/thumbnail/");
     }
 
-    @Bean
+	@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,106 @@
+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.MockMvcResultHandlers.print;
+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,16 @@
+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
diff --git a/backend/src/test/resources/datasets/stat.yml b/backend/src/test/resources/datasets/stat.yml
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
