Skip to content

refactor: Slack 전송을 RestClient로 옮기고 테스트를 실제 요청·응답으로 검증함 - #709

Merged
yoon6yo merged 1 commit into
mainfrom
refactor/v3.3.73-slack-rest-client
Oct 13, 2026
Merged

yoon6yo merged 1 commit into
mainfrom
refactor/v3.3.73-slack-rest-client

Conversation

@yoon6yo

@yoon6yo yoon6yo commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

변경

  • SlackApiService의 웹훅·회원 목록·DM 채널 열기·채널 글 올리기 호출을 RestTemplate에서 RestClient로 전환
  • 시간 제한(연결 3초, 읽기 5초)과 오류 분류는 그대로

테스트

  • 전송 객체를 흉내 내던 방식에서 가짜 HTTP 서버(MockRestServiceServer)로 실제 요청·응답을 주고받는 방식으로 다시 씀
  • 새로 확인되는 것
    • 웹훅 404·410·401·403은 설정 문제로, 400·500은 전송 실패로 분류
    • 회원 목록의 다음 쪽 표시가 한 번만 인코딩되어 나감
    • 연결 실패 때 웹훅 주소가 로그에 남지 않음(RestClient가 실제로 만든 예외로 확인)

확인

  • ./gradlew build --rerun-tasks 통과, 테스트 1332건 실패 0건

확인하지 못한 것

  • 실제 Slack 전송은 해 보지 않았다. 배포 뒤 알림 한 건이 도착하는지 확인한다.

🤖 Generated with Claude Code

- 웹훅·회원 목록·DM 채널 열기·채널 글 올리기 호출을 RestClient로 전환, 시간 제한과 오류 분류는 그대로
- 테스트는 전송 객체를 흉내 내지 않고 가짜 HTTP 서버로 상태 코드별 분류·주소 인코딩·로그 노출을 확인

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@yoon6yo
yoon6yo merged commit 832d396 into main Oct 13, 2026
1 check 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.

1 participant