Skip to content

[Test] STOMP 권한 회수 후 대시보드 세션 경계 검증 - #366

Merged
Gimini-3 merged 4 commits into
developfrom
test/365
Oct 1, 2026
Merged

Gimini-3 merged 4 commits into
developfrom
test/365

Conversation

@Gimini-3

@Gimini-3 Gimini-3 commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

🔍️ 작업 내용

  • Closes [Test] STOMP 권한 변경 후 대시보드 세션 경계 검증 #365
  • 실제 WebSocket으로 권한 회수 전후의 새 연결, 기존 연결의 새 구독, 기존 구독의 push를 분리해 검증합니다.
  • 추가로 실제 관리자 HTTP API, 로컬 PostgreSQL, Redis 캐시 무효화, STOMP 재검증을 한 테스트로 연결했습니다.
  • 제품 코드는 변경하지 않았고 실행 결과를 문서화했습니다.

✨ 상세 설명

  • 결정적 경계 테스트 3건은 역할 전이를 mock으로 제어합니다.
  • 실제 경로 테스트 1건은 임시 ADMIN 사용자 두 명을 만들고 DELETE /admin/users/{id}/roles/ADMIN을 호출합니다. HTTP 200, DB 역할 매핑 삭제, Redis 캐시 삭제·세대 증가를 각각 확인했습니다.
  • 회수 직후 새 세션의 관리자 구독은 거부되지만, 기존 ADMIN 세션은 다음 재검증 전까지 push를 받습니다. 재검증 뒤 기존 세션과 구독이 제거되고 추가 push는 전달되지 않았습니다. 즉시 회수를 보장한다고 주장하지 않습니다.
  • 자동 재검증을 테스트에서만 길게 설정하고 수동 호출해 검사 전후를 구분했습니다. 임시 DB 사용자는 테스트 후 0건이었습니다.
  • 전체 Backend 기본 테스트 첫 실행에서 기존 1초 JWT 만료 테스트가 연결 중 1건 실패했습니다. 해당 클래스와 신규 테스트 재실행은 5건 통과했고, 전체 재실행은 208 suites·1,250 tests, 실패·오류·건너뜀 0건으로 통과했습니다. 첫 실패 원인은 확정하지 않았습니다.
  • 이 결과는 로컬 DB·Redis 검증이며 GCP 3노드 OpenProxy의 primary/standby 라우팅이나 복제 지연은 검증하지 않습니다.

🛠️ 추후 리팩토링 및 고도화 계획

  • 역할 회수 직후 관리자 WebSocket 메시지 0건이 요구된다면, 주기 검사 외의 즉시 무효화 경계를 별도 수정 이슈로 설계합니다.
  • GCP 3노드에서 역할 조회의 primary/standby 최신성은 별도 실험이 필요합니다.

💬 리뷰 요구사항

  • 재검증 전 허용 창을 PASS로 숨기지 않고 현재 계약의 한계로 기록한 점을 확인 부탁드립니다.
  • 실제 HTTP→DB→Redis→STOMP 경로와 mock 기반 경계 테스트의 역할을 분리한 점을 확인 부탁드립니다.

@Gimini-3
Gimini-3 requested a review from kangcheolung October 1, 2026 06:06
@coderabbitai

coderabbitai Bot commented Oct 1, 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: Repository: DocGrid/docgrid/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: f1b47c65-5419-43cd-8bb8-fbecd063f5cd

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

@Gimini-3
Gimini-3 merged commit f6213bb into develop Oct 1, 2026
1 check passed
@Gimini-3 Gimini-3 self-assigned this Oct 1, 2026
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] STOMP 권한 변경 후 대시보드 세션 경계 검증

1 participant