Skip to content

Test / 고객사 격리·페이지 버전 검증과 포트폴리오 근거 보강 - #144

Merged
eraser502 merged 1 commit into
developfrom
codex/tenant-isolation
Oct 14, 2026
Merged

eraser502 merged 1 commit into
developfrom
codex/tenant-isolation

Conversation

@eraser502

Copy link
Copy Markdown
Contributor

Overview

고객사 격리와 작성본·공개본 분리의 동작을 검증할 회귀 테스트가 부족했습니다. 정상 CMS 로그인 쿠키와 실제 PostgreSQL/Flyway를 사용하는 계약 테스트 16개를 추가해 교차 고객사 접근 차단, 게시 전환·실패 롤백, 페이지 재사용 및 DB 제약을 확인합니다.

AppPage/AppVersion/AppVersionPage의 스냅샷 설계 평가와 포트폴리오 교체 원고도 포함합니다. 기존 DRAFT를 직접 승격하는 구조가 아니라 새 공개 버전을 만드는 구조임을 명시하며, DB 차원의 완전한 격리·불변성이나 성능 향상은 주장하지 않습니다. 운영 코드와 스키마 변경은 없습니다.

Related Issue

Closes #143

PR Checklist

  • ./gradlew test: 41개 성공, 실패·오류·건너뜀 0개
  • 신규 계약 테스트 16개 포함
  • JWT 고객사 대조·페이지 소속 검사·공개 상태 조건의 임시 변이 각각 예상 실패 확인 후 원본 복원
  • 테스트 및 설계·문구 읽기 리뷰: P1/P2 지적 없음
  • 실행 요약 JSON, 재현 방법, 검증 한계, 교체 원고 기록

Additional Information

  • 신규 검증은 로컬 MockMvc + 폐기용 PostgreSQL입니다. 실제 네트워크 HTTP·Ingress·홈서버 검증이나 성능 측정이 아닙니다.
  • 과거 문의번호 HTTP 13건 검증은 별도 근거로 연결했습니다.
  • 공유 페이지 불변성, 중복 path, 삭제 CASCADE, 고객사 FK 및 동시 게시 정책의 한계를 문서화했습니다. 버전 시스템 재설계는 포함하지 않습니다.
  • 기본 테스트에서 제외하는 integration/experiment 태그 작업은 로컬에서 이번에 재실행하지 않았습니다. 기존 PR CI가 전체 빌드와 홈랩 검증을 수행합니다.
  • 원본 포트폴리오 PDF는 보존하고 교체용 원고만 추가했습니다.

@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 77b30db1-c61a-4aa9-8ace-bbb026f37a9a
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@eraser502
eraser502 merged commit 1ae34d1 into develop Oct 14, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test / 고객사 격리와 페이지 버전 계약 검증 및 포트폴리오 근거 정리

1 participant