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

[Refactor] Post/Tab 서비스 테스트 코드 변경 #103

Merged
merged 10 commits into from
Jun 26, 2024
Merged

Conversation

Sosohy
Copy link
Collaborator

@Sosohy Sosohy commented Jun 25, 2024

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 코드 및 구조 개선

반영 브랜치

feat/post -> dev

변경 사항

  • db에 값이 없을 경우 통과하지 못하는 테스트 코드들이 존재하여 해당 부분을 수정
  • 게시글 콘텐츠 처리하는 함수에서 개행처리를 치환하여 처리했던 부분을 outputSetting을 활용하여 변경

테스트 결과

전체 통과 완료~

Sosohy added 9 commits June 19, 2024 10:29
포스트 생성 시 마다 tab 생성 및 할당으로 변경, 서비스에서 사용자 정보를 whoAmI()로 받아서 사용함으로 null로 수정
사용자 참여 게시글까지 수정 완료
DB값 상관없이 insert하며 테스트 진행
회원 아이디 null로 변경 -> 해당 서비스에서 whoAmI() 사용하므로 불필요
db에 데이터 없어도 insert 후 동작하도록 수정
@Sosohy Sosohy added the refactor Improve code or structure label Jun 25, 2024
@Sosohy Sosohy self-assigned this Jun 25, 2024
Copy link

cr-gpt bot commented Jun 25, 2024

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

Copy link

cr-gpt bot commented Jun 25, 2024

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

Copy link
Member

@moomint8 moomint8 left a comment

Choose a reason for hiding this comment

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

XSS 처리가 고도화되었네요!
테스트코드도 이제 더미 값에 의존하지 않아도 되겠어요
꾸준히 고도화해주셔서 감사합니다 ㅎㅎ
고생 많으셨어요!

Copy link
Member

@Leegiyeon Leegiyeon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :) 테스트 코드 수정 확인했습니다 👍

@Leegiyeon Leegiyeon merged commit 5189ec8 into dev Jun 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Improve code or structure
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants