Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SBL-150] object class로의 리팩터링 #69

Merged
merged 4 commits into from
Sep 26, 2024

Conversation

GulSauce
Copy link
Member

@GulSauce GulSauce commented Sep 24, 2024

📢 설명

프로퍼티 없이 companion object만 사용하는 클래스를 object 클래스로 변경
+) 시큐리티에 로그인 한 유저만 api/v1/ai/**에 요청 보낼 수 있게하는 설정 추가

✅ 체크 리스트

(프론트엔드 페이제를 이용하여 테스트하면 편리합니다)

  • 로그인 후 로그아웃을 수행 했을 때 JSESSIONID, user-profile 쿠키가 잘 지워지는지 확인
  • 회원가입을 하면 랜덤 닉네임이 잘 만들어지는 지 확인
  • 로그아웃후 /api/v1/admin/sessions/logged-in-users로 조회시 로그아웃된 그 계정이 조회되지 않는지 확인
  • 로컬 터미널로(세션 쿠키가 없기 때문에)
curl -X 'POST' \
  'http://localhost:8080/api/v1/ai/generate/survey' \
  -H 'accept: application/json;charset=UTF-8' \
  -H 'Content-Type: application/json;charset=UTF-8' \
  -d '{
  "job": "string",
  "groupName": "string",
  "fileUrl": "string"
}'

명령 보냈을 때

{"code":"GL0003","message":"로그인이 필요합니다.","errors":[]}

수신되는지 확인

  • 스웨거 페이지에서(세션 쿠키가 있기 때문에)
{
  "job": "string",
  "groupName": "string",
  "fileUrl": "string"
}

요청 보냈을 때

{
  "code": "FV0007",
  "message": "유효하지 않은 파일 주소입니다.",
  "errors": []
}

수신되는지 확인

@GulSauce GulSauce changed the title [SBL-150] 리팩터링 완료 [SBL-150] object class로의 리팩터링 Sep 24, 2024
@GulSauce GulSauce self-assigned this Sep 24, 2024
Copy link
Member

@JeongHunHui JeongHunHui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

체크리스트 다 잘 작동합니다! 코드도 LGTM입니다~

@GulSauce GulSauce force-pushed the SBL-150-to-object-class-refactor branch from 143a88d to b1545ca Compare September 25, 2024 16:16
Copy link

테스트 커버리지 리포트

Overall Project 25.84% -3.63% 🍏
Files changed 0% 🍏

File Coverage
SessionRegistryCleaner.kt 0% -72% 🍏
ResetSession.kt 0% -82.5% 🍏
RandomNicknameGenerator.kt 0% -99.13% 🍏
SecurityConfig.kt 0% -2.16% 🍏

@SUIN-BUNDANG-LINE SUIN-BUNDANG-LINE deleted a comment from github-actions bot Sep 25, 2024
@SUIN-BUNDANG-LINE SUIN-BUNDANG-LINE deleted a comment from github-actions bot Sep 25, 2024
Copy link
Member

@JeongHunHui JeongHunHui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

새로 추가된 변경사항도 잘 적용되었습니다~ 고생하셨습니다!

@GulSauce GulSauce merged commit d179c93 into develop Sep 26, 2024
1 check passed
@GulSauce GulSauce deleted the SBL-150-to-object-class-refactor branch September 26, 2024 04:35
Copy link

테스트 커버리지 리포트

Overall Project 25.84% -3.63% 🍏
Files changed 0% 🍏

File Coverage
SessionRegistryCleaner.kt 0% -72% 🍏
ResetSession.kt 0% -82.5% 🍏
RandomNicknameGenerator.kt 0% -99.13% 🍏
SecurityConfig.kt 0% -2.16% 🍏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants