Test #451
@@ -0,0 +1,357 @@
|
||||
# 라이브 크리에이터 입장 제한 구현 계획
|
||||
|
||||
> **For agentic workers:** REQUIRED SUB-SKILL: Use `subagent-driven-development` or `executing-plans` to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
|
||||
|
||||
| 문서 항목 | 내용 |
|
||||
|---|---|
|
||||
| 상태 | Phase 1~3 구현 및 검증 완료 |
|
||||
| 작성일 | `2026-09-17` |
|
||||
| 요구사항 기준 | `docs/20260917_라이브_크리에이터_입장제한/prd.md` |
|
||||
| API 기준 | 기존 API 응답 스키마 변경 없음 |
|
||||
| 현재 Phase | 완료 |
|
||||
| 현재 활성 Goal | 없음 |
|
||||
| 다음 Goal | 없음 |
|
||||
|
||||
## 목표
|
||||
|
||||
다른 크리에이터가 `isAvailableJoinCreator = false` 라이브 방을 볼 수도, 직접 입장할 수도 없게 하고, 입장 가능 성별(`genderRestriction`)도 조회와 입장 경계에서 일관되게 적용한다.
|
||||
|
||||
## 현재 상태
|
||||
|
||||
| Phase | 상태 | 완료 Task | 활성/다음 Goal | 차단 또는 남은 조건 |
|
||||
|---:|---|---:|---|---|
|
||||
| 1 | 완료 | `2/2` | 없음 | 없음 |
|
||||
| 2 | 완료 | `1/1` | 없음 | 없음 |
|
||||
| 3 | 완료 | `2/2` | 없음 | 없음 |
|
||||
|
||||
## 범위
|
||||
|
||||
### 포함
|
||||
|
||||
- `v2` 홈 추천/온에어 라이브 조회의 `isAvailableJoinCreator` 및 `genderRestriction` 필터 누락 보정.
|
||||
- `v2` 크리에이터 채널 라이브 탭의 조회자 크리에이터 판정 보정.
|
||||
- `/live/room/enter`에서 다른 크리에이터의 제한 방 직접 입장 차단과 기존 성별 입장 차단 회귀 검증.
|
||||
- focused test와 직접 영향 회귀 검증.
|
||||
- 후속 리뷰 `REV-001`: `/live/room/info/{id}`의 토큰 발급 전 크리에이터·성별 제한 적용.
|
||||
- 후속 리뷰 `REV-002`: 채널 소유자 본인의 제한 방 조회를 repository 회귀 테스트로 고정.
|
||||
|
||||
### 제외
|
||||
|
||||
- 공개 API DTO 필드 추가/삭제.
|
||||
- DB schema 변경.
|
||||
- 기존 성인/성별/차단/강퇴/비공개/결제 정책 재설계.
|
||||
- 관리자용 라이브 조회 정책 변경.
|
||||
|
||||
## 기술적 제약
|
||||
|
||||
- 기술 스택: Kotlin, Spring Boot 2.7.14, QueryDSL, Gradle Wrapper.
|
||||
- 기존 QueryDSL 조건 패턴을 재사용하고 새 abstraction은 만들지 않는다.
|
||||
- 크리에이터 제한 조건은 `isViewerCreator && memberId != null`일 때만 적용하며, 방 생성자는 `liveRoom.member.id.eq(memberId)`로 예외 처리한다.
|
||||
- 성별 제한 조건은 기존 조회 경로와 같은 정책을 따른다. `Gender.MALE`은 `ALL`, `MALE_ONLY`, `Gender.FEMALE`은 `ALL`, `FEMALE_ONLY`, `Gender.NONE` 또는 비로그인/null 유효 성별은 필터 없음이다.
|
||||
- 방 생성자 본인은 성별 제한 조건에서도 `liveRoom.member.id.eq(memberId)`로 예외 처리한다.
|
||||
- 모든 production 변경 Task는 `RED → GREEN → REFACTOR` 순서로 진행한다.
|
||||
- 공개 API schema 변경은 금지한다.
|
||||
|
||||
## Phase 1: 리스트 노출 제한 보정
|
||||
|
||||
**Phase 결과:** 다른 크리에이터가 문제 의심 리스트 경로에서 제한 방을 받지 않고, 성별 제한에 맞지 않는 사용자가 v2 홈 추천/온에어 라이브에서 제한 방을 받지 않는다.
|
||||
|
||||
**선행조건:** PRD 확정.
|
||||
|
||||
**Phase 완료 조건:** `P1-T1`, `P1-T2`, `P1-GATE` 완료 및 검증 기록 누적.
|
||||
|
||||
### 구현 항목
|
||||
|
||||
#### Task 1.1 v2 홈 추천/온에어 라이브 필터 적용
|
||||
|
||||
**Goal 실행 `P1-T1`:** `findLiveRecommendations()` 계열 호출에서 조회자 크리에이터 여부와 유효 성별을 전달하고 QueryDSL 필터를 적용한다.
|
||||
|
||||
- **시작 조건:** `LCR-001` 확정.
|
||||
- **완료 증거:** 다른 크리에이터가 제한 방을 받지 않고, 성별 제한에 맞지 않는 사용자가 제한 방을 받지 않으며, 방 생성자 본인은 예외인 focused test 통과.
|
||||
- **범위 밖:** `/live/room/enter` 입장 차단.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/v2/recommendation/port/out/HomeRecommendationQueryPort.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/v2/recommendation/application/HomeRecommendationQueryService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/v2/recommendation/adapter/out/persistence/DefaultHomeRecommendationQueryRepository.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/v2/api/home/application/HomeRecommendationFacade.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/v2/api/home/live/application/HomeOnAirLiveFacade.kt`
|
||||
- Test: repository/service 기존 테스트 위치를 우선 확인하고, 없으면 `src/test/kotlin/kr/co/vividnext/sodalive/v2/recommendation/adapter/out/persistence/DefaultHomeRecommendationQueryRepositoryTest.kt`
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- `findLiveRecommendations(offset, limit, memberId, includeAdultLives, isViewerCreator, effectiveViewerGender)` 형태로 파라미터를 추가한다.
|
||||
- `isViewerCreator` 기본값은 `false`로 둬 snapshot refresh 등 기존 호출을 보존한다.
|
||||
- `effectiveViewerGender` 기본값은 `null`로 둬 비로그인/성별 미설정과 snapshot refresh 등 기존 호출을 보존한다.
|
||||
- `HomeRecommendationFacade`와 `HomeOnAirLiveFacade`는 `member.auth?.gender` 우선, 없으면 `member.gender`로 기존 경로와 같은 유효 성별을 계산해 전달한다.
|
||||
|
||||
- [x] **RED:** 크리에이터 조회자에게 `isAvailableJoinCreator = false` 라이브가 제외되는 테스트, 성별 제한에 맞지 않는 조회자에게 `genderRestriction` 제한 라이브가 제외되는 테스트, 방 생성자 본인은 포함되는 테스트를 작성한다.
|
||||
- [x] **RED 확인:** `./gradlew test --tests '*DefaultHomeRecommendationQueryRepositoryTest'`를 실행해 필터 미구현으로 크리에이터 제한 방 또는 성별 제한 방이 포함되는 assertion 실패를 확인한다.
|
||||
- [x] **GREEN:** `HomeRecommendationQueryPort`, `HomeRecommendationQueryService`, `DefaultHomeRecommendationQueryRepository`, `HomeRecommendationFacade`, `HomeOnAirLiveFacade`에 `isViewerCreator`, `effectiveViewerGender` 전달과 조건을 최소 추가한다.
|
||||
- [x] **GREEN 확인:** `./gradlew test --tests '*DefaultHomeRecommendationQueryRepositoryTest'`를 다시 실행해 성공을 확인한다.
|
||||
- [x] **REFACTOR:** 이번 Task가 만든 중복만 정리하고 `./gradlew ktlintCheck` 및 focused test 결과를 Progress에 기록한다.
|
||||
|
||||
#### Task 1.2 v2 크리에이터 채널 라이브 탭 판정 보정
|
||||
|
||||
**Goal 실행 `P1-T2`:** 크리에이터 채널 라이브 탭에서 “조회자가 크리에이터인지” 기준으로 제한 방 노출을 막는다.
|
||||
|
||||
- **시작 조건:** `LCR-002` 확정.
|
||||
- **완료 증거:** 다른 크리에이터는 제한 방을 받지 않고 방 생성자 본인은 받을 수 있는 focused test 통과.
|
||||
- **범위 밖:** 크리에이터 채널 홈은 이미 `viewer.role == MemberRole.CREATOR`를 사용하므로 변경하지 않는다.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/v2/creator/channel/live/application/CreatorChannelLiveQueryService.kt`
|
||||
- Test: 기존 테스트 위치를 우선 확인하고, 없으면 `src/test/kotlin/kr/co/vividnext/sodalive/v2/creator/channel/live/application/CreatorChannelLiveQueryServiceTest.kt`
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- `queryPort.findCurrentLive(..., isViewerCreator = viewer.role == MemberRole.CREATOR, ...)`로 전달한다.
|
||||
- repository의 기존 `creatorJoinLiveCondition(viewerId, isViewerCreator)`와 본인 예외는 유지한다.
|
||||
|
||||
- [x] **RED:** 다른 크리에이터 viewer가 `isAvailableJoinCreator = false` 현재 라이브를 받지 않는 service test를 작성한다.
|
||||
- [x] **RED 확인:** `./gradlew test --tests '*CreatorChannelLiveQueryServiceTest'`를 실행해 기존 `viewerId == creatorId` 판정 때문에 제한 방이 반환되는 assertion 실패를 확인한다.
|
||||
- [x] **GREEN:** `CreatorChannelLiveQueryService`의 `isViewerCreator` 계산을 `viewer.role == MemberRole.CREATOR`로 변경한다.
|
||||
- [x] **GREEN 확인:** 같은 focused test를 다시 실행해 성공을 확인한다.
|
||||
- [x] **REFACTOR:** 중복 정리 없이 끝낼 수 있으면 그대로 두고 focused test 및 `./gradlew ktlintCheck` 결과를 Progress에 기록한다.
|
||||
|
||||
### 검증 방법
|
||||
|
||||
#### Phase 1 Gate
|
||||
|
||||
**Goal 실행 `P1-GATE`:** 리스트 노출 제한과 성별 제한 경로가 PRD 요구사항과 일치하는지 판정한다.
|
||||
|
||||
- **시작 조건:** `P1-T1`, `P1-T2` 완료.
|
||||
- **완료 증거:** 아래 명령 통과와 수동 대조 기록.
|
||||
- **범위 밖:** 입장 차단 구현.
|
||||
|
||||
```bash
|
||||
./gradlew test --tests '*DefaultHomeRecommendationQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest'
|
||||
./gradlew ktlintCheck
|
||||
```
|
||||
|
||||
**Expected:** focused test와 ktlint가 0 exit code로 종료한다.
|
||||
|
||||
수동 대조:
|
||||
|
||||
- [x] `HomeFollowingQueryService`와 `CreatorChannelHomeQueryService`의 기존 정상 조건은 변경하지 않았음을 확인한다.
|
||||
- [x] `HomeRecommendationFacade.getHomeRecommendations`, `HomeRecommendationFacade.getLives`, `HomeOnAirLiveFacade.getOnAirLives`가 모두 조회자 크리에이터 여부와 유효 성별을 전달함을 확인한다.
|
||||
|
||||
## Phase 2: 직접 입장 차단
|
||||
|
||||
**Phase 결과:** 다른 크리에이터가 제한 방 ID를 알고 있어도 `/live/room/enter`로 입장할 수 없고, 성별 제한에 맞지 않는 사용자의 직접 입장도 기존 정책대로 차단된다.
|
||||
|
||||
**선행조건:** `P1-GATE` 완료.
|
||||
|
||||
**Phase 완료 조건:** `P2-T1`, `P2-GATE` 완료 및 검증 기록 누적.
|
||||
|
||||
### 구현 항목
|
||||
|
||||
#### Task 2.1 `/live/room/enter` 크리에이터 제한 차단 및 성별 제한 회귀 검증
|
||||
|
||||
**Goal 실행 `P2-T1`:** `LiveRoomService.enterLive()`에서 다른 크리에이터의 제한 방 입장을 결제/roomInfo 변경 전 차단하고, 기존 성별 제한 입장 차단을 회귀 테스트로 고정한다.
|
||||
|
||||
- **시작 조건:** `LCR-003` 확정, `P1-GATE` 완료.
|
||||
- **완료 증거:** 다른 크리에이터는 예외, 성별 제한에 맞지 않는 사용자는 `live.room.gender_restricted` 예외, 방 생성자 본인과 일반 유저는 기존 정책 유지 test 통과.
|
||||
- **범위 밖:** error envelope schema 변경.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/live/room/LiveRoomService.kt`
|
||||
- Test: 기존 테스트 위치를 우선 확인하고, 없으면 `src/test/kotlin/kr/co/vividnext/sodalive/live/room/LiveRoomServiceTest.kt`
|
||||
- Optional Modify: 메시지 key 추가가 필요하다고 확인될 때만 `src/main/resources/messages*.properties` 계열 파일
|
||||
|
||||
**Implementation rule:**
|
||||
|
||||
`enterLive()`에서 room 조회 후, 결제나 `LiveRoomInfo` 변경 전에 아래 의미의 조건을 추가한다.
|
||||
|
||||
```kotlin
|
||||
if (
|
||||
member.role == MemberRole.CREATOR &&
|
||||
room.member!!.id!! != member.id!! &&
|
||||
!room.isAvailableJoinCreator
|
||||
) {
|
||||
throw SodaException(messageKey = "live.room.not_found")
|
||||
}
|
||||
```
|
||||
|
||||
기존 message key 재사용이 부적절하다고 확인되면 PRD와 plan-task를 먼저 갱신한 뒤 새 message key를 추가한다.
|
||||
|
||||
성별 제한은 현재 구현된 아래 의미의 조건을 유지한다. 이 작업에서는 해당 조건을 약화하지 말고 회귀 테스트로 고정한다.
|
||||
|
||||
```kotlin
|
||||
if (room.member!!.id!! != member.id!! && !member.canEnter(room.genderRestriction)) {
|
||||
throw SodaException(messageKey = "live.room.gender_restricted")
|
||||
}
|
||||
```
|
||||
|
||||
- [x] **RED:** 다른 크리에이터가 `isAvailableJoinCreator = false` 방에 입장하면 예외가 발생하고 `roomInfoRepository.save`와 `canPaymentService.spendCan`이 호출되지 않는 테스트를 작성한다. 성별 제한에 맞지 않는 사용자가 입장하면 `live.room.gender_restricted` 예외가 발생하는 회귀 테스트도 작성한다.
|
||||
- [x] **RED 확인:** `./gradlew test --tests '*LiveRoomServiceTest'`를 실행해 기존 구현이 입장을 허용하거나 후속 저장을 호출하는 assertion 실패를 확인한다.
|
||||
- [x] **GREEN:** `LiveRoomService.enterLive()`에 최소 조건을 추가한다.
|
||||
- [x] **GREEN 확인:** 같은 focused test를 다시 실행해 성공을 확인한다.
|
||||
- [x] **REFACTOR:** 방 생성자 본인, 일반 유저, `Gender.NONE` 사용자 회귀 테스트를 함께 실행하고 `./gradlew ktlintCheck` 결과를 Progress에 기록한다.
|
||||
|
||||
### 검증 방법
|
||||
|
||||
#### Phase 2 Gate
|
||||
|
||||
**Goal 실행 `P2-GATE`:** 입장 차단, 리스트 노출 차단, 성별 제한 전체 흐름을 최종 판정한다.
|
||||
|
||||
- **시작 조건:** `P2-T1` 완료.
|
||||
- **완료 증거:** 아래 명령 통과와 계획 문서 Progress 기록.
|
||||
- **범위 밖:** 실패 test 삭제·완화, 무관한 리팩터링.
|
||||
|
||||
```bash
|
||||
./gradlew test --tests '*DefaultHomeRecommendationQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest' --tests '*LiveRoomServiceTest'
|
||||
./gradlew ktlintCheck
|
||||
./gradlew test
|
||||
```
|
||||
|
||||
**Expected:** focused test, ktlint, 전체 test가 0 exit code로 종료한다. 크리에이터 제한과 성별 제한이 모두 조회/입장 경계에서 검증된다. 전체 test가 환경 문제로 실패하면 실패 원인과 focused/영향 범위 대체 검증을 Progress에 기록한다.
|
||||
|
||||
## Phase 3: 후속 리뷰 항목 구현
|
||||
|
||||
**근거:** `reviews/현재변경사항-review.md`의 `REV-001`, `REV-002` 및 두 항목을 후속 구현사항으로 추가하라는 사용자 요청.
|
||||
|
||||
**Phase 결과:** 제한 사용자는 방 정보 API로 토큰을 발급받지 못하고, 채널 소유자 본인의 제한 방 조회는 실제 repository 테스트로 보장된다.
|
||||
|
||||
**선행조건:** Phase 1~2 완료 및 후속 리뷰 확인. 기존 완료 체크박스와 검증 기록은 유지한다.
|
||||
|
||||
**Phase 완료 조건:** `P3-T1`, `P3-T2`, `P3-GATE` 완료와 실행 증거 기록.
|
||||
|
||||
### Task 3.1 토큰 발급 경계의 입장 제한 보강
|
||||
|
||||
- [x] **Task 3.1 완료**
|
||||
|
||||
**Goal 실행 `P3-T1`:** `REV-001`에 따라 `getRoomInfo()`에서 토큰 생성 전에 크리에이터·성별 제한을 적용하고 허용 사용자 회귀를 검증한다.
|
||||
|
||||
- **시작 조건:** `REV-001`의 제한 검사 누락 확인. 실제 RTC 우회 접속은 아직 재현되지 않았으므로 서버의 토큰 생성 호출 여부부터 검증한다.
|
||||
- **완료 증거:** 제한 사용자에 대한 실패 테스트 → 최소 수정 후 예외 key 및 RTC/RTM 생성기 무호출 검증 → 소유자·허용 사용자 회귀 통과.
|
||||
- **범위 밖:** 새로운 입장 완료 여부 검사, 성인/비공개/결제 정책 재설계, 기존 발급 토큰 회수, Agora SDK 변경, API/DB 스키마 변경.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `docs/20260917_라이브_크리에이터_입장제한/prd.md` — LCR-003 및 성공 기준에 방 정보 API의 토큰 발급 경계를 명시.
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/live/room/LiveRoomService.kt` — `getRoomInfo()`.
|
||||
- Test: `src/test/kotlin/kr/co/vividnext/sodalive/live/room/LiveRoomServiceTest.kt`.
|
||||
- Read: `src/main/kotlin/kr/co/vividnext/sodalive/live/room/LiveRoomController.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/member/Member.kt`.
|
||||
|
||||
**구현 기준:** 기존 방/roomInfo 존재 및 상호 차단 검사를 유지하고 첫 토큰 생성 호출 전에 `/enter`와 동일한 조건을 적용한다. 다른 크리에이터의 제한 방 접근은 `live.room.not_found`, 다른 사용자의 성별 불일치는 `live.room.gender_restricted`로 거절한다. 두 제한이 겹치면 크리에이터 제한을 먼저 판정한다. 생성자는 두 제한의 예외이며 성별은 `Member.canEnter()`를 재사용해 인증 성별 우선 및 `Gender.NONE` 허용 정책을 유지한다. RTC, RTM, v2v 토큰 모두 제한 검사를 통과한 뒤에만 생성한다.
|
||||
|
||||
- [x] **문서 정합성:** production 수정 전에 기존 PRD의 LCR-003, 관련 흐름·성공 기준에 토큰 발급 제한을 반영하고 후속 검증 항목은 미완료로 둔다.
|
||||
- [x] **RED:** roomInfo와 room이 존재하고 차단 관계가 없는 fixture에서 다른 크리에이터의 제한 방 접근 및 성별 불일치 접근을 테스트한다. 토큰 생성기는 테스트용 문자열을 반환하도록 준비하고 정상 응답 구성에 필요한 의존성도 설정해, 무관한 null 오류가 아닌 예상 예외 미발생으로 실패하는지 확인한다.
|
||||
- [x] **GREEN:** 위 조건을 최소 추가하고 예외 key, `rtcTokenBuilder`와 `rtmTokenBuilder`의 무호출을 검증한다. `/enter` 거절 후 `/info/{id}`에 해당하는 서비스 호출도 토큰을 생성하지 않는지 확인한다.
|
||||
- [x] **회귀:** 소유자는 크리에이터·성별 제한이 모두 걸려도 허용되고, 일반 사용자와 허용 방의 다른 크리에이터는 성별이 맞으면 응답을 받는다. 인증 성별 우선, `Gender.NONE` 허용, 기존 상호 차단 거절을 확인한다.
|
||||
- [x] **REFACTOR/기록:** 불필요한 추상화 없이 기존 스타일을 유지하고 아래 명령의 결과와 무엇을/왜/어떻게 검증했는지를 이 Task 아래에 누적한다.
|
||||
|
||||
```bash
|
||||
./gradlew test --tests '*LiveRoomServiceTest' --tests '*LiveRoomServiceAdultVisibilityPolicyTest'
|
||||
```
|
||||
|
||||
**Expected:** RED 단계는 제한 검사 누락으로 실패하고, GREEN 이후는 exit 0. 차단 경로에서는 토큰이 생성되지 않는다.
|
||||
|
||||
**검증 기록 (2026-09-17):** `LiveRoomServiceTest`에 정상 응답 fixture와 제한·허용 회귀 10건을 추가했다. RED에서 다른 크리에이터, 성별 불일치, 중첩 제한, `/enter` 거절 후 정보 조회 테스트가 모두 예상 `SodaException` 미발생으로 실패했다. `getRoomInfo()`의 기존 상호 차단 검사 뒤와 모든 RTC/RTM/v2v 토큰 생성 앞에 `/enter`와 동일한 크리에이터·성별 조건을 추가한 뒤 `./gradlew test --rerun-tasks --tests '*LiveRoomServiceTest' --tests '*LiveRoomServiceAdultVisibilityPolicyTest'`가 20건 모두 통과하며 `BUILD SUCCESSFUL`로 종료했다. 거절 경로의 token builder 무호출과 소유자·성별 일치 사용자·허용 방 크리에이터·인증 성별 우선·`Gender.NONE`·기존 상호 차단 회귀를 확인했다. 실제 HTTP/RTC 연결은 검증하지 않았다. P3-T1 리뷰 결과 Blocker는 0건이다.
|
||||
|
||||
### Task 3.2 채널 소유자 제한 방 조회 회귀 테스트
|
||||
|
||||
- [x] **Task 3.2 완료**
|
||||
|
||||
**Goal 실행 `P3-T2`:** `REV-002`에 따라 채널 소유자 본인이 `isAvailableJoinCreator = false`인 현재 방을 조회할 수 있음을 H2 repository 테스트로 고정한다.
|
||||
|
||||
- **시작 조건:** `REV-002` 확인. 기본 실행 순서는 `P3-T1` 이후이며 파일 변경은 독립적이다.
|
||||
- **완료 증거:** 실제 repository의 `findCurrentLive()` 결과가 생성한 제한 방 ID와 일치하고 기존 타인 필터 테스트도 통과한다.
|
||||
- **범위 밖:** production 조회 조건 수정, 채널 API/DTO 변경, 조회 정책 재설계.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Test: `src/test/kotlin/kr/co/vividnext/sodalive/v2/creator/channel/live/adapter/out/persistence/DefaultCreatorChannelLiveQueryRepositoryTest.kt`.
|
||||
- Read: `src/main/kotlin/kr/co/vividnext/sodalive/v2/creator/channel/live/adapter/out/persistence/DefaultCreatorChannelLiveQueryRepository.kt`.
|
||||
|
||||
**TDD 예외 사유:** 소유자 예외는 이미 구현돼 있고 이번 Task는 누락된 테스트만 추가한다. RED를 만들기 위해 정상 production 코드를 변경하지 않는다. 대체 검증은 실제 H2 조회 결과 assertion과 기존 타인 제외 테스트의 동시 통과다.
|
||||
|
||||
- [x] **테스트 추가:** 기존 fixture로 활성·비성인·유효 channelName의 현재 방을 생성하고 `isAvailableJoinCreator = false`로 설정한다. `flushAndClear()` 후 `viewerId == creatorId`, `isViewerCreator = true`, 성별 제한과 일치하는 유효 성별로 조회해 방 ID를 단정한다.
|
||||
- [x] **GREEN:** 아래 focused test로 소유자 조회와 기존 타인 제외 동작이 함께 통과하는지 확인한다.
|
||||
- [x] **REFACTOR/기록:** 기존 fixture를 재사용하고 이 Task 아래에 명령과 검증 결과를 누적한다. 예상 밖 실패는 원인을 판정한 뒤 production 수정이 필요하면 먼저 Task 범위를 갱신한다.
|
||||
|
||||
```bash
|
||||
./gradlew test --tests '*DefaultCreatorChannelLiveQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest'
|
||||
```
|
||||
|
||||
**Expected:** exit 0. 소유자에게 제한 방이 반환되고 다른 크리에이터에게는 기존대로 제외된다.
|
||||
|
||||
**검증 기록 (2026-09-17):** production 조건은 변경하지 않고 `DefaultCreatorChannelLiveQueryRepositoryTest.shouldReturnRestrictedCurrentLiveForChannelOwner`를 추가했다. 활성·비성인·유효 channelName·`isAvailableJoinCreator = false`·`MALE_ONLY` 방을 실제 H2에 저장하고 `viewerId == creatorId`, `isViewerCreator = true`, `Gender.MALE`로 조회해 생성한 방 ID를 단정했다. `./gradlew test --rerun-tasks --tests '*DefaultCreatorChannelLiveQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest'`가 기존 다른 크리에이터 제외 테스트와 함께 `BUILD SUCCESSFUL`로 종료했다. `git diff --check`도 통과했다.
|
||||
|
||||
### Phase 3 Gate
|
||||
|
||||
**Goal 실행 `P3-GATE`:** 후속 두 항목과 기존 목록·입장 정책의 회귀 검증을 완료한다.
|
||||
|
||||
- **시작 조건:** `P3-T1`, `P3-T2` 완료.
|
||||
- **완료 증거:** 아래 명령 exit 0, 토큰 생성 전 제한 검사 수동 대조, 소유자 조회 assertion 확인, PRD·리뷰·계획의 상태 및 검증 기록 갱신.
|
||||
- **범위 밖:** 실제 RTC 접속을 검증하지 않고 검증했다고 기록하는 행위, 무관한 리팩터링.
|
||||
|
||||
```bash
|
||||
./gradlew test --tests '*LiveRoomServiceTest' --tests '*LiveRoomServiceAdultVisibilityPolicyTest' --tests '*DefaultCreatorChannelLiveQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest' --tests '*DefaultHomeRecommendationQueryRepositoryTest'
|
||||
./gradlew ktlintCheck
|
||||
./gradlew tasks --all
|
||||
```
|
||||
|
||||
- [x] 위 명령을 실행하고 각 Task의 완료 증거를 대조한다.
|
||||
- [x] 실제 HTTP/RTC 연결 검증 여부와 환경 한계를 별도로 기록한다. 서비스 테스트 통과를 실제 RTC 접속 검증으로 간주하지 않는다.
|
||||
- [x] 전체 회귀는 공통 인증·예외·설정까지 변경하거나 focused test로 영향 범위를 판단할 수 없는 경우 또는 사용자 요청 시 실행한다. 생략 시 이유와 위 대체 회귀 명령을 Progress에 남긴다.
|
||||
- [x] `reviews/현재변경사항-review.md`에 수정 후 검증을 누적하고, PRD의 후속 성공 기준 및 Phase 3 상태를 실제 결과에 맞춰 갱신한다.
|
||||
|
||||
## 실행 순서와 의존성
|
||||
|
||||
| 순서 | Goal | 선행조건 | 병행 가능 | 차단 시 다음 행동 |
|
||||
|---:|---|---|---|---|
|
||||
| 1 | `P1-T1` | PRD 확정 | 아니요 | 호출부/port signature 대조 후 문서 갱신 |
|
||||
| 2 | `P1-T2` | PRD 확정 | 예, `P1-T1`과 파일 충돌 없음 | service test 작성 방식 확정 |
|
||||
| 3 | `P1-GATE` | `P1-T1`, `P1-T2` | 아니요 | 실패 소유 Task로 회귀 수정 |
|
||||
| 4 | `P2-T1` | `P1-GATE` | 아니요 | 오류 message key 결정 후 문서 갱신 |
|
||||
| 5 | `P2-GATE` | `P2-T1` | 아니요 | 실패 소유 Task로 회귀 수정 |
|
||||
| 6 | `P3-T1` | `P2-GATE`, `REV-001` | 기본 순차 | 토큰 생성 전 거절 재현과 PRD 정합성 확인 |
|
||||
| 7 | `P3-T2` | `REV-002` | `P3-T1`과 파일 독립 | 기존 fixture·현재 방 조건 확인 |
|
||||
| 8 | `P3-GATE` | `P3-T1`, `P3-T2` | 아니요 | 실패 소유 Task로 회귀 수정 |
|
||||
|
||||
```text
|
||||
P1-T1 ─┐
|
||||
├→ P1-GATE → P2-T1 → P2-GATE
|
||||
P1-T2 ─┘
|
||||
|
||||
P2-GATE → P3-T1 ─┐
|
||||
P3-T2 ─┴→ P3-GATE
|
||||
```
|
||||
|
||||
## 변경 금지 항목
|
||||
|
||||
- 공개 API 응답 DTO를 변경하지 않는다.
|
||||
- `isAvailableJoinCreator`의 DB schema나 기본값을 변경하지 않는다.
|
||||
- 기존 성인/성별/차단/강퇴/비공개/결제 조건을 완화하지 않는다.
|
||||
- 테스트를 삭제·skip·완화하지 않는다.
|
||||
- `as any`, 타입 억제, 불필요한 새 abstraction을 만들지 않는다.
|
||||
|
||||
## Progress
|
||||
|
||||
- 2026-09-17: PRD와 계획 문서 작성. 구현은 아직 시작하지 않음.
|
||||
- 2026-09-17: v2 홈 추천/온에어 라이브의 `genderRestriction` 조회 필터와 `/live/room/enter` 성별 제한 회귀 검증을 PRD/계획에 반영. 구현은 아직 시작하지 않음.
|
||||
- 2026-09-17: `P1-T1` RED 확인. 신규 인자 추가 전 `compileTestKotlin`에서 `isViewerCreator`/`effectiveViewerGender` 미정의로 실패했고, 시그니처 연결 후 repository focused test 79건 중 크리에이터 제한 1건과 성별 제한 2건이 assertion 실패했다. facade 전달 전 `HomeRecommendationControllerTest.shouldLogHomeRecommendationPageFailure`도 신규 인자 불일치로 실패했다.
|
||||
- 2026-09-17: `P1-T1` GREEN 확인. `./gradlew test --rerun-tasks --tests '*DefaultHomeRecommendationQueryRepositoryTest' --tests '*HomeRecommendationQueryServiceTest' --tests '*HomeRecommendationFacadeTest' --tests '*HomeOnAirLiveFacadeTest'`가 `BUILD SUCCESSFUL`로 종료했다. 컴파일 경고는 기존 deprecated API 및 기존 테스트 경고이며 신규 실패는 없다.
|
||||
- 2026-09-17: `P1-T1` REFACTOR 확인. 추가 추상화 없이 기존 QueryDSL 패턴을 재사용했고 `./gradlew ktlintCheck`가 `BUILD SUCCESSFUL`로 종료했다.
|
||||
- 2026-09-17: `P1-T2` RED 확인. 다른 ID의 `MemberRole.CREATOR` viewer 테스트에서 `currentLiveIsViewerCreator`가 기대값 `true`, 실제값 `false`로 실패했다.
|
||||
- 2026-09-17: `P1-T2` GREEN 확인. `CreatorChannelLiveQueryService`가 `viewer.role == MemberRole.CREATOR`를 전달하도록 최소 수정한 뒤 `./gradlew test --rerun-tasks --tests '*CreatorChannelLiveQueryServiceTest'`가 `BUILD SUCCESSFUL`로 종료했다.
|
||||
- 2026-09-17: `P1-T2` REFACTOR 확인. 별도 추상화 없이 기존 역할 enum을 사용했고 `./gradlew ktlintCheck`가 `BUILD SUCCESSFUL`로 종료했다.
|
||||
- 2026-09-17: `P1-GATE` 자동 검증. `./gradlew test --rerun-tasks --tests '*DefaultHomeRecommendationQueryRepositoryTest' --tests '*HomeRecommendationQueryServiceTest' --tests '*HomeRecommendationFacadeTest' --tests '*HomeOnAirLiveFacadeTest' --tests '*CreatorChannelLiveQueryServiceTest'`와 `./gradlew ktlintCheck`가 모두 `BUILD SUCCESSFUL`로 종료했다. 출력된 deprecated API 및 unchecked cast 경고는 변경 범위 밖의 기존 경고다.
|
||||
- 2026-09-17: `P1-GATE` 수동 대조. `HomeFollowingQueryService`와 `CreatorChannelHomeQueryService`는 변경하지 않았고, 홈 추천·라이브·온에어 facade 세 경로가 조회자 역할과 유효 성별을 전달함을 확인했다.
|
||||
- 2026-09-17: `P1-GATE` 리뷰 승인. Phase 1 전체 변경 범위의 PRD/계획 위반과 기존 필터 약화가 없음을 확인했으며 Blocker는 0건이다.
|
||||
- 2026-09-17: `P2-T1` RED 확인. 다른 크리에이터의 제한 방 입장 테스트가 `SodaException`을 기대했지만 예외가 발생하지 않아 실패했다.
|
||||
- 2026-09-17: `P2-T1` GREEN 확인. `LiveRoomService.enterLive()`의 결제 및 room-info 접근 전에 역할·방 생성자·`isAvailableJoinCreator` 조건을 추가한 뒤 `./gradlew test --rerun-tasks --tests '*LiveRoomServiceTest'`가 5건 모두 통과하며 `BUILD SUCCESSFUL`로 종료했다.
|
||||
- 2026-09-17: `P2-T1` REFACTOR 확인. 방 생성자, 일반 사용자, `Gender.NONE`, 성별 불일치 회귀 테스트와 차단 경로의 무부수효과 검증을 유지했고 `./gradlew ktlintCheck`가 `BUILD SUCCESSFUL`로 종료했다.
|
||||
- 2026-09-17: `P2-GATE` 자동 검증. `./gradlew test --rerun-tasks --tests '*DefaultHomeRecommendationQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest' --tests '*LiveRoomServiceTest'`, `./gradlew ktlintCheck`, `./gradlew test --rerun-tasks`가 모두 `BUILD SUCCESSFUL`로 종료했다.
|
||||
- 2026-09-17: `P2-GATE` 최종 리뷰 승인. 구현·테스트·문서의 PRD/계획 일치와 API/스키마 유지 여부를 재검토했으며 Blocker는 0건이다.
|
||||
- 2026-09-17: 현재 변경사항 후속 리뷰. focused 140건 및 직접 영향 회귀 42건, 총 182건 통과. `./gradlew ktlintCheck tasks --all` 성공(ktlint UP-TO-DATE). 소스 수정 없이 목록·입장 경계 중심의 검토이므로 전체 회귀는 재실행하지 않았다. 명시된 구현 범위는 적합하나, 기존 `/live/room/info/{id}` 토큰 발급 경계의 제한 누락과 채널 소유자 조회 테스트 보완점을 `reviews/현재변경사항-review.md`에 기록했다. 실제 HTTP/RTC 연결은 미검증이며, 제품 전체의 입장 차단 완결성 승인은 보류한다. 기존 완료 기록은 유지한다.
|
||||
- 2026-09-17: 사용자 요청에 따라 `REV-001`을 토큰 발급 제한 보강 `P3-T1`, `REV-002`를 채널 소유자 조회 테스트 `P3-T2`로 구체화하고 `P3-GATE`를 추가했다. 파일·조건·예외 key·회귀 시나리오·명령·완료 증거를 명시했으며 기존 Phase 1~2 완료 기록은 유지했다. `./gradlew tasks --all`은 exit 0, `BUILD SUCCESSFUL`; `git diff --check`도 통과했다. 이번 변경은 계획 문서만 갱신하므로 코드 테스트는 실행하지 않았다. 후속 구현은 미착수이며 다음 Goal은 `P3-T1`이다.
|
||||
- 2026-09-17: `P3-T1` 시작 전 문서 정합성 완료. PRD의 관련 review, 문제·목표·사용자 흐름, `LCR-003`, API 계약, 보안 기준과 미완료 성공 기준에 `/live/room/info/{id}` 토큰 발급 경계 및 `REV-002` 회귀 검증을 반영했다.
|
||||
- 2026-09-17: `P3-GATE` 자동 검증. `./gradlew test --rerun-tasks --tests '*LiveRoomServiceTest' --tests '*LiveRoomServiceAdultVisibilityPolicyTest' --tests '*DefaultCreatorChannelLiveQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest' --tests '*DefaultHomeRecommendationQueryRepositoryTest'`가 119건, 실패·오류·skip 0으로 `BUILD SUCCESSFUL` 종료했다. `./gradlew ktlintCheck`, `./gradlew tasks --all`, `git diff --check`도 모두 성공했다.
|
||||
- 2026-09-17: 토큰 발급 경계는 보안 관련 변경이므로 조건부 전체 회귀를 실행했다. `./gradlew test --rerun-tasks`가 `BUILD SUCCESSFUL`로 종료했다. 실제 HTTP 인증/직렬화 및 Agora RTC/RTM 연결은 환경상 실행하지 않았고, 실제 서비스 메서드 호출과 token builder 무호출 assertion 및 H2 repository 조회를 대체 실행 증거로 사용했다.
|
||||
- 2026-09-17: `P3-GATE` 최종 reviewer gate 승인. Phase 3 구현·테스트·PRD·계획·후속 리뷰 해소 기록을 전체 diff 기준으로 검토했으며 Blocker는 0건이다.
|
||||
@@ -0,0 +1,115 @@
|
||||
# 라이브 크리에이터 입장 제한 PRD
|
||||
|
||||
## 문서 정보
|
||||
|
||||
| 항목 | 내용 |
|
||||
|---|---|
|
||||
| 문서 상태 | Phase 1~3 구현 및 검증 완료 |
|
||||
| 작성일 | `2026-09-17` |
|
||||
| 최종 수정일 | `2026-09-17` |
|
||||
| 대상 제품 | 라이브 방 목록 노출 및 입장 제한 |
|
||||
| 작성자·결정권자 | Sisyphus / 사용자 |
|
||||
| 관련 API Contract | 기존 API 응답 스키마 변경 없음 |
|
||||
| 관련 구현 계획 | `docs/20260917_라이브_크리에이터_입장제한/plan-task.md` |
|
||||
| 관련 review | `reviews/현재변경사항-review.md` (`REV-001`, `REV-002`) |
|
||||
|
||||
## 1. Overview
|
||||
|
||||
라이브 방 생성자가 `isAvailableJoinCreator = false`로 설정한 방은 다른 크리에이터 계정에게 노출되거나 입장 가능하면 안 된다. 또한 `genderRestriction`으로 입장 가능 성별이 제한된 방은 조회 단계에서도 입장 가능한 사용자에게만 노출돼야 한다. 기존 `/live/room` 계열 조회는 대부분 이 정책을 적용하지만, v2 홈 추천/온에어 라이브와 일부 크리에이터 채널 라이브 경로에서 정책 누락 가능성이 확인됐다. 후속 리뷰에서 `/live/room/info/{id}`가 같은 제한을 확인하지 않고 RTC/RTM 토큰을 발급하는 경로도 확인됐다.
|
||||
|
||||
## 2. Problem Statement
|
||||
|
||||
- `v2` 홈 추천/온에어 라이브 조회는 `findLiveRecommendations()`에서 조회자의 크리에이터 여부와 유효 성별을 받지 않아 `isAvailableJoinCreator = false`인 방도 크리에이터에게 노출될 수 있고, 성별 제한 방도 입장 불가능한 사용자에게 노출될 수 있다.
|
||||
- `v2` 크리에이터 채널 라이브 탭은 조회자가 크리에이터인지가 아니라 조회자가 해당 채널 주인인지로만 `isViewerCreator`를 계산해, 다른 크리에이터에게 제한 방이 노출될 수 있다.
|
||||
- `/live/room/enter`는 성인, 비공개, 차단, 강퇴, 성별, 정원, 결제 조건은 확인하지만 크리에이터 입장 제한을 최종 차단하지 않는다.
|
||||
- `/live/room/info/{id}`는 방 정보와 상호 차단만 확인한 뒤 RTC/RTM 토큰을 생성하므로 `/enter`에서 거절된 사용자도 토큰 발급을 요청할 수 있다.
|
||||
|
||||
문제를 해결했다는 판단은 다른 크리에이터가 `isAvailableJoinCreator = false` 방을 리스트에서 보지 못하고, 성별 제한에 맞지 않는 사용자가 제한 방을 리스트에서 보지 못하며, 직접 `enter` 요청을 보내도 차단되는 것으로 한다.
|
||||
|
||||
## 3. Goals
|
||||
|
||||
- 다른 크리에이터 계정은 `isAvailableJoinCreator = false` 라이브 방을 문제 의심 경로에서 볼 수 없다.
|
||||
- 다른 크리에이터 계정은 `isAvailableJoinCreator = false` 라이브 방에 직접 입장할 수 없다.
|
||||
- 입장 가능 성별이 제한된 라이브 방은 성별이 맞는 사용자 또는 방 생성자 본인에게만 조회된다.
|
||||
- 입장 가능 성별이 제한된 라이브 방은 성별이 맞는 사용자 또는 방 생성자 본인만 입장할 수 있다.
|
||||
- 방을 만든 크리에이터 본인은 예외로 계속 조회 및 입장 가능하다.
|
||||
- 제한 사용자는 `/live/room/info/{id}`에서 RTC/RTM/v2v 토큰을 발급받을 수 없다.
|
||||
- 기존 응답 DTO와 공개 API 스키마는 변경하지 않는다.
|
||||
|
||||
## 4. Non-Goals
|
||||
|
||||
- `isAvailableJoinCreator` 필드명, 의미, DB 스키마 변경은 하지 않는다.
|
||||
- 일반 유저, 관리자, 봇, 에이전트의 기존 노출/입장 정책은 변경하지 않는다.
|
||||
- 기존 성인/성별/차단/강퇴/비공개/결제 정책은 재설계하지 않는다.
|
||||
- `Gender.NONE` 사용자의 기존 정책은 변경하지 않는다. 기존 `Member.canEnter()`와 리스트 조건처럼 성별 미설정 사용자는 성별 제한을 통과한다.
|
||||
- 새 API endpoint나 새 응답 필드는 만들지 않는다.
|
||||
|
||||
## 5. Target Users and Permissions
|
||||
|
||||
| 사용자 | 목표 | 정책 |
|
||||
|---|---|---|
|
||||
| 방 생성 크리에이터 | 본인이 만든 방 조회 및 입장 | `isAvailableJoinCreator = false`여도 허용 |
|
||||
| 다른 크리에이터 | 크리에이터 입장 제한 방 접근 방지 | `isAvailableJoinCreator = false`면 목록 미노출 및 입장 차단 |
|
||||
| 일반 유저 | 기존 라이브 조회 및 입장 | `genderRestriction`에 맞으면 허용, `Gender.NONE`은 기존처럼 허용 |
|
||||
|
||||
## 6. 핵심 사용자 흐름
|
||||
|
||||
1. 일반 유저 또는 방 생성자가 `isAvailableJoinCreator = false` 방을 조회하면 기존 정책에 따라 표시된다.
|
||||
2. 성별 제한에 맞지 않는 사용자가 `genderRestriction` 제한 방을 v2 홈 추천/온에어 라이브에서 조회하면 표시되지 않는다.
|
||||
3. 다른 크리에이터가 같은 방을 v2 홈 추천/온에어 라이브 또는 크리에이터 채널 라이브 경로에서 조회하면 표시되지 않는다.
|
||||
4. 성별 제한에 맞지 않는 사용자가 방 ID를 알고 `/live/room/enter`를 호출하면 입장이 거부된다.
|
||||
5. 다른 크리에이터가 방 ID를 알고 `/live/room/enter`를 호출하면 입장이 거부된다.
|
||||
6. 방 생성 크리에이터가 본인 방에 입장하면 기존처럼 허용된다.
|
||||
7. `/enter`에서 거절되는 다른 크리에이터 또는 성별 불일치 사용자가 `/live/room/info/{id}`를 호출하면 토큰 생성 전에 같은 정책으로 거절된다.
|
||||
8. 방 생성자, 성별이 맞는 일반 사용자, 크리에이터 입장이 허용된 방의 다른 크리에이터는 기존처럼 방 정보와 토큰을 받는다.
|
||||
|
||||
## 7. 기능 요구사항
|
||||
|
||||
| ID | 상태 | 요구사항 | 수용 기준 | 계약/Goal 연결 |
|
||||
|---|---|---|---|---|
|
||||
| `LCR-001` | 확정 | v2 홈 추천/온에어 라이브 조회에서 다른 크리에이터에게 `isAvailableJoinCreator = false` 방을 숨기고, 성별 제한에 맞지 않는 사용자에게 `genderRestriction` 제한 방을 숨긴다. | `HomeRecommendationQueryService.findLiveRecommendations()` 호출 경로가 조회자 크리에이터 여부와 유효 성별을 전달한다. repository는 `liveRoom.isAvailableJoinCreator.isTrue.or(liveRoom.member.id.eq(memberId))`와 `Gender.MALE -> ALL/MALE_ONLY`, `Gender.FEMALE -> ALL/FEMALE_ONLY`, `Gender.NONE/null -> 필터 없음` 조건을 적용한다. | `P1-T1` |
|
||||
| `LCR-002` | 확정 | v2 크리에이터 채널 라이브 탭에서 다른 크리에이터에게 제한 방을 숨긴다. | `CreatorChannelLiveQueryService`가 `viewer.role == MemberRole.CREATOR` 기준으로 조회자 크리에이터 여부를 전달한다. 방 생성자 본인은 예외다. | `P1-T2` |
|
||||
| `LCR-003` | 확정 | `/live/room/enter`와 `/live/room/info/{id}`에서 다른 크리에이터의 제한 방 입장·토큰 발급과 성별 제한에 맞지 않는 접근을 차단한다. | 다른 크리에이터 제한은 `live.room.not_found`, 성별 제한은 `live.room.gender_restricted`로 결제·입장 상태 변경 또는 RTC/RTM/v2v 토큰 생성 전에 거절한다. 생성자는 예외이며 성별 판정은 `Member.canEnter()`를 재사용한다. | `P2-T1`, `P3-T1` |
|
||||
| `LCR-004` | 확정 | 기존 정상 경로는 유지한다. | 일반 유저와 방 생성자 본인의 조회/입장 동작은 유지되고 기존 테스트 또는 신규 회귀 테스트로 확인된다. | `P2-GATE` |
|
||||
|
||||
## 8. API 계약
|
||||
|
||||
- 변경되는 공개 request/response 필드는 없다.
|
||||
- `/live/room/enter`는 기존 오류 envelope를 사용한다.
|
||||
- `/live/room/info/{id}`도 기존 오류 envelope와 message key를 재사용하며 응답 필드는 변경하지 않는다.
|
||||
- 신규 message key 추가 여부는 구현 단계에서 기존 `live.room.gender_restricted`, `common.error.invalid_request`, `live.room.not_found` 패턴을 확인해 결정한다. 새 key가 필요하면 메시지 리소스와 테스트를 함께 갱신한다.
|
||||
|
||||
## 9. 보안과 데이터 취급
|
||||
|
||||
- 숨김 정책은 클라이언트 표시만으로 끝내지 않고 서버 입장 경계에서 한 번 더 차단한다.
|
||||
- 제한 방 존재 여부를 다른 크리에이터에게 과도하게 노출하지 않는 오류 메시지를 우선한다.
|
||||
- 민감정보, 결제 정보, password, token은 로그나 테스트 fixture에 기록하지 않는다.
|
||||
- 제한 검사는 RTC, RTM, v2v 토큰 생성 호출보다 먼저 수행한다.
|
||||
|
||||
## 10. 성공 기준
|
||||
|
||||
- [x] 다른 크리에이터는 `isAvailableJoinCreator = false` 방을 v2 홈 추천/온에어 라이브에서 받지 않는다. (`LCR-001`)
|
||||
- [x] 성별 제한에 맞지 않는 사용자는 `genderRestriction` 제한 방을 v2 홈 추천/온에어 라이브에서 받지 않는다. (`LCR-001`)
|
||||
- [x] 다른 크리에이터는 `isAvailableJoinCreator = false` 방을 v2 크리에이터 채널 라이브 탭에서 받지 않는다. (`LCR-002`)
|
||||
- [x] 다른 크리에이터는 `isAvailableJoinCreator = false` 방에 직접 입장할 수 없다. (`LCR-003`)
|
||||
- [x] 성별 제한에 맞지 않는 사용자는 제한 방에 직접 입장할 수 없다. (`LCR-003`)
|
||||
- [x] 방 생성 크리에이터 본인은 본인 방을 조회하고 입장할 수 있다. (`LCR-001`~`LCR-003`)
|
||||
- [x] 공개 API 스키마 변경 없이 focused test와 영향 범위 회귀가 통과한다.
|
||||
- [x] 다른 크리에이터와 성별 불일치 사용자는 `/live/room/info/{id}`에서 토큰 생성 전에 거절되고 토큰 생성기가 호출되지 않는다. (`LCR-003`)
|
||||
- [x] 방 생성자, 성별이 맞는 일반 사용자, 크리에이터 입장이 허용된 방의 다른 크리에이터, `Gender.NONE` 사용자는 기존처럼 방 정보와 토큰을 받는다. (`LCR-003`, `LCR-004`)
|
||||
- [x] 채널 소유자 본인은 `isAvailableJoinCreator = false`인 현재 방을 repository 조회에서 받는다. (`LCR-002`, `LCR-004`)
|
||||
|
||||
## 11. Decision Log
|
||||
|
||||
| 일시 | 결정 | 근거 |
|
||||
|---|---|---|
|
||||
| 2026-09-17 | `isAvailableJoinCreator = false`는 다른 크리에이터만 제한하고 방 생성자 본인은 예외로 둔다. | 기존 QueryDSL 조건들이 `liveRoom.member.id.eq(viewerId)` 예외를 두는 패턴과 사용자 선택 A |
|
||||
| 2026-09-17 | 이 작업은 문서가 필요한 구현 작업으로 분류한다. | 저장소 `AGENTS.md`와 `docs/agent-guides/작업절차.md`가 모든 구현 작업 전 PRD와 plan-task를 요구 |
|
||||
| 2026-09-17 | v2 홈 추천/온에어 라이브에도 기존 조회 경로와 같은 `genderRestriction` 필터를 적용한다. | `/live/room`, 팔로잉, 크리에이터 채널 조회 경로가 이미 성별 조건을 조회 단계에서 적용하고 있으며 사용자가 문서 반영을 요청 |
|
||||
| 2026-09-17 | `/live/room/enter`의 성별 제한은 기존 `Member.canEnter()` 정책을 유지한다. | 현재 `LiveRoomService.enterLive()`가 이미 `room.member.id != member.id && !member.canEnter(room.genderRestriction)`을 차단하고, `Gender.NONE`은 `canEnter()`에서 허용됨 |
|
||||
| 2026-09-17 | `/live/room/info/{id}`에서도 `/enter`와 같은 크리에이터·성별 제한을 토큰 생성 전에 적용한다. | `REV-001`에서 `/enter` 거절 후에도 RTC/RTM/v2v 토큰 생성 경로가 열려 있음을 코드로 확인 |
|
||||
| 2026-09-17 | 채널 소유자의 제한 방 조회는 production 변경 없이 H2 repository 회귀 테스트로 고정한다. | `REV-002`와 기존 `creatorJoinLiveCondition()`의 소유자 예외 |
|
||||
|
||||
## 12. 열린 질문
|
||||
|
||||
- 없음. 구현 중 기존 메시지 key 재사용이 부적절하다고 확인되면 plan-task 범위를 먼저 갱신한다.
|
||||
@@ -0,0 +1,114 @@
|
||||
# 라이브 크리에이터 입장 제한 현재 변경사항 리뷰
|
||||
|
||||
## 1. 리뷰 정보
|
||||
|
||||
- 일자: 2026-09-17
|
||||
- 기준: `62c834d28f0d142e81f2d303273eb1220cab0611` 위의 현재 staged/working tree 변경사항. commit 자체에 대한 승인이 아니다.
|
||||
- 소스 diff SHA-256 (`git diff HEAD -- src`): `7b951c79f1575832ac65870a479005909da22e640772c5609a3cbdd983209b98`
|
||||
- 기준 문서: `../prd.md`, `../plan-task.md`
|
||||
- 범위: LCR-001~004, 변경 production 7개 파일과 관련 테스트, 기존 조회/입장 정책.
|
||||
- 상태: 판정 완료. production 및 테스트 코드는 수정하지 않았다.
|
||||
|
||||
## 2. 검토 기준 및 근거
|
||||
|
||||
- 홈 추천·라이브·온에어의 역할/인증 성별 전달, QueryDSL 필터와 생성자 예외를 대조한다.
|
||||
- 크리에이터 채널 라이브의 역할 판정과 기존 repository 조건을 대조한다.
|
||||
- 직접 입장 거부가 결제 및 roomInfo 접근보다 먼저 발생하는지 확인한다.
|
||||
- 공개 DTO, 일반 사용자, Gender.NONE, 기존 성인/차단/결제 정책의 회귀 여부를 확인한다.
|
||||
- `docs/sample/sample-review.md` 기준으로 확정 결함과 검증 한계를 구분한다.
|
||||
|
||||
## 3. 검토별 증거 기록
|
||||
|
||||
| 검토 | 기준 HEAD | 판정 | 근거 |
|
||||
|---|---|---|---|
|
||||
| 코드 품질 | `62c834d28f0d142e81f2d303273eb1220cab0611` + 현재 변경 | PASS | `/root/quality`: SQL WHERE 적용 후 페이지네이션, 호출 3곳 인자 전달, 기본값 및 생성자 예외, 부수효과 전 차단 확인. 이전 XML은 이번 실행 증거로 사용하지 않음. |
|
||||
| 요구사항 | `62c834d28f0d142e81f2d303273eb1220cab0611` + 현재 변경 | PASS | `/root/requirements`: LCR-001~004 구현 일치. 채널 소유자 조회의 직접 회귀 테스트는 보완 여지 있음. |
|
||||
| 주변 문맥 | `62c834d28f0d142e81f2d303273eb1220cab0611` + 현재 변경 | PASS | `/root/context`: 기존 팔로잉·채널 홈 조건 및 Member.canEnter와 일치. 기존 상세 조회의 제한 미검사는 이번 diff 밖임. |
|
||||
| 보안 | `62c834d28f0d142e81f2d303273eb1220cab0611` + 현재 변경 | FAIL | `/root/security`: getRoomInfo의 기존 RTC 토큰 발급 경로가 입장 제한을 검사하지 않음. 코드 추적 근거이며 실제 RTC 연결은 미검증. |
|
||||
| 자동 QA | `62c834d28f0d142e81f2d303273eb1220cab0611` + 현재 변경 | PASS | `/root/qa`: focused 140건 + 인접 회귀 42건 통과. 실제 HTTP/RTC QA는 INCONCLUSIVE. |
|
||||
| 런타임 감사 | `62c834d28f0d142e81f2d303273eb1220cab0611` + 현재 변경 | INCONCLUSIVE | H2 쿼리 결과, 서비스 거부 예외 및 무부수효과 assertion은 실행 확인. 실제 RTC 토큰을 사용한 연결 우회는 실행하지 않음. |
|
||||
|
||||
## 4. 실행 검증
|
||||
|
||||
환경: macOS, Java 17, Gradle Wrapper, repository 테스트 H2. 전체 회귀 대신 변경 파일 focused test 및 직접 영향받는 기능 회귀를 실행한다. 공개 DTO·공통 인증·설정 변경은 없으며 이 리뷰에서 소스를 수정하지 않아 전체 테스트를 재실행하지 않았다. 기존 계획의 전체 회귀 성공 기록은 이번 실행 결과로 간주하지 않는다.
|
||||
|
||||
```bash
|
||||
./gradlew test --rerun-tasks --tests '*DefaultHomeRecommendationQueryRepositoryTest' --tests '*CreatorChannelLiveQueryServiceTest' --tests '*LiveRoomServiceTest' --tests '*HomeRecommendationQueryServiceTest' --tests '*HomeRecommendationFacadeTest' --tests '*HomeOnAirLiveFacadeTest'
|
||||
```
|
||||
|
||||
- `/root/qa` 실행: BUILD SUCCESSFUL, 140건, 실패/오류/skip 0, 3분 19초.
|
||||
- `./gradlew ktlintCheck tasks --all`: BUILD SUCCESSFUL. ktlint 작업은 UP-TO-DATE이며 강제 재실행은 하지 않았다.
|
||||
- `git diff --check`: exit 0.
|
||||
- `./gradlew test --tests '*HomeRecommendationControllerTest' --tests '*DefaultCreatorChannelLiveQueryRepositoryTest' --tests '*HomeOnAirLiveControllerTest' --tests '*LiveRoomServiceAdultVisibilityPolicyTest'`: exit 0, 42건, 실패/오류/skip 0, 41초.
|
||||
- 합계: 182건 통과. 추천 repository 81, 추천 service 35, 추천 facade 6, 온에어 facade 4, 채널 service 9, 입장 service 5, 추천 controller 27, 온에어 controller 2, 채널 repository 8, 기존 성인 정책 5.
|
||||
- 실제 HTTP/RTC 연결은 미검증. localhost:8080에 실행 중 서버가 없고 운영 데이터·외부 서비스에 연결하지 않았다. 서비스 Mockito 테스트 및 H2 repository 테스트는 입장 거부/조회 필터의 실행 증거이며 실제 RTC 우회 재현 증거는 아니다.
|
||||
|
||||
## 5. 발견 사항 및 종료 판정
|
||||
|
||||
### REV-001 — 기존 방 정보 API의 토큰 발급 경계에 제한 검사 없음
|
||||
|
||||
- 심각도: High. 상태: 제한 검사 누락은 코드로 확정, 실제 RTC 접속 우회는 미검증.
|
||||
- 관련 목표: PRD §3의 다른 크리에이터·성별 불일치 사용자의 입장 차단.
|
||||
- 이번 diff가 도입한 결함은 아니며, 계획에 명시된 목록 및 `/enter` 수정 범위 밖의 기존 경로다.
|
||||
- `LiveRoomController.kt:129`의 `GET /live/room/info/{id}`는 로그인 확인 후 `getRoomInfo()`를 호출한다.
|
||||
- `LiveRoomService.kt:972`는 roomInfo·room 존재와 상호 차단만 검사한다. 크리에이터 제한, 성별 제한, 호출자의 입장 완료 여부는 확인하지 않는다.
|
||||
- 같은 파일 `:991`부터 RTC 토큰 생성기를 호출하고 `:1065`부터 채널명과 토큰을 응답에 넣는다. `RtcTokenBuilder.kt:83`은 JoinChannel 권한을 추가한다.
|
||||
- 확인 시나리오: roomInfo가 존재하는 제한 방에서, 호스트와 차단 관계가 없는 다른 크리에이터가 방 ID로 `/info/{id}`를 요청한다. `/enter`에서 거절됐더라도 이 경로의 토큰 발급 전 제한 검사가 없다.
|
||||
- 권장 조치: 토큰 발급 경계에도 생성자 예외를 포함한 크리에이터·성별 제한을 적용하고, 제한 사용자는 토큰 생성기가 호출되지 않는 회귀 테스트로 고정한다. 전체 입장 정책 재설계는 별도 범위다.
|
||||
|
||||
### REV-002 — 채널 소유자 조회의 직접 회귀 테스트 보완
|
||||
|
||||
- 심각도: Low, 비차단 검증 보완점.
|
||||
- `CreatorChannelLiveQueryServiceTest.kt:67`은 다른 크리에이터의 `isViewerCreator` 전달을 검증한다.
|
||||
- `DefaultCreatorChannelLiveQueryRepositoryTest.kt:206`은 다른 크리에이터 조회 필터를 검증하지만, 소유자 본인의 제한 방 반환을 직접 검증하지 않는다.
|
||||
- 소유자 허용 조건 자체는 `DefaultCreatorChannelLiveQueryRepository.kt:353`에 존재하므로 구현 누락으로 판정하지 않는다.
|
||||
- 권장 조치: viewerId == creatorId이고 isAvailableJoinCreator == false인 현재 방이 반환되는 repository 회귀 테스트 1건으로 P1-T2 완료 증거를 보강한다.
|
||||
|
||||
## 6. 후속 처리
|
||||
|
||||
코드 수정은 수행하지 않았다. 명시된 구현 범위의 기능 결함은 발견하지 못했으나, 제품 전체의 입장 차단 완료 판정에는 REV-001 확인·보강이 필요하다. 구현을 진행할 경우 기존 계획에 토큰 발급 경계 보강 Task를 추가한 뒤 실패 재현 → 최소 수정 → 회귀 검증 순서로 진행한다. 기존 완료 체크박스와 검증 기록은 유지한다.
|
||||
|
||||
**종합 판정:** 명시된 목록·`/enter` 구현 범위는 적합하다. 보안 검토 FAIL 및 실제 API/RTC 검증 한계 때문에 제품 전체의 입장 차단에 대한 무조건 승인은 보류한다. 이번 diff에서 새로 도입된 확정 결함은 발견하지 못했다.
|
||||
|
||||
**다음 행동:** REV-001의 토큰 발급 경계를 후속 검증·수정 범위로 반영한다.
|
||||
|
||||
## 7. Phase 3 후속 구현 검증
|
||||
|
||||
- 일자: 2026-09-17
|
||||
- 대상: `REV-001`, `REV-002` 후속 구현과 Phase 3 Gate.
|
||||
- [x] `REV-001` 해소: `LiveRoomService.getRoomInfo()`가 기존 room/roomInfo 존재 및 상호 차단 검사 뒤, RTC/RTM/v2v 토큰 생성 전에 다른 크리에이터와 성별 불일치 사용자를 각각 `live.room.not_found`, `live.room.gender_restricted`로 거절한다.
|
||||
- [x] `REV-001` 회귀: 제한 경로의 token builder 무호출, `/enter` 거절 후 정보 조회 거절, 소유자·성별 일치 사용자·허용 방 크리에이터·인증 성별 우선·`Gender.NONE` 허용, 기존 상호 차단 거절을 서비스 테스트로 확인했다.
|
||||
- [x] `REV-002` 해소: 실제 H2 repository 테스트가 채널 소유자에게 `isAvailableJoinCreator = false`인 본인 현재 방 ID를 반환하고, 기존 다른 크리에이터 제외 테스트도 통과했다.
|
||||
- [x] Phase 3 focused 회귀: 5개 테스트 클래스 119건이 실패·오류·skip 0으로 통과했다.
|
||||
- [x] 품질 및 명령 검증: `./gradlew ktlintCheck`, `./gradlew tasks --all`, `git diff --check`가 성공했다.
|
||||
- [x] 전체 회귀: 보안 경계 변경을 반영해 `./gradlew test --rerun-tasks`를 실행했고 `BUILD SUCCESSFUL`로 종료했다.
|
||||
- [x] 검증 한계 기록: 실제 HTTP 인증/직렬화 및 Agora RTC/RTM 연결은 실행하지 않았으며 서비스 메서드 호출, token builder 무호출 assertion, 실제 H2 조회를 대체 실행 증거로 사용했다.
|
||||
- [x] 최종 reviewer gate: Phase 3 구현·테스트·문서 정합성을 전체 diff 기준으로 재검토했고 Blocker 0건으로 승인됐다.
|
||||
|
||||
**후속 판정:** `REV-001`, `REV-002`는 구현 및 회귀 검증으로 해소됐다. 기존 리뷰의 보안 FAIL은 토큰 발급 전 서버 제한 검사 누락 기준에서 해소됐으며, 실제 RTC 연결 검증을 수행했다는 주장은 하지 않는다.
|
||||
|
||||
## 8. Phase 1~3 문서·코드 정적 재대조 — 2026-09-17
|
||||
|
||||
- 사용자 요청: 전체 자동 테스트는 이미 수행됐으므로 재실행하지 않고 문서와 실제 코드의 스펙 일치 여부를 확인한다.
|
||||
- 기준 HEAD: `62c834d28f0d142e81f2d303273eb1220cab0611` 및 현재 staged/working tree 변경.
|
||||
- 소스 diff SHA-256 (`git diff HEAD -- src`): `4d90f90b2a262d1ee081c8e9fc78085ce6d04199d9c50a3d5a7506ceaeed0fbe`.
|
||||
- 범위: LCR-001~004, P1~P3, REV-001/002, 호출 경로와 테스트 assertion. 이번 검토에서 테스트·애플리케이션은 실행하지 않는다. 기존 실행 성공 기록과 이번 정적 검토 판정은 구분한다.
|
||||
|
||||
| 검토 | 정적 판정 | 근거 |
|
||||
|---|---|---|
|
||||
| 요구사항 | PASS | `/root/spec_v3`: 전체 호출 3곳 역할·성별 전달, 목록 필터 및 생성자 예외, 입장/토큰 발급 전 거절, 채널 소유자 repository assertion까지 LCR-001~004 구현 확인. |
|
||||
| 보안 경계 | PASS | `/root/security_v3`: LiveRoomService.kt:986~995 검사 뒤 :1001/:1009/:1016 토큰 생성. 기존 존재·양방향 차단 검사 및 /enter의 결제 전 차단 유지. |
|
||||
| 코드 품질 | PASS | `/root/quality_v3`: 모든 호출부 인자 전달, WHERE 필터 후 페이지네이션, 내부 기본값 및 공개 API/DB 스키마 유지. |
|
||||
| 테스트 소스 | PASS | `/root/tests_v3`: LiveRoomServiceTest.kt:180~326 제한/허용/우선순위/무호출 assertion, :368 세 토큰 응답 및 호출 횟수, 채널 repository 테스트 :244~266 실제 소유자 방 ID assertion 확인. 실행 QA 판정은 아님. |
|
||||
| 문서·주변 정책 | PASS | `/root/context_v3`: Member.canEnter 및 기존 채널 조건과 일치. 이전 FAIL/미착수는 보존된 이력이며 후속 완료 기록으로 해소됨. |
|
||||
|
||||
위 판정은 명시된 HEAD와 소스 diff 조합에 한정한다.
|
||||
|
||||
**요구사항 대조:**
|
||||
|
||||
- LCR-001: 홈 추천·라이브·온에어의 역할/유효 성별 전달과 조회 필터 일치. 생성자 예외와 NONE/null 정책 유지.
|
||||
- LCR-002: 채널 조회자의 실제 CREATOR 역할 판정 및 생성자 예외 일치. REV-002의 소유자 H2 테스트 추가 확인.
|
||||
- LCR-003: /enter는 결제·상태 변경 전, /info는 모든 토큰 생성 전에 동일 제한 검사. 예외 key·우선순위·생성자 예외·Member.canEnter 재사용 일치. REV-001 해소 확인.
|
||||
- LCR-004: 허용 사용자·생성자·인증 성별 우선·Gender.NONE 회귀 assertion 존재. 공개 DTO/DB 스키마 및 기존 제한 조건 유지.
|
||||
|
||||
**최종 정적 판정: PASS.** 문서에 명시된 Phase 1~3 구현 및 후속 두 항목의 누락·스펙 위반은 발견하지 못했다. 새 수정 Task로 전환할 확정 항목은 없다. 코드·테스트는 변경하지 않았으며 이 절만 누적했다. 사용자 요청에 따라 테스트와 Gradle 작업을 재실행하지 않고 문서·소스·diff를 대조했다. 전체 자동 테스트 성공은 사용자 확인 및 기존 문서 기록으로 구분하며, 이번에 HTTP/RTC 실행 또는 과거 RED→GREEN을 재검증했다고 주장하지 않는다.
|
||||
@@ -0,0 +1,299 @@
|
||||
# 성인 콘텐츠 노출 정책 Deprecated 함수 제거 구현 계획
|
||||
|
||||
| 문서 항목 | 내용 |
|
||||
|---|---|
|
||||
| 상태 | Phase 1~3 완료, P3-R1 구현 완료·실행 검증 대기 |
|
||||
| 작성일 | `2026-09-17` |
|
||||
| 요구사항 기준 | `docs/20260917_성인콘텐츠노출정책_Deprecated함수_제거/prd.md` |
|
||||
| API 기준 | 기존 API 응답 스키마 변경 없음 |
|
||||
| 현재 Phase | Phase 3 후속 실행 검증 대기 |
|
||||
| 현재 활성 Goal | `P3-R1` 실행 검증 |
|
||||
| 다음 Goal | 없음 |
|
||||
|
||||
## 목표
|
||||
|
||||
`MemberContentPreferencePolicy.isAdultVisibleByPolicy(...)` / `resolveCountryCodeByPolicy(...)` Deprecated 경로를 `MemberContentPreferenceService` 정식 경로로 이전하고, Deprecated 함수와 전용 테스트를 제거한다. 사용자가 확정한 JP 강제 매핑의 `2L` 제거를 제외하고 성인 콘텐츠 노출 판정 결과는 유지한다.
|
||||
|
||||
## 현재 상태
|
||||
|
||||
| Phase | 상태 | 완료 Task | 활성/다음 Goal | 차단 또는 남은 조건 |
|
||||
|---:|---|---:|---|---|
|
||||
| 1 | 완료 | `1/1` | 없음 | 없음 |
|
||||
| 2 | 완료 | `4/4` | 없음 | 없음 |
|
||||
| 3 | 완료 | `1/1` | 없음 | 없음 |
|
||||
| 3 후속 | 진행 중 | `0/1` | 활성: `P3-R1` 실행 검증 | 회귀 테스트 구현·소스 검토 완료, 자동 실행 대기 |
|
||||
|
||||
## 범위
|
||||
|
||||
### 포함
|
||||
|
||||
- `MemberContentPreferenceService`에 `isAdultVisibleForQuery(member, isAdultContentVisible)` 추가.
|
||||
- production 12개 파일 27곳의 `isAdultVisibleByPolicy(...)` 호출 치환 및 `MemberContentPreferenceService` 생성자 주입.
|
||||
- `MemberContentPreferencePolicy.kt`, `MemberContentPreferencePolicyTest.kt` 제거와 고유 검증 이전.
|
||||
- 영향 범위 focused test 및 ktlint 검증.
|
||||
|
||||
### 제외
|
||||
|
||||
- `isAdult` 판정 알고리즘 변경 및 JP 강제 매핑의 `2L` 제거 외 국가 강제 매핑 회원 ID 변경.
|
||||
- 각 서비스 public 메서드 시그니처 변경, 공개 API 스키마 변경.
|
||||
- `CanController`의 통화 강제 로직.
|
||||
|
||||
## 기술적 제약
|
||||
|
||||
- 기술 스택: Kotlin, Spring Boot 2.7.14, Gradle Wrapper, ktlint.
|
||||
- 신규 abstraction을 만들지 않는다. 기존 `MemberContentPreferenceService`의 `resolveCountryCode` + `calculateIsAdultForQuery`를 재사용한다.
|
||||
- `CountryContext`는 `@RequestScope`이므로 마이그레이션 대상은 웹 요청 경로에서만 호출되는 서비스로 한정한다(`RISK-001`).
|
||||
- 모든 production 변경 Task는 `RED → GREEN → REFACTOR` 순서로 진행한다.
|
||||
- 테스트 실행은 변경 범위 focused test를 우선하고, 전체 회귀는 실행하지 않으며 생략 근거를 검증 기록에 남긴다.
|
||||
|
||||
## Phase 1: 정식 경로 단일 진입점 확보
|
||||
|
||||
**Phase 결과:** `MemberContentPreferenceService.isAdultVisibleForQuery(...)`가 Deprecated 함수와 동일한 판정을 제공한다.
|
||||
|
||||
**선행조건:** PRD 확정.
|
||||
|
||||
**Phase 완료 조건:** `P1-T1` 완료 및 검증 기록 누적.
|
||||
|
||||
### 구현 항목
|
||||
|
||||
#### Task 1.1 `isAdultVisibleForQuery` 추가
|
||||
|
||||
**Goal 실행 `P1-T1`:** `MemberContentPreferenceService`에 접속 국가 계산과 성인 노출 판정을 묶은 단일 메서드를 추가한다.
|
||||
|
||||
- **시작 조건:** `DEPREM-001` 확정.
|
||||
- **완료 증거:** KR + 미인증 → `false`, KR + 인증 → 전달값, 비KR → 전달값, 강제 매핑 회원은 헤더보다 강제 국가 우선인 focused test 통과.
|
||||
- **범위 밖:** 호출부 치환.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferenceService.kt`
|
||||
- Test: `src/test/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferenceServiceTest.kt`
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- `fun isAdultVisibleForQuery(member: Member, isAdultContentVisible: Boolean): Boolean`
|
||||
- 구현은 `calculateIsAdultForQuery(member, resolveCountryCode(member), isAdultContentVisible)`로 한다. DB 조회를 추가하지 않는다.
|
||||
|
||||
- [x] **RED:** `MemberContentPreferenceServiceTest`에 `isAdultVisibleForQuery` 판정 테스트 4건(KR 미인증/KR 인증/비KR/강제 매핑)을 작성한다.
|
||||
- [x] **RED 확인:** `MemberContentPreferenceServiceTest` 실행 결과 `Unresolved reference: isAdultVisibleForQuery` 컴파일 실패를 확인했다.
|
||||
- [x] **GREEN:** `MemberContentPreferenceService`에 `isAdultVisibleForQuery`를 추가했다.
|
||||
- [x] **GREEN 확인:** `MemberContentPreferenceServiceTest` 25/25 통과.
|
||||
- [x] **REFACTOR:** `resolveCountryCode` + `calculateIsAdultForQuery` 재사용만으로 구현해 중복 계산이 없음을 확인했고 ktlint 통과.
|
||||
|
||||
## Phase 2: production 호출부 마이그레이션
|
||||
|
||||
**Phase 결과:** production 코드에서 `isAdultVisibleByPolicy(...)` 호출이 0건이 된다.
|
||||
|
||||
**선행조건:** Phase 1 완료.
|
||||
|
||||
**Phase 완료 조건:** `P2-T1`~`P2-T4`와 `P2-GATE` 완료 및 검증 기록 누적.
|
||||
|
||||
### 구현 항목
|
||||
|
||||
#### Task 2.1 메인 탭 서비스 7개 치환
|
||||
|
||||
**Goal 실행 `P2-T1`:** 콘텐츠 메인 탭 서비스 7개가 `MemberContentPreferenceService`로 성인 노출 여부를 계산한다.
|
||||
|
||||
- **시작 조건:** `P1-T1` 완료.
|
||||
- **완료 증거:** 7개 파일에서 `isAdultVisibleByPolicy` import/호출 0건, 컴파일 및 영향 범위 테스트 통과.
|
||||
- **범위 밖:** `AudioContentService`, `ContentSeriesService`.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/tab/asmr/AudioContentMainTabAsmrService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/tab/replay/AudioContentMainTabLiveReplayService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/tab/series/AudioContentMainTabSeriesService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/tab/alarm/AudioContentMainTabAlarmService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/tab/content/AudioContentMainTabContentService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/tab/free/AudioContentMainTabFreeService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/tab/home/AudioContentMainTabHomeService.kt`
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- 각 서비스 생성자에 `private val memberContentPreferenceService: MemberContentPreferenceService`를 추가한다.
|
||||
- `isAdultVisibleByPolicy(member, isAdultContentVisible)` → `memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)`.
|
||||
- `AudioContentMainTabHomeService`의 nullable member 처리(`member?.let { ... } ?: false`)는 유지한다.
|
||||
|
||||
- [x] **RED:** 7개 서비스가 각각 대응 `*Controller`에서만 주입됨을 확인해 `RISK-001` 위반 경로가 없음을 검증했다.
|
||||
- [x] **GREEN:** 7개 파일의 호출부와 생성자를 치환했다.
|
||||
- [x] **GREEN 확인:** 테스트 컴파일과 `src/test/kotlin/kr/co/vividnext/sodalive/content` 42/42 통과로 확인했다.
|
||||
- [x] **REFACTOR:** `isAdultVisibleByPolicy` import를 `MemberContentPreferenceService` import로 교체했고 ktlint 통과.
|
||||
|
||||
#### Task 2.2 메인/큐레이션/테마 서비스 치환
|
||||
|
||||
**Goal 실행 `P2-T2`:** `AudioContentMainService`, `AudioContentCurationService`, `AudioContentThemeService`를 정식 경로로 치환한다.
|
||||
|
||||
- **시작 조건:** `P2-T1` 완료.
|
||||
- **완료 증거:** 3개 파일에서 `isAdultVisibleByPolicy` 0건, 컴파일 및 영향 범위 테스트 통과.
|
||||
- **범위 밖:** 다른 Task 대상 파일.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/AudioContentMainService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/main/curation/AudioContentCurationService.kt`
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/theme/AudioContentThemeService.kt`
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- `P2-T1`과 동일한 치환 규칙을 적용한다.
|
||||
|
||||
- [x] **RED:** 3개 서비스의 치환 대상이 모두 `member`를 받는 웹 조회 메서드임을 확인했다.
|
||||
- [x] **GREEN:** 3개 파일을 치환했다.
|
||||
- [x] **GREEN 확인:** 테스트 컴파일 성공 및 `content` 패키지 회귀 통과.
|
||||
- [x] **REFACTOR:** import 정리 후 ktlint 통과.
|
||||
|
||||
#### Task 2.3 `AudioContentService` 치환
|
||||
|
||||
**Goal 실행 `P2-T3`:** `AudioContentService`의 3곳을 정식 경로로 치환한다.
|
||||
|
||||
- **시작 조건:** `P2-T2` 완료.
|
||||
- **완료 증거:** `AudioContentServiceTest`가 갱신된 생성자로 통과한다.
|
||||
- **범위 밖:** `ContentSeriesService`.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/AudioContentService.kt`
|
||||
- Modify: `src/test/kotlin/kr/co/vividnext/sodalive/content/AudioContentServiceTest.kt`
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- `P2-T1`과 동일한 치환 규칙을 적용하고, 테스트에서는 `MemberContentPreferenceService`를 mock으로 주입한다.
|
||||
|
||||
- [x] **RED:** `AudioContentServiceTest.kt:118`에서 `No value passed for parameter 'memberContentPreferenceService'` 컴파일 실패를 확인했다.
|
||||
- [x] **GREEN:** `getDetail`, `getLatestCreatorAudioContent`, `getAudioContentList` 3곳을 치환하고 테스트에 mock을 주입했다.
|
||||
- [x] **GREEN 확인:** `AudioContentServiceTest` 16/16 통과.
|
||||
- [x] **REFACTOR:** import 정리 후 ktlint 통과.
|
||||
|
||||
#### Task 2.4 `ContentSeriesService` 치환
|
||||
|
||||
**Goal 실행 `P2-T4`:** `ContentSeriesService`의 5곳을 정식 경로로 치환한다.
|
||||
|
||||
- **시작 조건:** `P2-T3` 완료.
|
||||
- **완료 증거:** 시리즈 관련 영향 범위 테스트 통과, `isAdultVisibleByPolicy` 0건.
|
||||
- **범위 밖:** Deprecated 함수 제거.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/main/kotlin/kr/co/vividnext/sodalive/content/series/ContentSeriesService.kt`
|
||||
- Test: 필요 시 `src/test/kotlin/kr/co/vividnext/sodalive/content/series/main/SeriesMainControllerTest.kt` 등 영향 테스트 갱신
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- `P2-T1`과 동일한 치환 규칙을 적용한다.
|
||||
|
||||
- [x] **RED:** 테스트 컴파일을 실행해 `ContentSeriesService`를 직접 생성하는 테스트가 없어 추가 수정이 필요 없음을 확인했다.
|
||||
- [x] **GREEN:** 5곳(`getSeriesList`, `getSeriesListByGenre`, `getSeriesDetail`, `getSeriesContentList`, `getRecommendSeriesList`)을 치환했다.
|
||||
- [x] **GREEN 확인:** 테스트 컴파일 성공 및 Spring 컨텍스트 로딩 테스트 통과.
|
||||
- [x] **REFACTOR:** import 정리 후 ktlint 통과.
|
||||
|
||||
### Phase 2 Gate
|
||||
|
||||
**Goal 실행 `P2-GATE`:** production에서 Deprecated 호출이 0건임을 확인한다.
|
||||
|
||||
- [x] `isAdultVisibleByPolicy|resolveCountryCodeByPolicy` 검색 결과가 `src/main/kotlin`에서 0건이다.
|
||||
- [x] Phase 2 대상 파일의 영향 범위 테스트가 통과한다(`content` 42/42).
|
||||
|
||||
## Phase 3: Deprecated 함수 제거와 테스트 정리
|
||||
|
||||
**Phase 결과:** Deprecated 함수와 전용 테스트가 제거되고 고유 검증이 정식 경로 테스트에 남는다.
|
||||
|
||||
**선행조건:** Phase 2 완료.
|
||||
|
||||
**Phase 완료 조건:** `P3-T1`, `P3-GATE` 완료 및 검증 기록 누적.
|
||||
|
||||
### 구현 항목
|
||||
|
||||
#### Task 3.1 Deprecated 정의 제거와 검증 이전
|
||||
|
||||
**Goal 실행 `P3-T1`:** `MemberContentPreferencePolicy.kt`와 `MemberContentPreferencePolicyTest.kt`를 제거하고 고유 검증을 이전한다.
|
||||
|
||||
- **시작 조건:** `P2-GATE` 완료.
|
||||
- **완료 증거:** 두 파일이 삭제되고 `member/contentpreference` 테스트가 모두 통과한다.
|
||||
- **범위 밖:** `MemberContentPreferenceCountryResolver.kt` 변경.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Delete: `src/main/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferencePolicy.kt`
|
||||
- Delete: `src/test/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferencePolicyTest.kt`
|
||||
- Modify: `src/test/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferenceServiceTest.kt`
|
||||
|
||||
**Interfaces:**
|
||||
|
||||
- 이전 대상 검증: 헤더 공백/소문자 정규화(`" us "` → `US`), 국가 컨텍스트 없음 → `KR` fallback, 비KR에서 저장 `countryCode` 무시.
|
||||
- 이미 정식 경로 테스트에 존재하는 검증은 중복 추가하지 않는다.
|
||||
|
||||
- [x] **RED:** 로그인 회원의 헤더 공백/소문자 정규화와 KR fallback 검증을 `MemberContentPreferenceServiceTest`에 추가했다. 나머지 항목(비로그인 정규화, 저장 `countryCode` 미사용)은 기존 테스트가 이미 커버함을 확인해 중복 추가하지 않았다.
|
||||
- [x] **GREEN:** `MemberContentPreferencePolicy.kt`와 `MemberContentPreferencePolicyTest.kt`를 삭제했다.
|
||||
- [x] **GREEN 확인:** `member/contentpreference` 36/36 통과.
|
||||
- [x] **REFACTOR:** ktlint 통과 및 잔여 참조 0건 확인.
|
||||
|
||||
### Phase 3 Gate
|
||||
|
||||
**Goal 실행 `P3-GATE`:** 전체 정리 상태를 확인한다.
|
||||
|
||||
- [x] `isAdultVisibleByPolicy|resolveCountryCodeByPolicy` 참조가 `src/main/kotlin`, `src/test/kotlin`에서 0건이다.
|
||||
- [x] `member/contentpreference` 테스트와 Phase 2 영향 범위 테스트가 통과한다.
|
||||
- [x] `./gradlew ktlintMainSourceSetCheck ktlintTestSourceSetCheck`가 통과한다.
|
||||
- [x] PRD 성공 기준 체크박스를 갱신했다.
|
||||
|
||||
## Phase 3 후속: 리뷰 검증 보강
|
||||
|
||||
### Task 3.R1 저장 국가 무시 검증 이전
|
||||
|
||||
**Goal 실행 `P3-R1`:** 저장 국가가 KR인 미인증 회원이 US에서 조회할 때 저장 국가를 무시하는 기존 정책을 정식 서비스 테스트로 검증한다.
|
||||
|
||||
- **시작 조건:** DEPREM-004 및 `reviews/문서대비구현-review.md`의 REV-002 확정. REV-001은 사용자 확인으로 결함 판정 철회; `2L` 제거 유지.
|
||||
- **완료 증거:** 아래 테스트 추가 및 조건·assertion 소스 검토, 후속 실행이 허용된 경우 focused test 결과 기록. 실행하지 않았으면 실행 검증 대기로 남기고 완료 처리하지 않는다.
|
||||
- **범위 밖:** production 정책·resolver·매핑 변경, API 변경, 테스트 helper 리팩터링, 전체 회귀 실행.
|
||||
- **TDD 예외:** 현재 production은 올바르게 저장 국가를 무시하므로 신규 테스트는 처음부터 통과하는 것이 정상이다. 실패를 만들려고 정상 구현을 변경하지 않는다. 삭제된 테스트와 새 테스트의 입력·assertion 대조로 누락 검증 이전을 확인한다.
|
||||
- **실행 제약:** 이번 문서 갱신에서는 테스트를 실행하지 않는다. 기존 자동 테스트 미실행 지시가 유지되는 동안 후속 작업도 소스 검토까지만 진행하고 실행 상태를 별도로 남긴다.
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `src/test/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferenceServiceTest.kt`
|
||||
- Update: `docs/20260917_성인콘텐츠노출정책_Deprecated함수_제거/plan-task.md`
|
||||
- Update: `docs/20260917_성인콘텐츠노출정책_Deprecated함수_제거/reviews/문서대비구현-review.md`
|
||||
|
||||
**추가할 테스트:** 기존 `createMember`, `countryContext`, `service`, JUnit import를 재사용한다.
|
||||
|
||||
```kotlin
|
||||
@Test
|
||||
@DisplayName("비KR 요청에서는 회원의 저장 국가와 무관하게 전달한 성인 노출값을 사용한다")
|
||||
fun shouldIgnoreStoredCountryCodeWhenRequestCountryIsNotKr() {
|
||||
val member = createMember(id = 2401L, withAuth = false).apply {
|
||||
countryCode = "KR"
|
||||
}
|
||||
countryContext.setCountryCode("US")
|
||||
|
||||
assertEquals("US", service.resolveCountryCode(member))
|
||||
assertTrue(service.isAdultVisibleForQuery(member, isAdultContentVisible = true))
|
||||
assertFalse(service.isAdultVisibleForQuery(member, isAdultContentVisible = false))
|
||||
}
|
||||
```
|
||||
|
||||
- [x] **누락 확인:** 삭제된 정책 테스트와 비교해 저장 KR·요청 US·미인증 조건이 기존 정식 테스트에 없음을 확인했다(TDD 예외 적용).
|
||||
- [x] **최소 구현:** 위 테스트 1건만 추가했다. 강제 매핑이 없는 ID 2401을 사용하고 production은 변경하지 않았다.
|
||||
- [x] **소스 검토:** 국가 결과 US 및 노출 전달값 true/false 보존 assertion을 확인했다. 저장 국가 KR을 우선하면 국가 assertion과 `true` 전달값 assertion이 실패한다.
|
||||
- [ ] **실행 검증:** 자동 테스트 실행이 허용되는 후속 단계에서 아래 focused test와 ktlint를 실행하고 결과를 기록한다. 미실행이면 완료 표시하지 않는다.
|
||||
- [x] **기록 갱신:** P3-T1의 기존 완료 기록을 유지하고, REV-002 보강 결과와 실행 대기 상태를 본 Task와 리뷰에 누적했다.
|
||||
|
||||
**후속 실행용 명령(이번에는 미실행):** 저장소와 호환되는 JDK 환경에서 실행한다.
|
||||
|
||||
```bash
|
||||
./gradlew test --tests 'kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceServiceTest'
|
||||
./gradlew ktlintTestSourceSetCheck
|
||||
```
|
||||
|
||||
기대 결과: 신규 테스트를 포함한 서비스 테스트 및 테스트 소스 ktlint 성공. 테스트 한 파일의 검증 보강이므로 전체 회귀는 생략한다.
|
||||
|
||||
## 검증 기록
|
||||
|
||||
- 2026-09-17 Phase 3 후속 P3-R1: 삭제된 정책 테스트와 현재 서비스 테스트를 대조해 저장 KR·요청 US·미인증 조건의 누락을 확인하고 `MemberContentPreferenceServiceTest.shouldIgnoreStoredCountryCodeWhenRequestCountryIsNotKr` 1건을 추가했다. 국가 결과 US와 전달값 true/false 보존 assertion을 소스로 검토했다. production은 변경하지 않았으며 기존 자동 테스트 미실행 지시에 따라 focused test와 ktlint는 실행하지 않아 Task 완료 처리는 보류했다.
|
||||
- 2026-09-17 리뷰 후속 문서 갱신: 사용자가 JP 매핑의 `2L` 제거를 의도한 변경으로 확정했다. REV-002 검증 누락은 `P3-R1`로 추가하고 REV-003 호출 수는 27곳으로 정정했다. 문서와 테스트 소스의 식별자·조건을 대조했으며 사용자 지시에 따라 자동 테스트 및 Gradle 명령은 실행하지 않았다.
|
||||
|
||||
- 2026-09-17 Phase 1: `MemberContentPreferenceServiceTest` 실행 → RED에서 `Unresolved reference: isAdultVisibleForQuery` 확인, GREEN 후 25/25 통과. 검증 이유: 신규 판정 메서드가 Deprecated 함수와 동일한 결과를 내는지 확인.
|
||||
- 2026-09-17 Phase 2: `AudioContentServiceTest.kt:118` 컴파일 실패로 생성자 변경 영향을 확인한 뒤 mock 주입. `src/test/kotlin/kr/co/vividnext/sodalive/content` 42/42 통과, `AudioContentServiceTest` 16/16 통과.
|
||||
- 2026-09-17 Phase 3: `src/test/kotlin/kr/co/vividnext/sodalive/member/contentpreference` 36/36 통과(Deprecated 테스트 5건 삭제, 신규 3건 추가 반영).
|
||||
- 2026-09-17 빈 주입 회귀: `SpringBootIntegrationSampleTest` 1/1 통과로 12개 서비스에 추가된 `MemberContentPreferenceService` 주입이 애플리케이션 컨텍스트에서 정상 해석됨을 확인(순환 의존 없음).
|
||||
- 2026-09-17 스타일: `JAVA_HOME=<Android Studio JBR 21> ./gradlew ktlintMainSourceSetCheck ktlintTestSourceSetCheck` BUILD SUCCESSFUL. JetBrains JBR 25는 Gradle 8.1.1과 비호환이어서 Java 21 JBR을 사용했다.
|
||||
- 2026-09-17 전체 회귀 생략: `./gradlew test` 전체 회귀는 실행하지 않았다. 근거는 변경이 `isAdult` 계산 위임 경로 치환으로 한정되고 공개 스키마/DB/보안 설정 변경이 없으며, 대신 변경 범위 focused test(`content` 42, `member/contentpreference` 36, `AudioContentServiceTest` 16)와 컨텍스트 로딩 테스트, ktlint 전체 소스셋 검사를 실행했다.
|
||||
@@ -0,0 +1,132 @@
|
||||
# 성인 콘텐츠 노출 정책 Deprecated 함수 제거 PRD
|
||||
|
||||
## 문서 정보
|
||||
|
||||
| 항목 | 내용 |
|
||||
|---|---|
|
||||
| 문서 상태 | Phase 1~3 완료, 리뷰 후 고유 검증 보강 대기 |
|
||||
| 작성일 | `2026-09-17` |
|
||||
| 최종 수정일 | `2026-09-17` |
|
||||
| 대상 제품 | 성인 콘텐츠 노출 여부 / 접속 국가 코드 계산 정책 |
|
||||
| 작성자·결정권자 | Junie / 사용자 |
|
||||
| 관련 API Contract | 기존 API 응답 스키마 변경 없음 |
|
||||
| 관련 구현 계획 | `docs/20260917_성인콘텐츠노출정책_Deprecated함수_제거/plan-task.md` |
|
||||
| 관련 review | `reviews/문서대비구현-review.md` |
|
||||
|
||||
## 1. Overview
|
||||
|
||||
성인 콘텐츠 노출 여부(`isAdult`)와 접속 국가 코드 계산 정책은 현재 두 경로에 중복 구현되어 있다. 하나는 `@Deprecated`로 표시된 top-level 함수 `MemberContentPreferencePolicy.isAdultVisibleByPolicy(...)` / `resolveCountryCodeByPolicy(...)`이고, 다른 하나는 정식 경로인 `MemberContentPreferenceService`다. Deprecated 경로가 production 12개 파일에서 여전히 27곳 호출되고 있어 정책 변경 시 두 곳을 모두 고쳐야 하는 위험이 남아 있다. 이 작업은 Deprecated 경로 호출부를 `MemberContentPreferenceService`로 이전하고 Deprecated 함수와 그 전용 테스트를 제거한다.
|
||||
|
||||
## 2. Problem Statement
|
||||
|
||||
- `MemberContentPreferencePolicy.kt`의 두 함수는 `docs/20260623_메인_콘텐츠_추천_탭_API/plan-task.md`에서 `@Deprecated` 처리만 되고 호출부 이전은 완료되지 않았다.
|
||||
- 접속 국가 판정 기준이 두 경로에서 다르게 조달된다. Deprecated 경로는 `RequestContextHolder`에서 `CloudFront-Viewer-Country` 헤더를 직접 읽고, 정식 경로는 `CountryContext`(요청 스코프)에서 읽는다. 값 출처(같은 헤더, `CountryInterceptor`가 주입)는 같지만 경로가 둘이다.
|
||||
- Deprecated 함수의 유일한 테스트인 `MemberContentPreferencePolicyTest`의 5개 검증은 `MemberContentPreferenceServiceTest` / `MemberContentPreferenceIntegrationTest`에 이미 동등하게 존재해, 회원 ID 강제 매핑 정책을 변경할 때마다 같은 상수를 3~4개 파일에서 함께 고쳐야 한다.
|
||||
- 결과적으로 정책 변경(예: 국가 강제 매핑 회원 ID 추가/삭제)이 중복 지점 누락으로 회귀를 만들 수 있다.
|
||||
|
||||
문제를 해결했다는 판단은 `isAdultVisibleByPolicy` / `resolveCountryCodeByPolicy` 참조가 `src/main/kotlin`과 `src/test/kotlin`에서 0건이 되고, 기존 legacy 콘텐츠 조회 API의 `isAdult` 계산 결과가 그대로 유지되는 것으로 한다.
|
||||
|
||||
## 3. Goals
|
||||
|
||||
- 성인 콘텐츠 노출 여부 계산의 단일 진입점을 `MemberContentPreferenceService`로 만든다.
|
||||
- `MemberContentPreferencePolicy.kt`의 Deprecated 함수 2개와 파일, 전용 테스트를 제거한다.
|
||||
- 기존 legacy 콘텐츠 조회 API의 `isAdult` / 국가 코드 계산 동작을 유지하되, 사용자가 확정한 JP 강제 매핑의 `2L` 제거는 예외로 반영한다.
|
||||
- 공개 API request/response 스키마를 변경하지 않는다.
|
||||
|
||||
## 4. Non-Goals
|
||||
|
||||
- `isAdult` 판정 정책 자체(KR + 본인인증 여부, 해외 전달값 사용)를 변경하지 않는다.
|
||||
- JP 강제 매핑의 `2L` 제거 외에는 회원 ID 국가 강제 매핑(`FORCED_KR_MEMBER_IDS`, `FORCED_JP_MEMBER_IDS`) 값을 변경하지 않는다.
|
||||
- `isAdultContentVisible`을 서비스 파라미터에서 제거하거나 저장값 조회로 대체하는 리팩터링은 하지 않는다. 호출부 시그니처는 유지한다.
|
||||
- `CanController`의 통화(currency) 강제 지정 로직은 국가 강제 매핑과 무관하므로 건드리지 않는다.
|
||||
- `MemberContentPreferenceCountryResolver.kt`의 강제 매핑 알고리즘은 유지하고, JP 대상에서 `2L`만 제외한다.
|
||||
|
||||
## 5. 영향 범위
|
||||
|
||||
### 5.1 Deprecated 함수 정의
|
||||
|
||||
| 파일 | 대상 |
|
||||
|---|---|
|
||||
| `src/main/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferencePolicy.kt` | `resolveCountryCodeByPolicy(member)`, `isAdultVisibleByPolicy(member, isAdultContentVisible)` |
|
||||
|
||||
### 5.2 production 호출부
|
||||
|
||||
| 파일 | 호출 수 |
|
||||
|---|---:|
|
||||
| `content/series/ContentSeriesService.kt` | 5 |
|
||||
| `content/AudioContentService.kt` | 3 |
|
||||
| `content/main/AudioContentMainService.kt` | 2 |
|
||||
| `content/main/curation/AudioContentCurationService.kt` | 2 |
|
||||
| `content/theme/AudioContentThemeService.kt` | 2 |
|
||||
| `content/main/tab/free/AudioContentMainTabFreeService.kt` | 4 |
|
||||
| `content/main/tab/alarm/AudioContentMainTabAlarmService.kt` | 2 |
|
||||
| `content/main/tab/content/AudioContentMainTabContentService.kt` | 2 |
|
||||
| `content/main/tab/home/AudioContentMainTabHomeService.kt` | 2 |
|
||||
| `content/main/tab/asmr/AudioContentMainTabAsmrService.kt` | 1 |
|
||||
| `content/main/tab/replay/AudioContentMainTabLiveReplayService.kt` | 1 |
|
||||
| `content/main/tab/series/AudioContentMainTabSeriesService.kt` | 1 |
|
||||
|
||||
12개 파일, 총 27곳이다. 어떤 파일도 현재 `MemberContentPreferenceService`를 주입받지 않는다.
|
||||
|
||||
### 5.3 테스트 호출부
|
||||
|
||||
| 파일 | 처리 |
|
||||
|---|---|
|
||||
| `src/test/kotlin/.../member/contentpreference/MemberContentPreferencePolicyTest.kt` | 삭제. 동등 검증이 `MemberContentPreferenceServiceTest`, `MemberContentPreferenceIntegrationTest`에 이미 존재하며, 부족한 항목은 `MemberContentPreferenceServiceTest`로 이전한다. |
|
||||
|
||||
## 6. 기능 요구사항
|
||||
|
||||
| ID | 상태 | 요구사항 | 수용 기준 | 계약/Goal 연결 |
|
||||
|---|---|---|---|---|
|
||||
| `DEPREM-001` | 확정 | `MemberContentPreferenceService`에 `isAdultVisibleForQuery(member, isAdultContentVisible)`를 추가해 `resolveCountryCode(member)` + `calculateIsAdultForQuery(...)` 조합을 단일 메서드로 제공한다. | KR + `auth == null`이면 `false`, KR + `auth != null`이면 전달값, 비KR이면 전달값. 회원 ID 강제 매핑이 접속 국가 헤더보다 우선한다. | `P1-T1` |
|
||||
| `DEPREM-002` | 확정 | production 12개 파일의 `isAdultVisibleByPolicy(...)` 27곳을 `memberContentPreferenceService.isAdultVisibleForQuery(...)`로 치환하고 `MemberContentPreferenceService`를 생성자 주입한다. | 각 파일에서 `isAdultVisibleByPolicy` import와 호출이 사라지고, `isAdult` 계산 결과가 기존과 동일하다. | `P2-T1`~`P2-T4` |
|
||||
| `DEPREM-003` | 확정 | `MemberContentPreferencePolicy.kt`와 `MemberContentPreferencePolicyTest.kt`를 제거한다. | `isAdultVisibleByPolicy|resolveCountryCodeByPolicy` 검색 결과가 `src/main/kotlin`, `src/test/kotlin`에서 0건이다. | `P3-T1` |
|
||||
| `DEPREM-004` | 확정 | Deprecated 테스트에서 제거되는 검증 항목 중 정식 경로 테스트에 없는 것은 `MemberContentPreferenceServiceTest`로 이전한다. | 헤더 공백/소문자 정규화(`" us "` → `US`), 국가 컨텍스트 없음 → `KR` fallback, 비KR에서 저장 `countryCode` 무시 검증이 정식 경로 테스트에 존재한다. | `P3-T1` |
|
||||
| `DEPREM-005` | 확정 | 기존 legacy 콘텐츠 조회 동작을 회귀 없이 유지한다. | `AudioContentServiceTest` 등 영향 범위 테스트와 `member/contentpreference` 테스트가 모두 통과하고 ktlint가 통과한다. | `P3-GATE` |
|
||||
|
||||
## 7. API 계약
|
||||
|
||||
- 변경되는 공개 request/response 필드는 없다.
|
||||
- 신규 message key, 신규 endpoint는 없다.
|
||||
- 각 서비스의 public 메서드 시그니처(`isAdultContentVisible` 파라미터 포함)는 유지한다. 변경되는 것은 생성자 의존성뿐이다.
|
||||
|
||||
## 8. 보안과 데이터 취급
|
||||
|
||||
- 접속 국가 판정은 계속 `CloudFront-Viewer-Country` 헤더를 `CountryInterceptor`가 `CountryContext`에 주입한 값만 사용한다.
|
||||
- 성인 콘텐츠 노출은 KR에서 본인인증(`member.auth != null`)이 있는 경우에만 허용하는 기존 정책을 유지한다.
|
||||
- 회원 ID, 이메일 등 개인정보는 로그에 추가하지 않는다.
|
||||
|
||||
## 9. 리스크
|
||||
|
||||
| ID | 리스크 | 대응 |
|
||||
|---|---|---|
|
||||
| `RISK-001` | `CountryContext`는 `@RequestScope`이므로 비웹 스레드에서 호출되면 예외가 발생할 수 있다. Deprecated 함수는 `RequestContextHolder`가 없을 때 `KR`로 fallback했다. | 대상 12개 서비스가 controller 요청 경로에서만 호출되는지 Phase 2에서 파일별로 확인한다. 스케줄러/비동기 호출 경로가 발견되면 해당 Task를 진행하지 않고 계획을 먼저 갱신한다. |
|
||||
| `RISK-002` | 생성자 파라미터 추가로 기존 서비스 테스트가 컴파일 실패할 수 있다. | 해소됨. `AudioContentServiceTest` 1곳만 영향받아 mock 주입으로 해결했다. |
|
||||
| `RISK-003` | 순환 의존 발생 가능성. | `MemberContentPreferenceService`는 `MemberContentPreferenceRepository`, `MemberRepository`, `CountryContext`, `CacheManager`만 의존하므로 콘텐츠 서비스와 순환이 발생하지 않음을 확인했다. |
|
||||
|
||||
## 10. 성공 기준
|
||||
|
||||
- [x] `MemberContentPreferenceService.isAdultVisibleForQuery(...)`가 기존 `isAdultVisibleByPolicy(...)`와 동일한 판정을 한다. (`DEPREM-001`)
|
||||
- [x] production 12개 파일 27곳이 모두 정식 경로를 호출한다. (`DEPREM-002`)
|
||||
- [x] `MemberContentPreferencePolicy.kt`, `MemberContentPreferencePolicyTest.kt`가 제거되고 참조가 0건이다. (`DEPREM-003`)
|
||||
- [x] Deprecated 테스트의 고유 검증이 `MemberContentPreferenceServiceTest`에 남아 있다. (`DEPREM-004`)
|
||||
- [x] 영향 범위 focused test와 ktlint가 통과한다. (`DEPREM-005`)
|
||||
|
||||
## 11. Decision Log
|
||||
|
||||
기존 성공 기준 체크박스는 당시 완료 기록으로 유지한다. 후속 리뷰에서 DEPREM-004의 저장 국가 충돌 검증 누락을 확인했으며, `plan-task.md`의 `P3-R1` 완료 전까지 해당 검증 보강은 미완료다.
|
||||
|
||||
| 일시 | 결정 | 근거 |
|
||||
|---|---|---|
|
||||
| 2026-09-17 | Deprecated 함수 제거를 위해 production 호출부를 모두 마이그레이션하는 방식(선택지 3)을 택한다. | 사용자 선택. 테스트만 정식 경로로 바꾸면 production에서 쓰이는 Deprecated 경로가 커버리지 0이 되므로 부적절 |
|
||||
| 2026-09-17 | 호출부 1:1 치환을 위해 `MemberContentPreferenceService`에 `isAdultVisibleForQuery(member, isAdultContentVisible)` wrapper를 추가한다. | 호출부마다 `resolveCountryCode(...)` + `calculateIsAdultForQuery(...)` 2단계를 반복하면 중복이 27곳으로 늘어남 |
|
||||
| 2026-09-17 | `MemberContentPreferencePolicyTest`는 삭제하고 고유 검증만 `MemberContentPreferenceServiceTest`로 이전한다. | 5개 검증 중 대부분이 정식 경로 테스트와 중복이며, 함수 제거 후에는 테스트 대상이 사라짐 |
|
||||
| 2026-09-17 | Phase를 서비스 묶음 단위로 나눠 진행한다. | 27곳 동시 변경 시 실패 원인 추적이 어려움. 저장소 규칙("작은 단위로 안전하게 수정") 준수 |
|
||||
| 2026-09-17 | `RISK-001`은 해소로 판정한다. | 치환한 27곳이 모두 controller 진입 조회 경로임을 파일별로 확인했고, 스케줄러(`Recommendation/Ranking/ChargeEvent`)는 이 경로를 호출하지 않음 |
|
||||
|
||||
- 2026-09-17 사용자 확정: JP 강제 매핑에서 `2L`을 제거한 것은 의도된 정책 변경이다. REV-001은 결함 판정을 철회하며 `2L`을 복원하지 않는다. 나머지 매핑은 유지한다.
|
||||
- 2026-09-17 후속 계획: REV-002의 저장 국가 무시 검증을 `P3-R1`로 추가한다. 이번 작업은 문서 갱신이며 테스트 구현·실행은 하지 않는다.
|
||||
|
||||
## 12. 열린 질문
|
||||
|
||||
- 없음. Phase 2에서 비웹 호출 경로(`RISK-001`)가 발견되면 계획을 먼저 갱신하고 사용자에게 확인한다.
|
||||
@@ -0,0 +1,65 @@
|
||||
# 문서 대비 구현 리뷰
|
||||
|
||||
## 리뷰 정보와 범위
|
||||
|
||||
- 일자: 2026-09-17
|
||||
- 기준: HEAD `8c87286acd69174f6024d30a267458e838e35cf5` 및 리뷰 시작 시 존재한 미커밋 변경
|
||||
- 요구사항: `../prd.md`, `../plan-task.md`의 Phase 1~3
|
||||
- 방법: 문서, production 코드, 테스트 소스, 현재 diff와 호출 경로 정적 대조
|
||||
- 사용자 지시에 따라 자동 테스트·빌드·ktlint·애플리케이션 실행은 하지 않았다. 기존 실행 성공 기록을 재검증한 것으로 취급하지 않는다.
|
||||
- 구현 및 기존 PRD/계획은 수정하지 않았다. 아래 판정은 현재 작업 트리에 대한 것이며 변경 작성자나 의도를 추정하지 않는다.
|
||||
|
||||
## 발견 사항
|
||||
|
||||
### REV-001 — 강제 JP 매핑 값이 변경됨
|
||||
|
||||
- 심각도: High / 상태: 확정
|
||||
- 요구사항: PRD §3 기존 동작 유지, §4 강제 매핑 값 변경 금지; 계획 P3-T1의 resolver 변경 제외
|
||||
- 근거: `src/main/kotlin/kr/co/vividnext/sodalive/member/contentpreference/MemberContentPreferenceCountryResolver.kt:6`에서 `FORCED_JP_MEMBER_IDS`의 `2L`이 제거됐다.
|
||||
- 코드 추적: 회원 ID 2, 인증 없음, 요청 국가 KR 또는 헤더 없음, `isAdultContentVisible=true`이면 기존 JP 판정/성인 노출 true에서 KR 판정/false로 바뀐다.
|
||||
- 관련 테스트도 `MemberContentPreferenceServiceTest.kt:64`, `MemberContentPreferenceIntegrationTest.kt:187`에서 ID 2를 29721로 교체해 기존 매핑 회귀를 검출하지 못한다.
|
||||
- 권장 조치: 문서대로라면 기존 매핑과 해당 테스트 대상을 복원한다. 별도로 의도한 정책 변경이라면 해당 결정과 범위를 문서에 명시해야 한다.
|
||||
|
||||
### REV-002 — 저장 국가 무시 검증이 이전되지 않음
|
||||
|
||||
- 심각도: Medium / 상태: 확정
|
||||
- 요구사항: DEPREM-004, P3-T1
|
||||
- 근거: 삭제된 PolicyTest는 저장 `member.countryCode="KR"`, 요청 국가 US 조건을 검증했다. 현재 `MemberContentPreferenceServiceTest.kt:469`의 해외 노출 테스트와 `:536`의 회원 생성 함수는 저장 국가를 설정하지 않는다. IntegrationTest에도 이 충돌 조건이 없다.
|
||||
- 영향: 현재 resolver는 저장 국가를 참조하지 않아 구현 자체는 맞지만, 문서에서 이전하도록 요구한 고유 회귀 검증이 빠졌다. `plan-task.md:224`의 기존 테스트가 이미 커버한다는 기록은 부정확하다.
|
||||
- 권장 조치: 일반 미인증 회원의 저장 국가를 KR로 설정하고 요청 국가는 US로 두어, 정식 진입점의 국가 US/성인 노출 true를 확인하는 검증을 추가한다.
|
||||
|
||||
### REV-003 — 호출부 합계 오기
|
||||
|
||||
- 심각도: Low / 상태: 확정
|
||||
- 근거: PRD §5.2 표의 합계와 실제 치환은 모두 12개 파일 27곳이다. PRD와 계획에는 반복해서 24곳으로 기록돼 있다.
|
||||
- 영향: 호출 치환 누락은 없으며 문서 수치가 틀렸다.
|
||||
- 권장 조치: 두 문서의 합계를 27곳으로 정정한다.
|
||||
|
||||
## 충족 항목과 한계
|
||||
|
||||
| 항목 | 정적 검토 결과 |
|
||||
|---|---|
|
||||
| DEPREM-001 단일 진입점 | 기존 resolveCountryCode + calculateIsAdultForQuery 재사용, 추가 DB 조회 없음 |
|
||||
| DEPREM-002 호출부 이전 | 12개 파일 27곳 이전, 인자 및 홈의 member=null → false 유지 |
|
||||
| DEPREM-003 정의/참조 제거 | 두 파일 삭제, src/main/kotlin 및 src/test/kotlin의 두 Deprecated 함수 이름 검색 0건 |
|
||||
| DEPREM-004 고유 검증 이전 | 정규화/null 국가값 fallback은 존재, 저장 국가 충돌 검증 누락 |
|
||||
| DEPREM-005 실행 검증 | 사용자 요청에 따라 재실행 제외 |
|
||||
| API 계약 | 기존 공개 메서드 인자와 request/response 스키마 변경 없음 |
|
||||
| 요청 경로/의존성 | 변경된 조회 메서드의 비웹 호출 및 새 순환 의존 발견 없음 |
|
||||
|
||||
국가값 null 검증은 요청 스코프 자체가 없는 상황과 다르다. 새 경로는 요청 스코프와 회원 ID를 요구하므로 모든 입력에서 구 함수와 동등하다고 일반화할 수 없다. 다만 검토한 변경 호출 경로에서 해당 비웹/ID 없는 입력은 확인하지 못해 별도 운영 결함으로 확정하지 않았다.
|
||||
|
||||
## 리뷰 종료 판정과 후속 작업
|
||||
|
||||
요구사항, 테스트 소스, 코드 품질, 보안, 주변 호출 맥락의 검토 결과를 종합했다. 확정 불일치 3건으로 문서와 완전히 일치하는 구현이라고 판정할 수 없다.
|
||||
|
||||
이번 요청은 검토이므로 수정 goal은 실행하지 않았다. 후속 구현 시 기존 완료 기록을 유지하면서 REV-001 매핑 일치, REV-002 고유 검증 이전, REV-003 문서 수치 정정을 별도 회귀 Task로 계획에 추가한다. 자동 검증은 이번 검토에서 수행하지 않았다.
|
||||
|
||||
## 후속 판정 정정 — 2026-09-17
|
||||
|
||||
위 내용은 최초 검토 당시 기록이며, 사용자 확인 이후 현재 판정은 다음과 같다.
|
||||
|
||||
- **REV-001: 결함 판정 철회.** 사용자가 `2L` 제거를 의도한 변경으로 확정했다. PRD·계획에 해당 예외를 반영했으며 매핑과 테스트 대상을 복원하지 않는다.
|
||||
- **REV-002: 구현 완료 / 실행 검증 대기.** `MemberContentPreferenceServiceTest.shouldIgnoreStoredCountryCodeWhenRequestCountryIsNotKr`를 추가해 저장 KR·요청 US·미인증 회원 조건에서 국가 결과 US와 전달값 true/false 보존을 검증한다. production은 변경하지 않았고, 기존 자동 테스트 미실행 지시에 따라 focused test와 ktlint는 실행하지 않았다.
|
||||
- **REV-003: 문서 정정 완료.** PRD·계획의 24곳 표기를 실제 합계 27곳으로 정정했다.
|
||||
- **현재 결론:** `P3-R1` 테스트 구현과 소스 검토는 완료했으며, 남은 항목은 focused test와 ktlint 실행 검증이다.
|
||||
@@ -38,7 +38,7 @@ import kr.co.vividnext.sodalive.i18n.translation.ResourceTranslationJobScheduler
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberRepository
|
||||
import kr.co.vividnext.sodalive.member.block.BlockMemberRepository
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.utils.generateFileName
|
||||
import kr.co.vividnext.sodalive.v2.home.following.application.HomeFollowingNewsPublishService
|
||||
import kr.co.vividnext.sodalive.v2.recommendation.application.CreatorContentViewHistoryService
|
||||
@@ -90,6 +90,7 @@ class AudioContentService(
|
||||
|
||||
private val contentThemeTranslationRepository: ContentThemeTranslationRepository,
|
||||
private val homeFollowingNewsPublishService: HomeFollowingNewsPublishService,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService,
|
||||
|
||||
@Value("\${cloud.aws.s3.content-bucket}")
|
||||
private val audioContentBucket: String,
|
||||
@@ -616,7 +617,7 @@ class AudioContentService(
|
||||
isAdultContentVisible: Boolean,
|
||||
timezone: String
|
||||
): GetAudioContentDetailResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
|
||||
// 오디오 콘텐츠 조회 (content_id, 제목, 내용, 테마, 태그, 19여부, 이미지, 콘텐츠 PATH)
|
||||
val audioContent = repository.findByIdOrNull(id)
|
||||
@@ -977,7 +978,7 @@ class AudioContentService(
|
||||
member: Member,
|
||||
isAdultContentVisible: Boolean
|
||||
): GetAudioContentListItem? {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
|
||||
if (isBlockedBetweenMembers(memberId = member.id!!, creatorId = creatorId)) {
|
||||
return null
|
||||
@@ -1051,7 +1052,7 @@ class AudioContentService(
|
||||
offset: Long,
|
||||
limit: Long
|
||||
): GetAudioContentListResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val isCreator = member.id == creatorId
|
||||
|
||||
if (!isCreator && isBlockedBetweenMembers(memberId = member.id!!, creatorId = creatorId)) {
|
||||
|
||||
@@ -13,7 +13,7 @@ import kr.co.vividnext.sodalive.event.EventItem
|
||||
import kr.co.vividnext.sodalive.i18n.LangContext
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.block.BlockMemberRepository
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import org.springframework.beans.factory.annotation.Value
|
||||
import org.springframework.cache.annotation.Cacheable
|
||||
import org.springframework.data.domain.Pageable
|
||||
@@ -33,6 +33,8 @@ class AudioContentMainService(
|
||||
|
||||
private val langContext: LangContext,
|
||||
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService,
|
||||
|
||||
@Value("\${cloud.aws.cloud-front.host}")
|
||||
private val imageHost: String
|
||||
) {
|
||||
@@ -69,7 +71,7 @@ class AudioContentMainService(
|
||||
} else {
|
||||
emptyList()
|
||||
},
|
||||
isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
contentType = contentType,
|
||||
offset = pageable.offset,
|
||||
limit = pageable.pageSize.toLong()
|
||||
@@ -88,7 +90,7 @@ class AudioContentMainService(
|
||||
* - AS-IS theme은 한글만 처리하도록 되어 있음
|
||||
* - TO-BE 번역된 theme이 들어와도 동일한 동작을 하도록 처리
|
||||
*/
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val themeListRaw = if (theme.isBlank()) {
|
||||
audioContentThemeRepository.getActiveThemeOfContent(
|
||||
isAdult = isAdult,
|
||||
|
||||
+4
-3
@@ -6,7 +6,7 @@ import kr.co.vividnext.sodalive.content.SortType
|
||||
import kr.co.vividnext.sodalive.content.main.tab.GetContentCurationResponse
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.block.BlockMemberRepository
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import org.springframework.beans.factory.annotation.Value
|
||||
import org.springframework.data.domain.Pageable
|
||||
import org.springframework.stereotype.Service
|
||||
@@ -17,6 +17,7 @@ class AudioContentCurationService(
|
||||
private val repository: AudioContentCurationQueryRepository,
|
||||
private val contentRepository: AudioContentRepository,
|
||||
private val blockMemberRepository: BlockMemberRepository,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService,
|
||||
|
||||
@Value("\${cloud.aws.cloud-front.host}")
|
||||
private val cloudFrontHost: String
|
||||
@@ -31,14 +32,14 @@ class AudioContentCurationService(
|
||||
): GetCurationContentResponse {
|
||||
val totalCount = repository.findTotalCountByCurationId(
|
||||
curationId = curationId,
|
||||
isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
contentType = contentType
|
||||
)
|
||||
|
||||
val audioContentList = repository.findByCurationId(
|
||||
curationId = curationId,
|
||||
cloudfrontHost = cloudFrontHost,
|
||||
isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
contentType = contentType,
|
||||
sortType = sortType,
|
||||
offset = pageable.offset,
|
||||
|
||||
+5
-4
@@ -8,7 +8,7 @@ import kr.co.vividnext.sodalive.content.main.curation.AudioContentCurationQueryR
|
||||
import kr.co.vividnext.sodalive.content.main.tab.GetContentCurationResponse
|
||||
import kr.co.vividnext.sodalive.event.EventService
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.rank.RankingService
|
||||
import org.springframework.stereotype.Service
|
||||
import java.time.DayOfWeek
|
||||
@@ -21,14 +21,15 @@ class AudioContentMainTabAlarmService(
|
||||
private val contentRepository: AudioContentRepository,
|
||||
private val rankingService: RankingService,
|
||||
private val eventService: EventService,
|
||||
private val curationRepository: AudioContentCurationQueryRepository
|
||||
private val curationRepository: AudioContentCurationQueryRepository,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
fun fetchData(
|
||||
isAdultContentVisible: Boolean,
|
||||
contentType: ContentType,
|
||||
member: Member
|
||||
): GetContentMainTabAlarmResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val memberId = member.id!!
|
||||
|
||||
val contentBannerList = bannerService.getBannerList(
|
||||
@@ -106,7 +107,7 @@ class AudioContentMainTabAlarmService(
|
||||
}
|
||||
|
||||
val memberId = member.id!!
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
|
||||
val totalCount = contentRepository.totalAlarmCountByTheme(
|
||||
memberId = memberId,
|
||||
|
||||
+4
-3
@@ -9,7 +9,7 @@ import kr.co.vividnext.sodalive.content.main.tab.AudioContentMainTabRepository
|
||||
import kr.co.vividnext.sodalive.content.main.tab.GetContentCurationResponse
|
||||
import kr.co.vividnext.sodalive.event.EventService
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.rank.RankingService
|
||||
import org.springframework.stereotype.Service
|
||||
|
||||
@@ -20,14 +20,15 @@ class AudioContentMainTabAsmrService(
|
||||
private val contentRepository: AudioContentRepository,
|
||||
private val rankingService: RankingService,
|
||||
private val eventService: EventService,
|
||||
private val curationRepository: AudioContentCurationQueryRepository
|
||||
private val curationRepository: AudioContentCurationQueryRepository,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
fun fetchData(
|
||||
isAdultContentVisible: Boolean,
|
||||
contentType: ContentType,
|
||||
member: Member
|
||||
): GetContentMainTabAsmrResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val memberId = member.id!!
|
||||
val theme = "ASMR"
|
||||
val tabId = 5L
|
||||
|
||||
+5
-4
@@ -10,7 +10,7 @@ import kr.co.vividnext.sodalive.content.main.tab.GetContentCurationResponse
|
||||
import kr.co.vividnext.sodalive.content.theme.AudioContentThemeQueryRepository
|
||||
import kr.co.vividnext.sodalive.event.EventService
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.rank.RankingService
|
||||
import org.springframework.stereotype.Service
|
||||
import java.time.LocalDateTime
|
||||
@@ -23,7 +23,8 @@ class AudioContentMainTabContentService(
|
||||
private val rankingService: RankingService,
|
||||
private val eventService: EventService,
|
||||
private val tagCurationService: ContentMainTabTagCurationService,
|
||||
private val curationRepository: AudioContentCurationQueryRepository
|
||||
private val curationRepository: AudioContentCurationQueryRepository,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
fun fetchData(
|
||||
isAdultContentVisible: Boolean,
|
||||
@@ -31,7 +32,7 @@ class AudioContentMainTabContentService(
|
||||
member: Member
|
||||
): GetContentMainTabContentResponse {
|
||||
val memberId = member.id!!
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val tabId = 3L
|
||||
|
||||
// 단편 배너
|
||||
@@ -191,7 +192,7 @@ class AudioContentMainTabContentService(
|
||||
contentType: ContentType,
|
||||
member: Member
|
||||
): List<GetAudioContentMainItem> {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
|
||||
val themeList = if (theme.isBlank()) {
|
||||
audioContentThemeRepository.getActiveThemeOfContent(isAdult = isAdult, contentType = contentType)
|
||||
|
||||
+7
-6
@@ -11,7 +11,7 @@ import kr.co.vividnext.sodalive.content.main.tab.GetContentCurationResponse
|
||||
import kr.co.vividnext.sodalive.content.main.tab.RecommendSeriesRepository
|
||||
import kr.co.vividnext.sodalive.content.theme.AudioContentThemeQueryRepository
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.rank.RankingService
|
||||
import org.springframework.stereotype.Service
|
||||
|
||||
@@ -24,14 +24,15 @@ class AudioContentMainTabFreeService(
|
||||
private val contentRepository: AudioContentRepository,
|
||||
private val rankingService: RankingService,
|
||||
private val audioContentRepository: AudioContentRepository,
|
||||
private val audioContentThemeRepository: AudioContentThemeQueryRepository
|
||||
private val audioContentThemeRepository: AudioContentThemeQueryRepository,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
fun fetchData(
|
||||
isAdultContentVisible: Boolean,
|
||||
contentType: ContentType,
|
||||
member: Member
|
||||
): GetContentMainTabFreeResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val memberId = member.id!!
|
||||
val tabId = 7L
|
||||
|
||||
@@ -135,7 +136,7 @@ class AudioContentMainTabFreeService(
|
||||
offset: Long,
|
||||
limit: Long
|
||||
): List<GetAudioContentMainItem> {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val memberId = member.id!!
|
||||
|
||||
val introduceCreatorCuration = curationRepository.findByContentMainTabIdAndTitle(
|
||||
@@ -172,7 +173,7 @@ class AudioContentMainTabFreeService(
|
||||
listOf(theme)
|
||||
} else {
|
||||
audioContentThemeRepository.getActiveThemeOfContent(
|
||||
isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
isFree = true,
|
||||
contentType = contentType
|
||||
).filter {
|
||||
@@ -186,7 +187,7 @@ class AudioContentMainTabFreeService(
|
||||
it != "자기소개"
|
||||
}
|
||||
},
|
||||
isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
contentType = contentType,
|
||||
offset = offset,
|
||||
limit = limit,
|
||||
|
||||
+9
-4
@@ -5,7 +5,7 @@ import kr.co.vividnext.sodalive.content.main.GetAudioContentRankingItem
|
||||
import kr.co.vividnext.sodalive.content.main.banner.AudioContentBannerService
|
||||
import kr.co.vividnext.sodalive.event.EventService
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.notice.ServiceNoticeService
|
||||
import kr.co.vividnext.sodalive.rank.RankingService
|
||||
import org.springframework.stereotype.Service
|
||||
@@ -19,7 +19,8 @@ class AudioContentMainTabHomeService(
|
||||
private val noticeService: ServiceNoticeService,
|
||||
private val bannerService: AudioContentBannerService,
|
||||
private val rankingService: RankingService,
|
||||
private val eventService: EventService
|
||||
private val eventService: EventService,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
fun fetchData(
|
||||
isAdultContentVisible: Boolean,
|
||||
@@ -43,7 +44,9 @@ class AudioContentMainTabHomeService(
|
||||
val formattedLastMonday = startDate.format(startDateFormatter)
|
||||
val formattedLastSunday = endDate.format(endDateFormatter)
|
||||
|
||||
val isAdult = member?.let { isAdultVisibleByPolicy(it, isAdultContentVisible) } ?: false
|
||||
val isAdult = member
|
||||
?.let { memberContentPreferenceService.isAdultVisibleForQuery(it, isAdultContentVisible) }
|
||||
?: false
|
||||
|
||||
// 최근 공지사항
|
||||
val latestNotice = noticeService.getLatestNotice()
|
||||
@@ -131,7 +134,9 @@ class AudioContentMainTabHomeService(
|
||||
contentType: ContentType,
|
||||
member: Member?
|
||||
): List<GetAudioContentRankingItem> {
|
||||
val isAdult = member?.let { isAdultVisibleByPolicy(it, isAdultContentVisible) } ?: false
|
||||
val isAdult = member
|
||||
?.let { memberContentPreferenceService.isAdultVisibleForQuery(it, isAdultContentVisible) }
|
||||
?: false
|
||||
|
||||
val currentDateTime = LocalDateTime.now()
|
||||
val startDate = currentDateTime
|
||||
|
||||
+4
-3
@@ -9,7 +9,7 @@ import kr.co.vividnext.sodalive.content.main.tab.AudioContentMainTabRepository
|
||||
import kr.co.vividnext.sodalive.content.main.tab.GetContentCurationResponse
|
||||
import kr.co.vividnext.sodalive.event.EventService
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.rank.RankingService
|
||||
import org.springframework.stereotype.Service
|
||||
|
||||
@@ -20,14 +20,15 @@ class AudioContentMainTabLiveReplayService(
|
||||
private val contentRepository: AudioContentRepository,
|
||||
private val rankingService: RankingService,
|
||||
private val eventService: EventService,
|
||||
private val curationRepository: AudioContentCurationQueryRepository
|
||||
private val curationRepository: AudioContentCurationQueryRepository,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
fun fetchData(
|
||||
isAdultContentVisible: Boolean,
|
||||
contentType: ContentType,
|
||||
member: Member
|
||||
): GetContentMainTabLiveReplayResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val memberId = member.id!!
|
||||
val theme = "다시듣기"
|
||||
val tabId = 6L
|
||||
|
||||
+4
-3
@@ -8,7 +8,7 @@ import kr.co.vividnext.sodalive.content.series.ContentSeriesService
|
||||
import kr.co.vividnext.sodalive.content.series.GetSeriesListResponse
|
||||
import kr.co.vividnext.sodalive.event.EventService
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.rank.RankingService
|
||||
import org.springframework.stereotype.Service
|
||||
import java.time.DayOfWeek
|
||||
@@ -24,14 +24,15 @@ class AudioContentMainTabSeriesService(
|
||||
private val rankingService: RankingService,
|
||||
private val recommendSeriesRepository: RecommendSeriesRepository,
|
||||
private val eventService: EventService,
|
||||
private val curationRepository: AudioContentCurationQueryRepository
|
||||
private val curationRepository: AudioContentCurationQueryRepository,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
fun fetchData(
|
||||
isAdultContentVisible: Boolean,
|
||||
contentType: ContentType,
|
||||
member: Member
|
||||
): GetContentMainTabSeriesResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
val memberId = member.id!!
|
||||
|
||||
// 메인 배너 (시리즈)
|
||||
|
||||
@@ -19,7 +19,7 @@ import kr.co.vividnext.sodalive.i18n.translation.LanguageTranslationTargetType
|
||||
import kr.co.vividnext.sodalive.i18n.translation.ResourceTranslationJobScheduler
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.block.BlockMemberRepository
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import org.springframework.beans.factory.annotation.Value
|
||||
import org.springframework.stereotype.Service
|
||||
import org.springframework.transaction.annotation.Transactional
|
||||
@@ -40,6 +40,7 @@ class ContentSeriesService(
|
||||
private val seriesGenreTranslationRepository: SeriesGenreTranslationRepository,
|
||||
private val contentTranslationRepository: ContentTranslationRepository,
|
||||
private val resourceTranslationJobScheduler: ResourceTranslationJobScheduler,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService,
|
||||
|
||||
@Value("\${cloud.aws.cloud-front.host}")
|
||||
private val coverImageHost: String
|
||||
@@ -147,7 +148,7 @@ class ContentSeriesService(
|
||||
offset: Long = 0,
|
||||
limit: Long = 20
|
||||
): GetSeriesListResponse {
|
||||
val isAuth = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAuth = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
|
||||
val totalCount = repository.getSeriesTotalCount(
|
||||
creatorId = creatorId,
|
||||
@@ -185,7 +186,7 @@ class ContentSeriesService(
|
||||
offset: Long = 0,
|
||||
limit: Long = 20
|
||||
): GetSeriesListResponse {
|
||||
val isAuth = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAuth = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
|
||||
val totalCount = repository.getSeriesByGenreTotalCount(
|
||||
genreId = genreId,
|
||||
@@ -219,7 +220,7 @@ class ContentSeriesService(
|
||||
): GetSeriesDetailResponse {
|
||||
val series = repository.getSeriesDetail(
|
||||
seriesId = seriesId,
|
||||
isAuth = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAuth = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
contentType = contentType
|
||||
) ?: throw SodaException(messageKey = "series.error.invalid_series_retry")
|
||||
|
||||
@@ -364,7 +365,7 @@ class ContentSeriesService(
|
||||
offset: Long,
|
||||
limit: Long
|
||||
): GetSeriesContentListResponse {
|
||||
val isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
|
||||
val totalCount = seriesContentRepository.getContentCount(seriesId, isAdult = isAdult, contentType = contentType)
|
||||
val contentList = seriesContentRepository.getContentList(
|
||||
@@ -427,7 +428,7 @@ class ContentSeriesService(
|
||||
contentType: ContentType,
|
||||
member: Member
|
||||
): List<GetSeriesListResponse.SeriesListItem> {
|
||||
val isAuth = isAdultVisibleByPolicy(member, isAdultContentVisible)
|
||||
val isAuth = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible)
|
||||
return repository.getRecommendSeriesListV2(
|
||||
imageHost = coverImageHost,
|
||||
isAuth = isAuth,
|
||||
|
||||
@@ -11,7 +11,7 @@ import kr.co.vividnext.sodalive.i18n.LangContext
|
||||
import kr.co.vividnext.sodalive.i18n.translation.LanguageTranslationTargetType
|
||||
import kr.co.vividnext.sodalive.i18n.translation.ResourceTranslationJobScheduler
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.isAdultVisibleByPolicy
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import org.springframework.stereotype.Service
|
||||
import org.springframework.transaction.annotation.Transactional
|
||||
|
||||
@@ -22,7 +22,8 @@ class AudioContentThemeService(
|
||||
private val contentThemeTranslationRepository: ContentThemeTranslationRepository,
|
||||
|
||||
private val resourceTranslationJobScheduler: ResourceTranslationJobScheduler,
|
||||
private val langContext: LangContext
|
||||
private val langContext: LangContext,
|
||||
private val memberContentPreferenceService: MemberContentPreferenceService
|
||||
) {
|
||||
@Transactional(readOnly = true)
|
||||
fun getThemes(): List<GetAudioContentThemeResponse> {
|
||||
@@ -100,7 +101,7 @@ class AudioContentThemeService(
|
||||
val totalCount = contentRepository.totalCountByTheme(
|
||||
memberId = member.id!!,
|
||||
theme = listOf(theme.theme),
|
||||
isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
contentType = contentType
|
||||
)
|
||||
|
||||
@@ -108,7 +109,7 @@ class AudioContentThemeService(
|
||||
memberId = member.id!!,
|
||||
theme = listOf(theme.theme),
|
||||
sortType = sortType,
|
||||
isAdult = isAdultVisibleByPolicy(member, isAdultContentVisible),
|
||||
isAdult = memberContentPreferenceService.isAdultVisibleForQuery(member, isAdultContentVisible),
|
||||
contentType = contentType,
|
||||
offset = offset,
|
||||
limit = limit
|
||||
|
||||
@@ -812,6 +812,14 @@ class LiveRoomService(
|
||||
)
|
||||
}
|
||||
|
||||
if (
|
||||
member.role == MemberRole.CREATOR &&
|
||||
room.member!!.id!! != member.id!! &&
|
||||
!room.isAvailableJoinCreator
|
||||
) {
|
||||
throw SodaException(messageKey = "live.room.not_found")
|
||||
}
|
||||
|
||||
if (room.member!!.id!! != member.id!! && !member.canEnter(room.genderRestriction)) {
|
||||
throw SodaException(messageKey = "live.room.gender_restricted")
|
||||
}
|
||||
@@ -975,6 +983,18 @@ class LiveRoomService(
|
||||
throw SodaException(messageKey = "live.room.info_not_found")
|
||||
}
|
||||
|
||||
if (
|
||||
member.role == MemberRole.CREATOR &&
|
||||
room.member!!.id!! != member.id!! &&
|
||||
!room.isAvailableJoinCreator
|
||||
) {
|
||||
throw SodaException(messageKey = "live.room.not_found")
|
||||
}
|
||||
|
||||
if (room.member!!.id!! != member.id!! && !member.canEnter(room.genderRestriction)) {
|
||||
throw SodaException(messageKey = "live.room.gender_restricted")
|
||||
}
|
||||
|
||||
val currentTimeStamp = Date().time
|
||||
val expireTimestamp = (currentTimeStamp + (60 * 60 * 24 * 1000)) / 1000
|
||||
|
||||
|
||||
+1
-1
@@ -3,7 +3,7 @@ package kr.co.vividnext.sodalive.member.contentpreference
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
|
||||
private val FORCED_KR_MEMBER_IDS = setOf(16L, 17L, 17958L, 44144L)
|
||||
private val FORCED_JP_MEMBER_IDS = setOf(2L, 29721L, 32050L, 37543L, 40850L)
|
||||
private val FORCED_JP_MEMBER_IDS = setOf(29721L, 32050L, 37543L, 40850L)
|
||||
|
||||
fun resolveCountryCodeWithForcedMapping(member: Member?, requestCountryCode: String?): String {
|
||||
val memberId = member?.id
|
||||
|
||||
-21
@@ -1,21 +0,0 @@
|
||||
package kr.co.vividnext.sodalive.member.contentpreference
|
||||
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import org.springframework.web.context.request.RequestContextHolder
|
||||
import org.springframework.web.context.request.ServletRequestAttributes
|
||||
|
||||
@Deprecated("Use MemberContentPreferenceService.canViewAdultContent(member)")
|
||||
fun resolveCountryCodeByPolicy(member: Member): String {
|
||||
val requestAttributes = RequestContextHolder.getRequestAttributes() as? ServletRequestAttributes
|
||||
val requestCountryCode = requestAttributes?.request?.getHeader("CloudFront-Viewer-Country")
|
||||
return resolveCountryCodeWithForcedMapping(member, requestCountryCode)
|
||||
}
|
||||
|
||||
@Deprecated("Use MemberContentPreferenceService.canViewAdultContent(member)")
|
||||
fun isAdultVisibleByPolicy(member: Member, isAdultContentVisible: Boolean): Boolean {
|
||||
return if (resolveCountryCodeByPolicy(member) == "KR") {
|
||||
member.auth != null && isAdultContentVisible
|
||||
} else {
|
||||
isAdultContentVisible
|
||||
}
|
||||
}
|
||||
+8
@@ -157,6 +157,14 @@ class MemberContentPreferenceService(
|
||||
return getStoredPreference(member).isAdult
|
||||
}
|
||||
|
||||
fun isAdultVisibleForQuery(member: Member, isAdultContentVisible: Boolean): Boolean {
|
||||
return calculateIsAdultForQuery(
|
||||
member = member,
|
||||
countryCode = resolveCountryCode(member),
|
||||
isAdultContentVisible = isAdultContentVisible
|
||||
)
|
||||
}
|
||||
|
||||
fun resolveCountryCode(member: Member?): String {
|
||||
if (member != null) requireMemberId(member)
|
||||
return resolveCountryCodeWithForcedMapping(member, countryContext.countryCode)
|
||||
|
||||
+12
-2
@@ -2,7 +2,9 @@ package kr.co.vividnext.sodalive.v2.api.home.application
|
||||
|
||||
import kr.co.vividnext.sodalive.event.EventItem
|
||||
import kr.co.vividnext.sodalive.i18n.Lang
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberRole
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.v2.api.common.dto.RecommendationBannerResponse
|
||||
import kr.co.vividnext.sodalive.v2.api.home.dto.recommendation.HomeActiveCreatorItem
|
||||
@@ -53,7 +55,9 @@ class HomeRecommendationFacade(
|
||||
lives = queryService.findLiveRecommendations(
|
||||
limit = HOME_LIVE_LIMIT,
|
||||
memberId = member?.id,
|
||||
includeAdultLives = includeAdult
|
||||
includeAdultLives = includeAdult,
|
||||
isViewerCreator = member?.role == MemberRole.CREATOR,
|
||||
effectiveViewerGender = member?.effectiveGender()
|
||||
).map { it.toItem() },
|
||||
banners = queryService.findHomeBanners(
|
||||
limit = HOME_BANNER_LIMIT,
|
||||
@@ -125,7 +129,9 @@ class HomeRecommendationFacade(
|
||||
offset = page.toOffset(size),
|
||||
limit = size + 1,
|
||||
memberId = member.id,
|
||||
includeAdultLives = resolveAdultVisibility(member)
|
||||
includeAdultLives = resolveAdultVisibility(member),
|
||||
isViewerCreator = member.role == MemberRole.CREATOR,
|
||||
effectiveViewerGender = member.effectiveGender()
|
||||
)
|
||||
fetched.toPage(page, size) { it.toItem() }
|
||||
}.onSuccess {
|
||||
@@ -209,6 +215,10 @@ class HomeRecommendationFacade(
|
||||
return memberContentPreferenceService.canViewAdultContent(member)
|
||||
}
|
||||
|
||||
private fun Member.effectiveGender(): Gender {
|
||||
return auth?.let { if (it.gender == 1) Gender.MALE else Gender.FEMALE } ?: gender
|
||||
}
|
||||
|
||||
private fun Int.toOffset(size: Int): Long = this.toLong() * size
|
||||
|
||||
private fun <S, T> List<S>.toPage(
|
||||
|
||||
+9
-1
@@ -1,6 +1,8 @@
|
||||
package kr.co.vividnext.sodalive.v2.api.home.live.application
|
||||
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberRole
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.v2.api.home.live.dto.HomeOnAirLivePageResponse
|
||||
import kr.co.vividnext.sodalive.v2.api.home.live.dto.HomeOnAirLiveResponse
|
||||
@@ -24,7 +26,9 @@ class HomeOnAirLiveFacade(
|
||||
offset = normalizedPage.toLong() * PAGE_SIZE,
|
||||
limit = PAGE_SIZE + 1,
|
||||
memberId = member.id,
|
||||
includeAdultLives = memberContentPreferenceService.canViewAdultContent(member)
|
||||
includeAdultLives = memberContentPreferenceService.canViewAdultContent(member),
|
||||
isViewerCreator = member.role == MemberRole.CREATOR,
|
||||
effectiveViewerGender = member.effectiveGender()
|
||||
)
|
||||
val items = fetched.take(PAGE_SIZE).map { it.toResponse() }
|
||||
|
||||
@@ -57,6 +61,10 @@ class HomeOnAirLiveFacade(
|
||||
return atOffset(ZoneOffset.UTC).toInstant().toString()
|
||||
}
|
||||
|
||||
private fun Member.effectiveGender(): Gender {
|
||||
return auth?.let { if (it.gender == 1) Gender.MALE else Gender.FEMALE } ?: gender
|
||||
}
|
||||
|
||||
companion object {
|
||||
private const val PAGE_SIZE = 20
|
||||
private const val MAX_PAGE = 10_000
|
||||
|
||||
+1
-1
@@ -58,7 +58,7 @@ class CreatorChannelLiveQueryService(
|
||||
validateCreatorRole(creator)
|
||||
|
||||
val canViewAdultContent = memberContentPreferenceService.canViewAdultContent(viewer)
|
||||
val isViewerCreator = viewerId == creatorId
|
||||
val isViewerCreator = viewer.role == MemberRole.CREATOR
|
||||
val effectiveViewerGender = viewer.effectiveGender()
|
||||
val fetchedContents = queryPort.findLiveReplayAudioContents(
|
||||
creatorId = creatorId,
|
||||
|
||||
+22
-1
@@ -22,7 +22,9 @@ import kr.co.vividnext.sodalive.explorer.profile.creatorCommunity.QCreatorCommun
|
||||
import kr.co.vividnext.sodalive.explorer.profile.creatorCommunity.comment.QCreatorCommunityComment.creatorCommunityComment
|
||||
import kr.co.vividnext.sodalive.explorer.profile.creatorCommunity.like.QCreatorCommunityLike.creatorCommunityLike
|
||||
import kr.co.vividnext.sodalive.i18n.Lang
|
||||
import kr.co.vividnext.sodalive.live.room.GenderRestriction
|
||||
import kr.co.vividnext.sodalive.live.room.QLiveRoom.liveRoom
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.MemberKind
|
||||
import kr.co.vividnext.sodalive.member.MemberRole
|
||||
import kr.co.vividnext.sodalive.member.QMember
|
||||
@@ -58,7 +60,9 @@ class DefaultHomeRecommendationQueryRepository(
|
||||
offset: Long,
|
||||
limit: Int,
|
||||
memberId: Long?,
|
||||
includeAdultLives: Boolean
|
||||
includeAdultLives: Boolean,
|
||||
isViewerCreator: Boolean,
|
||||
effectiveViewerGender: Gender?
|
||||
): List<HomeLiveRecommendationRecord> {
|
||||
return queryFactory
|
||||
.select(
|
||||
@@ -79,6 +83,8 @@ class DefaultHomeRecommendationQueryRepository(
|
||||
liveRoom.channelName.isNotNull,
|
||||
liveRoom.channelName.isNotEmpty,
|
||||
includeAdultLiveCondition(includeAdultLives),
|
||||
genderLiveCondition(memberId, effectiveViewerGender),
|
||||
creatorJoinLiveCondition(memberId, isViewerCreator),
|
||||
notBlockedCreatorCondition(memberId, member.id),
|
||||
member.isActive.isTrue
|
||||
)
|
||||
@@ -1294,6 +1300,21 @@ class DefaultHomeRecommendationQueryRepository(
|
||||
return if (includeAdultLives) null else liveRoom.isAdult.isFalse
|
||||
}
|
||||
|
||||
private fun genderLiveCondition(viewerId: Long?, effectiveViewerGender: Gender?): BooleanExpression? {
|
||||
if (effectiveViewerGender == null || effectiveViewerGender == Gender.NONE) return null
|
||||
val genderCondition = when (effectiveViewerGender) {
|
||||
Gender.MALE -> liveRoom.genderRestriction.`in`(GenderRestriction.ALL, GenderRestriction.MALE_ONLY)
|
||||
Gender.FEMALE -> liveRoom.genderRestriction.`in`(GenderRestriction.ALL, GenderRestriction.FEMALE_ONLY)
|
||||
Gender.NONE -> return null
|
||||
}
|
||||
return viewerId?.let { genderCondition.or(liveRoom.member.id.eq(it)) } ?: genderCondition
|
||||
}
|
||||
|
||||
private fun creatorJoinLiveCondition(viewerId: Long?, isViewerCreator: Boolean): BooleanExpression? {
|
||||
if (!isViewerCreator || viewerId == null) return null
|
||||
return liveRoom.isAvailableJoinCreator.isTrue.or(liveRoom.member.id.eq(viewerId))
|
||||
}
|
||||
|
||||
private fun excludeAiCharacterCondition(excludeCharacterIds: List<Long>): BooleanExpression? {
|
||||
return if (excludeCharacterIds.isEmpty()) null else chatCharacter.id.notIn(excludeCharacterIds)
|
||||
}
|
||||
|
||||
+12
-2
@@ -2,6 +2,7 @@ package kr.co.vividnext.sodalive.v2.recommendation.application
|
||||
|
||||
import kr.co.vividnext.sodalive.i18n.Lang
|
||||
import kr.co.vividnext.sodalive.i18n.LangContext
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.v2.common.domain.CreatorActivityType
|
||||
import kr.co.vividnext.sodalive.v2.creator.channel.community.translation.application.CreatorCommunityTranslationService
|
||||
import kr.co.vividnext.sodalive.v2.recommendation.domain.RecommendationSnapshotWindowPolicy
|
||||
@@ -41,9 +42,18 @@ class HomeRecommendationQueryService(
|
||||
offset: Long = 0,
|
||||
limit: Int = DEFAULT_LIVE_LIMIT,
|
||||
memberId: Long? = null,
|
||||
includeAdultLives: Boolean = false
|
||||
includeAdultLives: Boolean = false,
|
||||
isViewerCreator: Boolean = false,
|
||||
effectiveViewerGender: Gender? = null
|
||||
): List<HomeLiveRecommendationRecord> {
|
||||
return queryPort.findLiveRecommendations(offset, limit, memberId, includeAdultLives)
|
||||
return queryPort.findLiveRecommendations(
|
||||
offset,
|
||||
limit,
|
||||
memberId,
|
||||
includeAdultLives,
|
||||
isViewerCreator,
|
||||
effectiveViewerGender
|
||||
)
|
||||
}
|
||||
|
||||
fun findHomeBanners(
|
||||
|
||||
+4
-1
@@ -1,6 +1,7 @@
|
||||
package kr.co.vividnext.sodalive.v2.recommendation.port.out
|
||||
|
||||
import kr.co.vividnext.sodalive.i18n.Lang
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.v2.common.domain.CreatorActivityType
|
||||
import java.time.LocalDateTime
|
||||
|
||||
@@ -9,7 +10,9 @@ interface HomeRecommendationQueryPort {
|
||||
offset: Long = 0,
|
||||
limit: Int,
|
||||
memberId: Long? = null,
|
||||
includeAdultLives: Boolean = false
|
||||
includeAdultLives: Boolean = false,
|
||||
isViewerCreator: Boolean = false,
|
||||
effectiveViewerGender: Gender? = null
|
||||
): List<HomeLiveRecommendationRecord>
|
||||
|
||||
fun findHomeBanners(
|
||||
|
||||
@@ -23,6 +23,7 @@ import kr.co.vividnext.sodalive.i18n.translation.ResourceTranslationJobScheduler
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberRepository
|
||||
import kr.co.vividnext.sodalive.member.block.BlockMemberRepository
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.v2.home.following.application.HomeFollowingNewsPublishService
|
||||
import kr.co.vividnext.sodalive.v2.recommendation.application.CreatorContentViewHistoryService
|
||||
import org.junit.jupiter.api.Assertions.assertEquals
|
||||
@@ -63,6 +64,7 @@ class AudioContentServiceTest {
|
||||
private lateinit var applicationEventPublisher: ApplicationEventPublisher
|
||||
private lateinit var contentThemeTranslationRepository: ContentThemeTranslationRepository
|
||||
private lateinit var homeFollowingNewsPublishService: HomeFollowingNewsPublishService
|
||||
private lateinit var memberContentPreferenceService: MemberContentPreferenceService
|
||||
|
||||
private lateinit var service: AudioContentService
|
||||
|
||||
@@ -88,6 +90,7 @@ class AudioContentServiceTest {
|
||||
applicationEventPublisher = Mockito.mock(ApplicationEventPublisher::class.java)
|
||||
contentThemeTranslationRepository = Mockito.mock(ContentThemeTranslationRepository::class.java)
|
||||
homeFollowingNewsPublishService = Mockito.mock(HomeFollowingNewsPublishService::class.java)
|
||||
memberContentPreferenceService = Mockito.mock(MemberContentPreferenceService::class.java)
|
||||
|
||||
service = AudioContentService(
|
||||
repository = repository,
|
||||
@@ -113,6 +116,7 @@ class AudioContentServiceTest {
|
||||
langContext = LangContext(),
|
||||
contentThemeTranslationRepository = contentThemeTranslationRepository,
|
||||
homeFollowingNewsPublishService = homeFollowingNewsPublishService,
|
||||
memberContentPreferenceService = memberContentPreferenceService,
|
||||
audioContentBucket = "audio-bucket",
|
||||
coverImageBucket = "cover-bucket",
|
||||
coverImageHost = "https://cdn.test"
|
||||
|
||||
@@ -0,0 +1,429 @@
|
||||
package kr.co.vividnext.sodalive.live.room
|
||||
|
||||
import com.fasterxml.jackson.databind.ObjectMapper
|
||||
import kr.co.vividnext.sodalive.agora.RtcTokenBuilder
|
||||
import kr.co.vividnext.sodalive.agora.RtmTokenBuilder
|
||||
import kr.co.vividnext.sodalive.aws.s3.S3Uploader
|
||||
import kr.co.vividnext.sodalive.can.CanRepository
|
||||
import kr.co.vividnext.sodalive.can.charge.ChargeRepository
|
||||
import kr.co.vividnext.sodalive.can.payment.CanPaymentService
|
||||
import kr.co.vividnext.sodalive.can.use.UseCanCalculateRepository
|
||||
import kr.co.vividnext.sodalive.common.SodaException
|
||||
import kr.co.vividnext.sodalive.explorer.ExplorerQueryRepository
|
||||
import kr.co.vividnext.sodalive.explorer.profile.CreatorDonationRankingService
|
||||
import kr.co.vividnext.sodalive.fcm.PushTokenRepository
|
||||
import kr.co.vividnext.sodalive.i18n.LangContext
|
||||
import kr.co.vividnext.sodalive.i18n.SodaMessageSource
|
||||
import kr.co.vividnext.sodalive.live.reservation.LiveReservationRepository
|
||||
import kr.co.vividnext.sodalive.live.room.cancel.LiveRoomCancelRepository
|
||||
import kr.co.vividnext.sodalive.live.room.info.LiveRoomInfo
|
||||
import kr.co.vividnext.sodalive.live.room.info.LiveRoomInfoRedisRepository
|
||||
import kr.co.vividnext.sodalive.live.room.kickout.LiveRoomKickOutService
|
||||
import kr.co.vividnext.sodalive.live.room.menu.LiveRoomMenuService
|
||||
import kr.co.vividnext.sodalive.live.room.visit.LiveRoomVisitService
|
||||
import kr.co.vividnext.sodalive.live.roulette.NewRouletteRepository
|
||||
import kr.co.vividnext.sodalive.live.signature.SignatureCanRepository
|
||||
import kr.co.vividnext.sodalive.live.tag.LiveTagRepository
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberRepository
|
||||
import kr.co.vividnext.sodalive.member.MemberRole
|
||||
import kr.co.vividnext.sodalive.member.auth.Auth
|
||||
import kr.co.vividnext.sodalive.member.block.BlockMemberRepository
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import org.junit.jupiter.api.Assertions.assertEquals
|
||||
import org.junit.jupiter.api.BeforeEach
|
||||
import org.junit.jupiter.api.DisplayName
|
||||
import org.junit.jupiter.api.Test
|
||||
import org.junit.jupiter.api.assertThrows
|
||||
import org.mockito.Mockito
|
||||
import java.time.LocalDateTime
|
||||
import java.util.Optional
|
||||
|
||||
class LiveRoomServiceTest {
|
||||
private lateinit var repository: LiveRoomRepository
|
||||
private lateinit var roomInfoRepository: LiveRoomInfoRedisRepository
|
||||
private lateinit var canPaymentService: CanPaymentService
|
||||
private lateinit var memberRepository: MemberRepository
|
||||
private lateinit var blockMemberRepository: BlockMemberRepository
|
||||
private lateinit var explorerQueryRepository: ExplorerQueryRepository
|
||||
private lateinit var rouletteRepository: NewRouletteRepository
|
||||
private lateinit var pushTokenRepository: PushTokenRepository
|
||||
private lateinit var rtcTokenBuilder: RtcTokenBuilder
|
||||
private lateinit var rtmTokenBuilder: RtmTokenBuilder
|
||||
private lateinit var service: LiveRoomService
|
||||
|
||||
@BeforeEach
|
||||
fun setup() {
|
||||
repository = mock()
|
||||
roomInfoRepository = mock()
|
||||
canPaymentService = mock()
|
||||
memberRepository = mock()
|
||||
blockMemberRepository = mock()
|
||||
explorerQueryRepository = mock()
|
||||
rouletteRepository = mock()
|
||||
pushTokenRepository = mock()
|
||||
rtcTokenBuilder = mock()
|
||||
rtmTokenBuilder = mock()
|
||||
|
||||
service = LiveRoomService(
|
||||
menuService = mock<LiveRoomMenuService>(),
|
||||
messageSource = mock<SodaMessageSource>(),
|
||||
langContext = LangContext(),
|
||||
repository = repository,
|
||||
rouletteRepository = rouletteRepository,
|
||||
roomInfoRepository = roomInfoRepository,
|
||||
roomCancelRepository = mock<LiveRoomCancelRepository>(),
|
||||
kickOutService = mock<LiveRoomKickOutService>(),
|
||||
blockMemberRepository = blockMemberRepository,
|
||||
signatureCanRepository = mock<SignatureCanRepository>(),
|
||||
applicationEventPublisher = mock(),
|
||||
useCanCalculateRepository = mock<UseCanCalculateRepository>(),
|
||||
reservationRepository = mock<LiveReservationRepository>(),
|
||||
explorerQueryRepository = explorerQueryRepository,
|
||||
creatorDonationRankingService = mock<CreatorDonationRankingService>(),
|
||||
roomVisitService = mock<LiveRoomVisitService>(),
|
||||
canPaymentService = canPaymentService,
|
||||
chargeRepository = mock<ChargeRepository>(),
|
||||
pushTokenRepository = pushTokenRepository,
|
||||
memberRepository = memberRepository,
|
||||
tagRepository = mock<LiveTagRepository>(),
|
||||
memberContentPreferenceService = mock<MemberContentPreferenceService>(),
|
||||
canRepository = mock<CanRepository>(),
|
||||
objectMapper = ObjectMapper(),
|
||||
s3Uploader = mock<S3Uploader>(),
|
||||
rtcTokenBuilder = rtcTokenBuilder,
|
||||
rtmTokenBuilder = rtmTokenBuilder,
|
||||
agoraAppId = "test-agora-app-id",
|
||||
agoraAppCertificate = "test-agora-app-certificate",
|
||||
coverImageBucket = "test-cover-image-bucket",
|
||||
cloudFrontHost = "https://test-cloudfront-host"
|
||||
)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("다른 크리에이터는 크리에이터 입장 제한 방에 입장할 수 없다")
|
||||
fun shouldRejectDifferentCreatorBeforePaymentOrRoomInfoMutation() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val enteringCreator = createMember(id = 2L, role = MemberRole.CREATOR).also { it.pgChargeCan = 10 }
|
||||
val room = createRoom(owner = owner, price = 10)
|
||||
prepareAllowedEntry(room)
|
||||
Mockito.`when`(memberRepository.findById(enteringCreator.id!!)).thenReturn(Optional.of(enteringCreator))
|
||||
|
||||
val exception = assertThrows<SodaException> {
|
||||
service.enterLive(EnterOrQuitLiveRoomRequest(roomId = room.id!!, container = "web"), enteringCreator)
|
||||
}
|
||||
|
||||
assertEquals("live.room.not_found", exception.messageKey)
|
||||
Mockito.verifyNoInteractions(roomInfoRepository, canPaymentService)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("성별 제한에 맞지 않는 일반 사용자는 입장할 수 없다")
|
||||
fun shouldRejectGenderMismatchedUserBeforePaymentOrRoomInfoMutation() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val member = createMember(id = 2L, gender = Gender.FEMALE)
|
||||
val room = createRoom(owner = owner, price = 10, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
Mockito.`when`(repository.getLiveRoom(room.id!!)).thenReturn(room)
|
||||
|
||||
val exception = assertThrows<SodaException> {
|
||||
service.enterLive(EnterOrQuitLiveRoomRequest(roomId = room.id!!, container = "web"), member)
|
||||
}
|
||||
|
||||
assertEquals("live.room.gender_restricted", exception.messageKey)
|
||||
Mockito.verifyNoInteractions(roomInfoRepository, canPaymentService)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("방 생성자는 크리에이터 및 성별 제한과 무관하게 입장할 수 있다")
|
||||
fun shouldAllowRoomOwner() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR, gender = Gender.MALE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.FEMALE_ONLY)
|
||||
val roomInfo = prepareAllowedEntry(room)
|
||||
|
||||
service.enterLive(EnterOrQuitLiveRoomRequest(roomId = room.id!!, container = "web"), owner)
|
||||
|
||||
assertEquals(1, roomInfo.speakerCount)
|
||||
Mockito.verify(roomInfoRepository).save(roomInfo)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("일반 사용자는 조건에 맞으면 크리에이터 입장 제한 방에 입장할 수 있다")
|
||||
fun shouldAllowEligibleUser() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val member = createMember(id = 2L, gender = Gender.MALE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
val roomInfo = prepareAllowedEntry(room)
|
||||
|
||||
service.enterLive(EnterOrQuitLiveRoomRequest(roomId = room.id!!, container = "web"), member)
|
||||
|
||||
assertEquals(1, roomInfo.listenerCount)
|
||||
Mockito.verify(roomInfoRepository).save(roomInfo)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("성별 미설정 사용자는 기존 성별 제한 정책대로 입장할 수 있다")
|
||||
fun shouldAllowUserWithNoGender() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val member = createMember(id = 2L, gender = Gender.NONE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
val roomInfo = prepareAllowedEntry(room)
|
||||
|
||||
service.enterLive(EnterOrQuitLiveRoomRequest(roomId = room.id!!, container = "web"), member)
|
||||
|
||||
assertEquals(1, roomInfo.listenerCount)
|
||||
Mockito.verify(roomInfoRepository).save(roomInfo)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("다른 크리에이터는 제한 방 정보 조회로 토큰을 발급받을 수 없다")
|
||||
fun shouldRejectDifferentCreatorFromRoomInfoBeforeTokenGeneration() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, role = MemberRole.CREATOR)
|
||||
val room = createRoom(owner = owner)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
val exception = assertThrows<SodaException> {
|
||||
service.getRoomInfo(roomId = room.id!!, member = viewer)
|
||||
}
|
||||
|
||||
assertEquals("live.room.not_found", exception.messageKey)
|
||||
Mockito.verifyNoInteractions(rtcTokenBuilder, rtmTokenBuilder)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("성별 제한에 맞지 않는 사용자는 방 정보 조회로 토큰을 발급받을 수 없다")
|
||||
fun shouldRejectGenderMismatchedNonOwnerFromRoomInfoBeforeTokenGeneration() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, gender = Gender.FEMALE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
val exception = assertThrows<SodaException> {
|
||||
service.getRoomInfo(roomId = room.id!!, member = viewer)
|
||||
}
|
||||
|
||||
assertEquals("live.room.gender_restricted", exception.messageKey)
|
||||
Mockito.verifyNoInteractions(rtcTokenBuilder, rtmTokenBuilder)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("크리에이터와 성별 제한이 함께 적용되면 크리에이터 제한을 먼저 반환한다")
|
||||
fun shouldPrioritizeCreatorRestrictionOverGenderRestrictionInRoomInfo() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, role = MemberRole.CREATOR, gender = Gender.FEMALE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
val exception = assertThrows<SodaException> {
|
||||
service.getRoomInfo(roomId = room.id!!, member = viewer)
|
||||
}
|
||||
|
||||
assertEquals("live.room.not_found", exception.messageKey)
|
||||
Mockito.verifyNoInteractions(rtcTokenBuilder, rtmTokenBuilder)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("입장 거절 뒤 같은 제한 방 정보 조회도 토큰을 발급하지 않는다")
|
||||
fun shouldNotGenerateTokensAfterEnterLiveRejectsDifferentCreator() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, role = MemberRole.CREATOR)
|
||||
val room = createRoom(owner = owner)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
Mockito.`when`(repository.getLiveRoom(room.id!!)).thenReturn(room)
|
||||
|
||||
val enterException = assertThrows<SodaException> {
|
||||
service.enterLive(EnterOrQuitLiveRoomRequest(roomId = room.id!!, container = "web"), viewer)
|
||||
}
|
||||
val roomInfoException = assertThrows<SodaException> {
|
||||
service.getRoomInfo(roomId = room.id!!, member = viewer)
|
||||
}
|
||||
|
||||
assertEquals("live.room.not_found", enterException.messageKey)
|
||||
assertEquals("live.room.not_found", roomInfoException.messageKey)
|
||||
Mockito.verifyNoInteractions(rtcTokenBuilder, rtmTokenBuilder)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("방 생성자는 모든 크리에이터와 성별 제한이 있어도 방 정보 토큰을 발급받는다")
|
||||
fun shouldAllowRoomOwnerToGetRoomInfoTokens() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR, gender = Gender.MALE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.FEMALE_ONLY)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
assertTokenizedRoomInfo(room, owner)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("성별이 맞는 일반 사용자는 제한 방 정보 토큰을 발급받는다")
|
||||
fun shouldAllowGenderMatchedUserToGetRoomInfoTokens() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, gender = Gender.MALE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
assertTokenizedRoomInfo(room, viewer)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("크리에이터 입장이 허용된 다른 크리에이터는 방 정보 토큰을 발급받는다")
|
||||
fun shouldAllowDifferentCreatorWhenCreatorJoiningIsEnabledToGetRoomInfoTokens() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, role = MemberRole.CREATOR, gender = Gender.MALE)
|
||||
val room = createRoom(
|
||||
owner = owner,
|
||||
isAvailableJoinCreator = true,
|
||||
genderRestriction = GenderRestriction.MALE_ONLY
|
||||
)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
assertTokenizedRoomInfo(room, viewer)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("인증 성별은 프로필 성별보다 우선하여 방 정보 제한을 판정한다")
|
||||
fun shouldUseAuthGenderBeforeProfileGenderForRoomInfoTokens() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, gender = Gender.MALE)
|
||||
viewer.auth = Auth(
|
||||
name = "viewer",
|
||||
birth = "20000101",
|
||||
uniqueCi = "unique-ci",
|
||||
di = "di",
|
||||
gender = 2
|
||||
)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.FEMALE_ONLY)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
assertTokenizedRoomInfo(room, viewer)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("성별 미설정 사용자는 기존 정책대로 방 정보 토큰을 발급받는다")
|
||||
fun shouldAllowUserWithNoGenderToGetRoomInfoTokens() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, gender = Gender.NONE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
|
||||
assertTokenizedRoomInfo(room, viewer)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("상호 차단된 사용자는 기존 방 정보 예외로 토큰을 발급받을 수 없다")
|
||||
fun shouldRejectMutuallyBlockedUserFromRoomInfoBeforeTokenGeneration() {
|
||||
val owner = createMember(id = 1L, role = MemberRole.CREATOR)
|
||||
val viewer = createMember(id = 2L, gender = Gender.MALE)
|
||||
val room = createRoom(owner = owner, genderRestriction = GenderRestriction.MALE_ONLY)
|
||||
prepareSuccessfulRoomInfo(room)
|
||||
Mockito.`when`(blockMemberRepository.isBlocked(Mockito.anyLong(), Mockito.anyLong())).thenReturn(true)
|
||||
|
||||
val exception = assertThrows<SodaException> {
|
||||
service.getRoomInfo(roomId = room.id!!, member = viewer)
|
||||
}
|
||||
|
||||
assertEquals("live.room.info_not_found", exception.messageKey)
|
||||
Mockito.verifyNoInteractions(rtcTokenBuilder, rtmTokenBuilder)
|
||||
}
|
||||
|
||||
private fun prepareAllowedEntry(room: LiveRoom): LiveRoomInfo {
|
||||
val roomInfo = LiveRoomInfo(roomId = room.id!!)
|
||||
Mockito.`when`(repository.getLiveRoom(room.id!!)).thenReturn(room)
|
||||
Mockito.`when`(roomInfoRepository.findById(room.id!!)).thenReturn(Optional.of(roomInfo))
|
||||
Mockito.`when`(roomInfoRepository.save(roomInfo)).thenReturn(roomInfo)
|
||||
return roomInfo
|
||||
}
|
||||
|
||||
private fun prepareSuccessfulRoomInfo(room: LiveRoom): LiveRoomInfo {
|
||||
val roomInfo = LiveRoomInfo(roomId = room.id!!)
|
||||
room.member!!.isVisibleDonationRank = false
|
||||
room.channelName = "SODA_LIVE_CHANNEL_TEST"
|
||||
|
||||
Mockito.`when`(roomInfoRepository.findById(room.id!!)).thenReturn(Optional.of(roomInfo))
|
||||
Mockito.`when`(repository.findById(room.id!!)).thenReturn(Optional.of(room))
|
||||
Mockito.`when`(blockMemberRepository.isBlocked(Mockito.anyLong(), Mockito.anyLong())).thenReturn(false)
|
||||
Mockito.`when`(
|
||||
rtcTokenBuilder.buildTokenWithUid(
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyInt()
|
||||
)
|
||||
).thenReturn("rtc-token")
|
||||
Mockito.`when`(
|
||||
rtmTokenBuilder.buildToken(
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyInt()
|
||||
)
|
||||
).thenReturn("rtm-token")
|
||||
Mockito.`when`(explorerQueryRepository.getNotificationUserIds(room.member!!.id!!)).thenReturn(emptyList())
|
||||
Mockito.`when`(rouletteRepository.findByCreatorId(room.member!!.id!!)).thenReturn(emptyList())
|
||||
Mockito.`when`(pushTokenRepository.findByMemberId(room.member!!.id!!)).thenReturn(emptyList())
|
||||
return roomInfo
|
||||
}
|
||||
|
||||
private fun assertTokenizedRoomInfo(room: LiveRoom, member: Member) {
|
||||
val response = service.getRoomInfo(roomId = room.id!!, member = member)
|
||||
|
||||
assertEquals(room.channelName, response.channelName)
|
||||
assertEquals("rtc-token", response.rtcToken)
|
||||
assertEquals("rtm-token", response.rtmToken)
|
||||
assertEquals("rtc-token", response.v2vWorkerToken)
|
||||
Mockito.verify(rtcTokenBuilder, Mockito.times(2)).buildTokenWithUid(
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyInt()
|
||||
)
|
||||
Mockito.verify(rtmTokenBuilder).buildToken(
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyString(),
|
||||
Mockito.anyInt()
|
||||
)
|
||||
}
|
||||
|
||||
private fun createRoom(
|
||||
owner: Member,
|
||||
price: Int = 0,
|
||||
isAvailableJoinCreator: Boolean = false,
|
||||
genderRestriction: GenderRestriction = GenderRestriction.ALL
|
||||
): LiveRoom {
|
||||
return LiveRoom(
|
||||
title = "test room",
|
||||
notice = "test notice",
|
||||
beginDateTime = LocalDateTime.now(),
|
||||
numberOfPeople = 10,
|
||||
coverImage = "cover/image.png",
|
||||
isAdult = false,
|
||||
price = price,
|
||||
isAvailableJoinCreator = isAvailableJoinCreator,
|
||||
genderRestriction = genderRestriction
|
||||
).also {
|
||||
it.id = 100L
|
||||
it.member = owner
|
||||
}
|
||||
}
|
||||
|
||||
private fun createMember(
|
||||
id: Long,
|
||||
role: MemberRole = MemberRole.USER,
|
||||
gender: Gender = Gender.NONE
|
||||
): Member {
|
||||
return Member(
|
||||
email = "member$id@test.com",
|
||||
password = "password",
|
||||
nickname = "member$id",
|
||||
gender = gender,
|
||||
role = role
|
||||
).also { it.id = id }
|
||||
}
|
||||
|
||||
private inline fun <reified T> mock(): T {
|
||||
return Mockito.mock(T::class.java)
|
||||
}
|
||||
}
|
||||
+2
-2
@@ -29,7 +29,7 @@ class MemberContentPreferenceIntegrationTest @Autowired constructor(
|
||||
private val entityManager: EntityManager
|
||||
) {
|
||||
companion object {
|
||||
private val FORCED_MEMBER_IDS = setOf(2L, 16L, 17L, 29721L, 32050L, 37543L, 40850L)
|
||||
private val FORCED_MEMBER_IDS = setOf(16L, 17L, 29721L, 32050L, 37543L, 40850L)
|
||||
}
|
||||
|
||||
private lateinit var service: MemberContentPreferenceService
|
||||
@@ -184,7 +184,7 @@ class MemberContentPreferenceIntegrationTest @Autowired constructor(
|
||||
countryContext.setCountryCode("US")
|
||||
|
||||
val jpMember = Member(email = "jp@test.com", password = "password", nickname = "jp-member")
|
||||
.apply { id = 2L }
|
||||
.apply { id = 29721L }
|
||||
val jpMemberNew = Member(email = "jp-new@test.com", password = "password", nickname = "jp-member-new")
|
||||
.apply { id = 37543L }
|
||||
val krMember = Member(email = "kr@test.com", password = "password", nickname = "kr-member")
|
||||
|
||||
-89
@@ -1,89 +0,0 @@
|
||||
package kr.co.vividnext.sodalive.member.contentpreference
|
||||
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import org.junit.jupiter.api.AfterEach
|
||||
import org.junit.jupiter.api.Assertions.assertEquals
|
||||
import org.junit.jupiter.api.Assertions.assertFalse
|
||||
import org.junit.jupiter.api.Assertions.assertTrue
|
||||
import org.junit.jupiter.api.DisplayName
|
||||
import org.junit.jupiter.api.Test
|
||||
import org.springframework.mock.web.MockHttpServletRequest
|
||||
import org.springframework.web.context.request.RequestContextHolder
|
||||
import org.springframework.web.context.request.ServletRequestAttributes
|
||||
|
||||
class MemberContentPreferencePolicyTest {
|
||||
@AfterEach
|
||||
fun cleanup() {
|
||||
RequestContextHolder.resetRequestAttributes()
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("요청 국가 헤더를 기준으로 국가 코드를 계산한다")
|
||||
fun shouldResolveCountryCodeByRequestHeader() {
|
||||
setRequestCountry(" us ")
|
||||
val member = createMember(id = 200L, countryCode = "KR")
|
||||
|
||||
assertEquals("US", resolveCountryCodeByPolicy(member))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("강제 매핑 대상 회원 ID는 요청 국가 헤더보다 우선한다")
|
||||
fun shouldPrioritizeForcedCountryMapping() {
|
||||
setRequestCountry("US")
|
||||
|
||||
val forcedJpMember = createMember(id = 2L, countryCode = "KR")
|
||||
val forcedJpMemberNew = createMember(id = 37543L, countryCode = "KR")
|
||||
val forcedKrMember = createMember(id = 16L, countryCode = "US")
|
||||
|
||||
assertEquals("JP", resolveCountryCodeByPolicy(forcedJpMember))
|
||||
assertEquals("JP", resolveCountryCodeByPolicy(forcedJpMemberNew))
|
||||
assertEquals("KR", resolveCountryCodeByPolicy(forcedKrMember))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("요청 국가가 KR이면 인증 미완료 사용자는 성인 노출이 false다")
|
||||
fun shouldHideAdultContentForKrWithoutAuth() {
|
||||
setRequestCountry("KR")
|
||||
val member = createMember(id = 1L, countryCode = "US")
|
||||
|
||||
assertFalse(isAdultVisibleByPolicy(member, isAdultContentVisible = true))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("요청 국가가 KR이 아니면 멤버 countryCode와 무관하게 전달값을 사용한다")
|
||||
fun shouldIgnoreStoredCountryCodeWhenRequestCountryIsNotKr() {
|
||||
setRequestCountry("US")
|
||||
val member = createMember(id = 201L, countryCode = "KR")
|
||||
|
||||
assertTrue(isAdultVisibleByPolicy(member, isAdultContentVisible = true))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("요청 컨텍스트가 없으면 KR fallback 정책을 사용한다")
|
||||
fun shouldFallbackToKrWhenRequestContextIsMissing() {
|
||||
RequestContextHolder.resetRequestAttributes()
|
||||
val member = createMember(id = 202L, countryCode = "US")
|
||||
|
||||
assertEquals("KR", resolveCountryCodeByPolicy(member))
|
||||
assertFalse(isAdultVisibleByPolicy(member, isAdultContentVisible = true))
|
||||
}
|
||||
|
||||
private fun setRequestCountry(countryCode: String?) {
|
||||
val request = MockHttpServletRequest()
|
||||
if (countryCode != null) {
|
||||
request.addHeader("CloudFront-Viewer-Country", countryCode)
|
||||
}
|
||||
RequestContextHolder.setRequestAttributes(ServletRequestAttributes(request))
|
||||
}
|
||||
|
||||
private fun createMember(id: Long, countryCode: String?): Member {
|
||||
return Member(
|
||||
email = "member$id@test.com",
|
||||
password = "password",
|
||||
nickname = "member$id"
|
||||
).apply {
|
||||
this.id = id
|
||||
this.countryCode = countryCode
|
||||
}
|
||||
}
|
||||
}
|
||||
+54
-2
@@ -61,10 +61,10 @@ class MemberContentPreferenceServiceTest {
|
||||
@Test
|
||||
@DisplayName("회원 ID 강제 매핑(JP)이 헤더보다 우선 적용된다")
|
||||
fun shouldResolveCountryCodeByForcedJapanMappingFirst() {
|
||||
val member = createMember(id = 2L)
|
||||
val member = createMember(id = 29721L)
|
||||
val preference = createPreference(member)
|
||||
countryContext.setCountryCode("US")
|
||||
Mockito.`when`(repository.findByMemberId(2L)).thenReturn(preference)
|
||||
Mockito.`when`(repository.findByMemberId(29721L)).thenReturn(preference)
|
||||
|
||||
val result = service.getStoredPreference(member)
|
||||
|
||||
@@ -449,6 +449,58 @@ class MemberContentPreferenceServiceTest {
|
||||
assertTrue(service.calculateIsAdultForQuery(noAuthMember, "US", true))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("로그인 회원의 접속 국가 코드는 공백과 소문자를 정규화하고 없으면 KR로 계산된다")
|
||||
fun shouldNormalizeCountryCodeForLoggedInMember() {
|
||||
val member = createMember(id = 2400L)
|
||||
|
||||
countryContext.setCountryCode(" us ")
|
||||
assertEquals("US", service.resolveCountryCode(member))
|
||||
|
||||
countryContext.setCountryCode(" ")
|
||||
assertEquals("KR", service.resolveCountryCode(member))
|
||||
|
||||
countryContext.setCountryCode(null)
|
||||
assertEquals("KR", service.resolveCountryCode(member))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("조회용 성인 노출 여부는 접속 국가와 인증 여부로 계산된다")
|
||||
fun shouldCalculateIsAdultVisibleForQueryByCountryAndAuth() {
|
||||
val noAuthMember = createMember(id = 2300L, withAuth = false)
|
||||
val authMember = createMember(id = 2301L, withAuth = true)
|
||||
|
||||
countryContext.setCountryCode("KR")
|
||||
assertFalse(service.isAdultVisibleForQuery(noAuthMember, isAdultContentVisible = true))
|
||||
assertTrue(service.isAdultVisibleForQuery(authMember, isAdultContentVisible = true))
|
||||
|
||||
countryContext.setCountryCode("US")
|
||||
assertTrue(service.isAdultVisibleForQuery(noAuthMember, isAdultContentVisible = true))
|
||||
assertFalse(service.isAdultVisibleForQuery(authMember, isAdultContentVisible = false))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("비KR 요청에서는 회원의 저장 국가와 무관하게 전달한 성인 노출값을 사용한다")
|
||||
fun shouldIgnoreStoredCountryCodeWhenRequestCountryIsNotKr() {
|
||||
val member = createMember(id = 2401L, withAuth = false).apply {
|
||||
countryCode = "KR"
|
||||
}
|
||||
countryContext.setCountryCode("US")
|
||||
|
||||
assertEquals("US", service.resolveCountryCode(member))
|
||||
assertTrue(service.isAdultVisibleForQuery(member, isAdultContentVisible = true))
|
||||
assertFalse(service.isAdultVisibleForQuery(member, isAdultContentVisible = false))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("조회용 성인 노출 여부도 회원 ID 강제 매핑이 헤더보다 우선 적용된다")
|
||||
fun shouldApplyForcedCountryMappingForIsAdultVisibleForQuery() {
|
||||
val forcedKrMember = createMember(id = 16L, withAuth = false)
|
||||
countryContext.setCountryCode("US")
|
||||
|
||||
assertFalse(service.isAdultVisibleForQuery(forcedKrMember, isAdultContentVisible = true))
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("성인 콘텐츠 조회 가능 여부는 저장 preference의 조회용 성인 정책 결과를 반환한다")
|
||||
fun shouldReturnStoredPreferenceAdultPolicyForCanViewAdultContent() {
|
||||
|
||||
+4
-1
@@ -3,6 +3,7 @@ package kr.co.vividnext.sodalive.v2.api.home
|
||||
import kr.co.vividnext.sodalive.chat.character.ChatCharacter
|
||||
import kr.co.vividnext.sodalive.explorer.profile.creatorCommunity.CreatorCommunity
|
||||
import kr.co.vividnext.sodalive.live.room.LiveRoom
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberAdapter
|
||||
import kr.co.vividnext.sodalive.member.MemberKind
|
||||
@@ -372,7 +373,9 @@ class HomeRecommendationControllerTest @Autowired constructor(
|
||||
offset = 0,
|
||||
limit = 21,
|
||||
memberId = member.id,
|
||||
includeAdultLives = false
|
||||
includeAdultLives = false,
|
||||
isViewerCreator = false,
|
||||
effectiveViewerGender = Gender.NONE
|
||||
)
|
||||
)
|
||||
.thenThrow(IllegalStateException("page query failed"))
|
||||
|
||||
+49
-2
@@ -2,7 +2,10 @@ package kr.co.vividnext.sodalive.v2.api.home.application
|
||||
|
||||
import kr.co.vividnext.sodalive.i18n.Lang
|
||||
import kr.co.vividnext.sodalive.i18n.LangContext
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberRole
|
||||
import kr.co.vividnext.sodalive.member.auth.Auth
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.v2.creator.channel.community.translation.application.CreatorCommunityTranslationService
|
||||
import kr.co.vividnext.sodalive.v2.recommendation.application.HomeRecommendationQueryService
|
||||
@@ -86,6 +89,42 @@ class HomeRecommendationFacadeTest {
|
||||
assertTrue(queryPort.bannerLang == Lang.KO)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("홈 추천은 크리에이터 여부와 인증 성별 우선 유효 성별을 라이브 조회에 전달한다")
|
||||
fun shouldPassCreatorRoleAndAuthGenderToHomeLiveQuery() {
|
||||
val member = createMember(id = 12L, role = MemberRole.CREATOR, gender = Gender.FEMALE, authGender = 1)
|
||||
|
||||
facade.getHomeRecommendations(member)
|
||||
|
||||
assertTrue(queryPort.liveIsViewerCreator == true)
|
||||
assertTrue(queryPort.liveEffectiveViewerGender == Gender.MALE)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("홈 라이브 목록은 크리에이터 여부와 회원 성별 fallback을 라이브 조회에 전달한다")
|
||||
fun shouldPassCreatorRoleAndMemberGenderToLivePageQuery() {
|
||||
val member = createMember(id = 13L, role = MemberRole.CREATOR, gender = Gender.FEMALE)
|
||||
|
||||
facade.getLives(member, page = 0, size = 20)
|
||||
|
||||
assertTrue(queryPort.liveIsViewerCreator == true)
|
||||
assertTrue(queryPort.liveEffectiveViewerGender == Gender.FEMALE)
|
||||
}
|
||||
|
||||
private fun createMember(
|
||||
id: Long,
|
||||
role: MemberRole = MemberRole.USER,
|
||||
gender: Gender = Gender.NONE,
|
||||
authGender: Int? = null
|
||||
): Member {
|
||||
return Member(password = "password", nickname = "member$id", role = role, gender = gender).apply {
|
||||
this.id = id
|
||||
authGender?.let {
|
||||
Auth(name = "name", birth = "19900101", uniqueCi = "ci$id", di = "di$id", gender = it).member = this
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun firstAudio(price: Int, isPointAvailable: Boolean): HomeFirstAudioContentRecord {
|
||||
return HomeFirstAudioContentRecord(
|
||||
contentId = price.toLong() + 1,
|
||||
@@ -105,13 +144,21 @@ class HomeRecommendationFacadeTest {
|
||||
var firstAudioContents: List<HomeFirstAudioContentRecord> = emptyList()
|
||||
var bannerIncludeAdultBanners: Boolean = false
|
||||
var bannerLang: Lang? = null
|
||||
var liveIsViewerCreator: Boolean? = null
|
||||
var liveEffectiveViewerGender: Gender? = null
|
||||
|
||||
override fun findLiveRecommendations(
|
||||
offset: Long,
|
||||
limit: Int,
|
||||
memberId: Long?,
|
||||
includeAdultLives: Boolean
|
||||
): List<HomeLiveRecommendationRecord> = emptyList()
|
||||
includeAdultLives: Boolean,
|
||||
isViewerCreator: Boolean,
|
||||
effectiveViewerGender: Gender?
|
||||
): List<HomeLiveRecommendationRecord> {
|
||||
liveIsViewerCreator = isViewerCreator
|
||||
liveEffectiveViewerGender = effectiveViewerGender
|
||||
return emptyList()
|
||||
}
|
||||
|
||||
override fun findHomeBanners(
|
||||
limit: Int,
|
||||
|
||||
+64
-7
@@ -1,7 +1,9 @@
|
||||
package kr.co.vividnext.sodalive.v2.api.home.live.application
|
||||
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberRole
|
||||
import kr.co.vividnext.sodalive.member.auth.Auth
|
||||
import kr.co.vividnext.sodalive.member.contentpreference.MemberContentPreferenceService
|
||||
import kr.co.vividnext.sodalive.v2.recommendation.application.HomeRecommendationQueryService
|
||||
import kr.co.vividnext.sodalive.v2.recommendation.port.out.HomeLiveRecommendationRecord
|
||||
@@ -25,7 +27,9 @@ class HomeOnAirLiveFacadeTest {
|
||||
eqValue(0L),
|
||||
eqValue(21),
|
||||
eqValue(member.id),
|
||||
eqValue(true)
|
||||
eqValue(true),
|
||||
eqValue(false),
|
||||
eqValue(Gender.NONE)
|
||||
)
|
||||
|
||||
val response = facade.getOnAirLives(member, page = 0)
|
||||
@@ -34,7 +38,14 @@ class HomeOnAirLiveFacadeTest {
|
||||
assertEquals(20, response.size)
|
||||
assertEquals(true, response.hasNext)
|
||||
assertEquals(20, response.items.size)
|
||||
Mockito.verify(queryService).findLiveRecommendations(eqValue(0L), eqValue(21), eqValue(member.id), eqValue(true))
|
||||
Mockito.verify(queryService).findLiveRecommendations(
|
||||
eqValue(0L),
|
||||
eqValue(21),
|
||||
eqValue(member.id),
|
||||
eqValue(true),
|
||||
eqValue(false),
|
||||
eqValue(Gender.NONE)
|
||||
)
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -46,7 +57,9 @@ class HomeOnAirLiveFacadeTest {
|
||||
eqValue(0L),
|
||||
eqValue(21),
|
||||
eqValue(member.id),
|
||||
eqValue(false)
|
||||
eqValue(false),
|
||||
eqValue(false),
|
||||
eqValue(Gender.NONE)
|
||||
)
|
||||
|
||||
val response = facade.getOnAirLives(member, page = 0)
|
||||
@@ -60,13 +73,46 @@ class HomeOnAirLiveFacadeTest {
|
||||
val member = createMember(100L)
|
||||
Mockito.doReturn(false).`when`(preferenceService).canViewAdultContent(member)
|
||||
Mockito.doReturn(listOf(record(1L, beginDateTime = LocalDateTime.of(2026, 6, 26, 12, 30)))).`when`(queryService)
|
||||
.findLiveRecommendations(eqValue(0L), eqValue(21), eqValue(member.id), eqValue(false))
|
||||
.findLiveRecommendations(
|
||||
eqValue(0L),
|
||||
eqValue(21),
|
||||
eqValue(member.id),
|
||||
eqValue(false),
|
||||
eqValue(false),
|
||||
eqValue(Gender.NONE)
|
||||
)
|
||||
|
||||
val response = facade.getOnAirLives(member, page = 0)
|
||||
|
||||
assertEquals("2026-06-26T12:30:00Z", response.items.single().beginDateTimeUtc)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("현재 진행 중인 라이브 facade는 크리에이터 여부와 인증 성별 우선 유효 성별을 전달한다")
|
||||
fun shouldPassCreatorRoleAndAuthGenderToLiveRecommendationQuery() {
|
||||
val member = createMember(id = 100L, role = MemberRole.CREATOR, gender = Gender.FEMALE, authGender = 1)
|
||||
Mockito.doReturn(true).`when`(preferenceService).canViewAdultContent(member)
|
||||
Mockito.doReturn(emptyList<HomeLiveRecommendationRecord>()).`when`(queryService).findLiveRecommendations(
|
||||
eqValue(0L),
|
||||
eqValue(21),
|
||||
eqValue(member.id),
|
||||
eqValue(true),
|
||||
eqValue(true),
|
||||
eqValue(Gender.MALE)
|
||||
)
|
||||
|
||||
facade.getOnAirLives(member, page = 0)
|
||||
|
||||
Mockito.verify(queryService).findLiveRecommendations(
|
||||
eqValue(0L),
|
||||
eqValue(21),
|
||||
eqValue(member.id),
|
||||
eqValue(true),
|
||||
eqValue(true),
|
||||
eqValue(Gender.MALE)
|
||||
)
|
||||
}
|
||||
|
||||
private fun record(
|
||||
id: Long,
|
||||
creatorProfileImage: String? = "profile.png",
|
||||
@@ -80,13 +126,24 @@ class HomeOnAirLiveFacadeTest {
|
||||
beginDateTime = beginDateTime
|
||||
)
|
||||
|
||||
private fun createMember(id: Long): Member {
|
||||
private fun createMember(
|
||||
id: Long,
|
||||
role: MemberRole = MemberRole.USER,
|
||||
gender: Gender = Gender.NONE,
|
||||
authGender: Int? = null
|
||||
): Member {
|
||||
return Member(
|
||||
email = "viewer$id@test.com",
|
||||
password = "password",
|
||||
nickname = "viewer$id",
|
||||
role = MemberRole.USER
|
||||
).apply { this.id = id }
|
||||
role = role,
|
||||
gender = gender
|
||||
).apply {
|
||||
this.id = id
|
||||
authGender?.let {
|
||||
Auth(name = "name", birth = "19900101", uniqueCi = "ci$id", di = "di$id", gender = it).member = this
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private fun <T> eqValue(value: T): T {
|
||||
|
||||
+27
@@ -239,6 +239,33 @@ class DefaultCreatorChannelLiveQueryRepositoryTest @Autowired constructor(
|
||||
assertEquals(visible.id, live!!.liveId)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("채널 소유자는 크리에이터 입장 제한이 설정된 본인 현재 라이브를 조회한다")
|
||||
fun shouldReturnRestrictedCurrentLiveForChannelOwner() {
|
||||
val now = LocalDateTime.of(2026, 6, 17, 12, 0)
|
||||
val creator = saveMember("restricted-live-owner", MemberRole.CREATOR)
|
||||
val restrictedLive = saveLiveRoom(
|
||||
creator = creator,
|
||||
beginDateTime = now.minusMinutes(1),
|
||||
channelName = "restricted-owner-live",
|
||||
isAdult = false,
|
||||
genderRestriction = GenderRestriction.MALE_ONLY,
|
||||
isAvailableJoinCreator = false
|
||||
)
|
||||
flushAndClear()
|
||||
|
||||
val live = repository.findCurrentLive(
|
||||
creatorId = creator.id!!,
|
||||
now = now,
|
||||
canViewAdultContent = false,
|
||||
viewerId = creator.id!!,
|
||||
isViewerCreator = true,
|
||||
effectiveViewerGender = Gender.MALE
|
||||
)
|
||||
|
||||
assertEquals(restrictedLive.id, live!!.liveId)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("크리에이터 조회와 차단 관계 조회는 live service port 계약을 만족한다")
|
||||
fun shouldFindCreatorAndBlockedRelationship() {
|
||||
|
||||
+22
-1
@@ -62,6 +62,25 @@ class CreatorChannelLiveQueryServiceTest {
|
||||
assertEquals("https://cdn.test/audio/1.png", tab.liveReplayContents.first().imageUrl)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("다른 크리에이터가 조회하면 현재 라이브 조회에 크리에이터 여부를 전달한다")
|
||||
fun shouldPassCreatorViewerStatusToCurrentLiveQuery() {
|
||||
val port = FakeCreatorChannelLiveQueryPort()
|
||||
val service = createService(port)
|
||||
val viewer = createMember(id = 10L, role = MemberRole.CREATOR)
|
||||
|
||||
service.getLiveTab(
|
||||
creatorId = 1L,
|
||||
viewer = viewer,
|
||||
sort = ContentSort.LATEST,
|
||||
page = 0,
|
||||
size = 20,
|
||||
now = LocalDateTime.of(2026, 6, 17, 10, 0)
|
||||
)
|
||||
|
||||
assertTrue(port.currentLiveIsViewerCreator == true)
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 탭 서비스는 size + 1개 조회 결과를 응답 size로 제한하고 hasNext를 true로 반환한다")
|
||||
fun shouldAssembleLiveTabWithHasNextWhenFetchedMoreThanRequestedSize() {
|
||||
@@ -236,7 +255,8 @@ class CreatorChannelLiveQueryServiceTest {
|
||||
private fun createMember(
|
||||
id: Long,
|
||||
gender: Gender = Gender.NONE,
|
||||
authGender: Int? = null
|
||||
authGender: Int? = null,
|
||||
role: MemberRole = MemberRole.USER
|
||||
): Member {
|
||||
val member = Member(
|
||||
email = "member$id@test.com",
|
||||
@@ -246,6 +266,7 @@ class CreatorChannelLiveQueryServiceTest {
|
||||
gender = gender
|
||||
)
|
||||
member.id = id
|
||||
member.role = role
|
||||
authGender?.let {
|
||||
Auth(
|
||||
name = "name",
|
||||
|
||||
+196
-2
@@ -32,7 +32,9 @@ import kr.co.vividnext.sodalive.explorer.profile.creatorCommunity.CreatorCommuni
|
||||
import kr.co.vividnext.sodalive.explorer.profile.creatorCommunity.comment.CreatorCommunityComment
|
||||
import kr.co.vividnext.sodalive.explorer.profile.creatorCommunity.like.CreatorCommunityLike
|
||||
import kr.co.vividnext.sodalive.i18n.Lang
|
||||
import kr.co.vividnext.sodalive.live.room.GenderRestriction
|
||||
import kr.co.vividnext.sodalive.live.room.LiveRoom
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.member.Member
|
||||
import kr.co.vividnext.sodalive.member.MemberKind
|
||||
import kr.co.vividnext.sodalive.member.MemberRole
|
||||
@@ -194,6 +196,194 @@ class DefaultHomeRecommendationQueryRepositoryTest @Autowired constructor(
|
||||
assertEquals(listOf(visibleLive.id), lives.map { it.liveRoomId })
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 추천은 다른 크리에이터에게 크리에이터 입장 제한 라이브를 제외하고 방 생성자는 유지한다")
|
||||
fun shouldExcludeCreatorJoinRestrictedLiveForOtherCreatorButKeepRoomOwner() {
|
||||
val baseAt = LocalDateTime.of(2026, 9, 17, 10, 0)
|
||||
val viewer = saveMember("creator-viewer", MemberRole.CREATOR)
|
||||
val otherCreator = saveMember("restricted-live-creator", MemberRole.CREATOR)
|
||||
val availableLive = saveLiveRoom(otherCreator, baseAt, channelName = "available-live")
|
||||
saveLiveRoom(
|
||||
otherCreator,
|
||||
baseAt.plusMinutes(1),
|
||||
channelName = "creator-restricted-live",
|
||||
isAvailableJoinCreator = false
|
||||
)
|
||||
val ownerLive = saveLiveRoom(
|
||||
viewer,
|
||||
baseAt.plusMinutes(2),
|
||||
channelName = "owner-restricted-live",
|
||||
isAvailableJoinCreator = false
|
||||
)
|
||||
flushAndClear()
|
||||
|
||||
val lives = repository.findLiveRecommendations(
|
||||
limit = 10,
|
||||
memberId = viewer.id,
|
||||
isViewerCreator = true,
|
||||
effectiveViewerGender = Gender.NONE
|
||||
)
|
||||
|
||||
assertEquals(listOf(ownerLive.id, availableLive.id), lives.map { it.liveRoomId })
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 추천은 일반 사용자에게 크리에이터 입장 제한 라이브를 노출한다")
|
||||
fun shouldKeepCreatorJoinRestrictedLiveVisibleForUserViewer() {
|
||||
val viewer = saveMember("user-viewer", MemberRole.USER)
|
||||
val creator = saveMember("creator-restricted-live-creator", MemberRole.CREATOR)
|
||||
val restrictedLive = saveLiveRoom(
|
||||
creator,
|
||||
LocalDateTime.of(2026, 9, 17, 10, 0),
|
||||
channelName = "creator-restricted-live",
|
||||
isAvailableJoinCreator = false
|
||||
)
|
||||
flushAndClear()
|
||||
|
||||
val lives = repository.findLiveRecommendations(
|
||||
limit = 10,
|
||||
memberId = viewer.id,
|
||||
isViewerCreator = false,
|
||||
effectiveViewerGender = Gender.NONE
|
||||
)
|
||||
|
||||
assertEquals(listOf(restrictedLive.id), lives.map { it.liveRoomId })
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 추천은 남성에게 남성 전용과 전체 라이브를 노출하고 방 생성자는 성별 제한에서 예외 처리한다")
|
||||
fun shouldFilterFemaleOnlyLiveForMaleViewerButKeepOwnerLive() {
|
||||
val baseAt = LocalDateTime.of(2026, 9, 17, 10, 0)
|
||||
val viewer = saveMember("male-viewer", MemberRole.CREATOR)
|
||||
val creator = saveMember("gender-live-creator", MemberRole.CREATOR)
|
||||
val maleOnlyLive = saveLiveRoom(
|
||||
creator,
|
||||
baseAt,
|
||||
channelName = "male-only-live",
|
||||
genderRestriction = GenderRestriction.MALE_ONLY
|
||||
)
|
||||
val allLive = saveLiveRoom(creator, baseAt.plusMinutes(1), channelName = "all-live")
|
||||
saveLiveRoom(
|
||||
creator,
|
||||
baseAt.plusMinutes(2),
|
||||
channelName = "female-only-live",
|
||||
genderRestriction = GenderRestriction.FEMALE_ONLY
|
||||
)
|
||||
val ownerLive = saveLiveRoom(
|
||||
viewer,
|
||||
baseAt.plusMinutes(3),
|
||||
channelName = "owner-female-only-live",
|
||||
genderRestriction = GenderRestriction.FEMALE_ONLY
|
||||
)
|
||||
flushAndClear()
|
||||
|
||||
val lives = repository.findLiveRecommendations(
|
||||
limit = 10,
|
||||
memberId = viewer.id,
|
||||
isViewerCreator = true,
|
||||
effectiveViewerGender = Gender.MALE
|
||||
)
|
||||
|
||||
assertEquals(listOf(ownerLive.id, allLive.id, maleOnlyLive.id), lives.map { it.liveRoomId })
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 추천은 여성 방 생성자에게 남성 전용인 자신의 라이브를 노출한다")
|
||||
fun shouldKeepMaleOnlyLiveVisibleForFemaleRoomOwner() {
|
||||
val baseAt = LocalDateTime.of(2026, 9, 17, 10, 0)
|
||||
val viewer = saveMember("female-owner", MemberRole.CREATOR)
|
||||
val creator = saveMember("female-owner-filter-creator", MemberRole.CREATOR)
|
||||
val femaleOnlyLive = saveLiveRoom(
|
||||
creator,
|
||||
baseAt.plusMinutes(1),
|
||||
channelName = "female-only-live",
|
||||
genderRestriction = GenderRestriction.FEMALE_ONLY
|
||||
)
|
||||
val ownerLive = saveLiveRoom(
|
||||
viewer,
|
||||
baseAt.plusMinutes(2),
|
||||
channelName = "owner-male-only-live",
|
||||
genderRestriction = GenderRestriction.MALE_ONLY
|
||||
)
|
||||
flushAndClear()
|
||||
|
||||
val lives = repository.findLiveRecommendations(
|
||||
limit = 10,
|
||||
memberId = viewer.id,
|
||||
isViewerCreator = true,
|
||||
effectiveViewerGender = Gender.FEMALE
|
||||
)
|
||||
|
||||
assertEquals(listOf(ownerLive.id, femaleOnlyLive.id), lives.map { it.liveRoomId })
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 추천은 여성에게 여성 전용과 전체 라이브만 노출한다")
|
||||
fun shouldFilterMaleOnlyLiveForFemaleViewer() {
|
||||
val baseAt = LocalDateTime.of(2026, 9, 17, 10, 0)
|
||||
val viewer = saveMember("female-viewer", MemberRole.USER)
|
||||
val creator = saveMember("female-filter-creator", MemberRole.CREATOR)
|
||||
val femaleOnlyLive = saveLiveRoom(
|
||||
creator,
|
||||
baseAt.plusMinutes(2),
|
||||
channelName = "female-only-live",
|
||||
genderRestriction = GenderRestriction.FEMALE_ONLY
|
||||
)
|
||||
val allLive = saveLiveRoom(creator, baseAt.plusMinutes(1), channelName = "all-live")
|
||||
saveLiveRoom(
|
||||
creator,
|
||||
baseAt,
|
||||
channelName = "male-only-live",
|
||||
genderRestriction = GenderRestriction.MALE_ONLY
|
||||
)
|
||||
flushAndClear()
|
||||
|
||||
val lives = repository.findLiveRecommendations(
|
||||
limit = 10,
|
||||
memberId = viewer.id,
|
||||
effectiveViewerGender = Gender.FEMALE
|
||||
)
|
||||
|
||||
assertEquals(listOf(femaleOnlyLive.id, allLive.id), lives.map { it.liveRoomId })
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 추천은 성별 미설정 또는 유효 성별이 없으면 성별 제한을 적용하지 않는다")
|
||||
fun shouldNotApplyGenderRestrictionForNoneOrNullGender() {
|
||||
val baseAt = LocalDateTime.of(2026, 9, 17, 10, 0)
|
||||
val viewer = saveMember("unrestricted-viewer", MemberRole.USER)
|
||||
val creator = saveMember("unrestricted-creator", MemberRole.CREATOR)
|
||||
val maleOnlyLive = saveLiveRoom(
|
||||
creator,
|
||||
baseAt,
|
||||
channelName = "male-only-live",
|
||||
genderRestriction = GenderRestriction.MALE_ONLY
|
||||
)
|
||||
val femaleOnlyLive = saveLiveRoom(
|
||||
creator,
|
||||
baseAt.plusMinutes(1),
|
||||
channelName = "female-only-live",
|
||||
genderRestriction = GenderRestriction.FEMALE_ONLY
|
||||
)
|
||||
val allLive = saveLiveRoom(creator, baseAt.plusMinutes(2), channelName = "all-live")
|
||||
flushAndClear()
|
||||
|
||||
val noneGenderLives = repository.findLiveRecommendations(
|
||||
limit = 10,
|
||||
memberId = viewer.id,
|
||||
effectiveViewerGender = Gender.NONE
|
||||
)
|
||||
val nullGenderLives = repository.findLiveRecommendations(
|
||||
limit = 10,
|
||||
memberId = viewer.id,
|
||||
effectiveViewerGender = null
|
||||
)
|
||||
|
||||
val expectedIds = listOf(allLive.id, femaleOnlyLive.id, maleOnlyLive.id)
|
||||
assertEquals(expectedIds, noneGenderLives.map { it.liveRoomId })
|
||||
assertEquals(expectedIds, nullGenderLives.map { it.liveRoomId })
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("홈 배너는 활성 배너를 orders 오름차순 최대 20개로 조회하고 동일 orders는 랜덤 tie-breaker를 적용한다")
|
||||
fun shouldFindActiveHomeBannersWithOrderLimitAndRandomTieBreaker() {
|
||||
@@ -2856,7 +3046,9 @@ class DefaultHomeRecommendationQueryRepositoryTest @Autowired constructor(
|
||||
isAdult: Boolean = false,
|
||||
title: String = "live-${creator.nickname}-$beginDateTime",
|
||||
price: Int = 0,
|
||||
isActive: Boolean = true
|
||||
isActive: Boolean = true,
|
||||
isAvailableJoinCreator: Boolean = true,
|
||||
genderRestriction: GenderRestriction = GenderRestriction.ALL
|
||||
): LiveRoom {
|
||||
val room = LiveRoom(
|
||||
title = title,
|
||||
@@ -2864,7 +3056,9 @@ class DefaultHomeRecommendationQueryRepositoryTest @Autowired constructor(
|
||||
beginDateTime = beginDateTime,
|
||||
numberOfPeople = 0,
|
||||
isAdult = isAdult,
|
||||
price = price
|
||||
price = price,
|
||||
isAvailableJoinCreator = isAvailableJoinCreator,
|
||||
genderRestriction = genderRestriction
|
||||
)
|
||||
room.member = creator
|
||||
room.channelName = channelName
|
||||
|
||||
+15
-4
@@ -2,6 +2,7 @@ package kr.co.vividnext.sodalive.v2.recommendation.application
|
||||
|
||||
import kr.co.vividnext.sodalive.i18n.Lang
|
||||
import kr.co.vividnext.sodalive.i18n.LangContext
|
||||
import kr.co.vividnext.sodalive.member.Gender
|
||||
import kr.co.vividnext.sodalive.v2.common.domain.CreatorActivityType
|
||||
import kr.co.vividnext.sodalive.v2.creator.channel.community.translation.application.CreatorCommunityDisplayContent
|
||||
import kr.co.vividnext.sodalive.v2.creator.channel.community.translation.application.CreatorCommunityTranslationService
|
||||
@@ -86,19 +87,23 @@ class HomeRecommendationQueryServiceTest {
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("라이브 추천 조회는 paging과 성인 노출 여부를 조회 포트에 그대로 위임한다")
|
||||
fun shouldDelegateLiveRecommendationQueryWithPagingAndAdultFlag() {
|
||||
@DisplayName("라이브 추천 조회는 paging, 성인 노출, 크리에이터 여부와 유효 성별을 조회 포트에 그대로 위임한다")
|
||||
fun shouldDelegateLiveRecommendationQueryWithViewerContext() {
|
||||
val recommendations = service.findLiveRecommendations(
|
||||
offset = 40,
|
||||
limit = 21,
|
||||
memberId = 100L,
|
||||
includeAdultLives = true
|
||||
includeAdultLives = true,
|
||||
isViewerCreator = true,
|
||||
effectiveViewerGender = Gender.MALE
|
||||
)
|
||||
|
||||
assertEquals(40, port.liveOffset)
|
||||
assertEquals(21, port.liveLimit)
|
||||
assertEquals(100L, port.liveMemberId)
|
||||
assertEquals(true, port.liveIncludeAdultLives)
|
||||
assertEquals(true, port.liveIsViewerCreator)
|
||||
assertEquals(Gender.MALE, port.liveEffectiveViewerGender)
|
||||
assertEquals(port.liveRecommendations, recommendations)
|
||||
}
|
||||
|
||||
@@ -1092,6 +1097,8 @@ class HomeRecommendationQueryServiceTest {
|
||||
var liveOffset: Long? = null
|
||||
var liveMemberId: Long? = null
|
||||
var liveIncludeAdultLives: Boolean? = null
|
||||
var liveIsViewerCreator: Boolean? = null
|
||||
var liveEffectiveViewerGender: Gender? = null
|
||||
var bannerLimit: Int? = null
|
||||
var bannerMemberId: Long? = null
|
||||
var bannerIncludeAdultBanners: Boolean? = null
|
||||
@@ -1187,12 +1194,16 @@ class HomeRecommendationQueryServiceTest {
|
||||
offset: Long,
|
||||
limit: Int,
|
||||
memberId: Long?,
|
||||
includeAdultLives: Boolean
|
||||
includeAdultLives: Boolean,
|
||||
isViewerCreator: Boolean,
|
||||
effectiveViewerGender: Gender?
|
||||
): List<HomeLiveRecommendationRecord> {
|
||||
liveOffset = offset
|
||||
liveLimit = limit
|
||||
liveMemberId = memberId
|
||||
liveIncludeAdultLives = includeAdultLives
|
||||
liveIsViewerCreator = isViewerCreator
|
||||
liveEffectiveViewerGender = effectiveViewerGender
|
||||
return liveRecommendations
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user