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

[feat] PositionModalTest 수정 #237

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

[feat] PositionModalTest 수정 #237

wants to merge 3 commits into from

Conversation

BellYun
Copy link
Contributor

@BellYun BellYun commented Jan 1, 2025

변경 사항

수정된 PositionModal에 따라 PositionModalTest 수정

테스트 결과 (테스트를 했으면)

ex) 베이스 브랜치에 포함되기 위한 코드는 모두 정상적으로 동작해야 합니다. 결과물에 대한 스크린샷, GIF, 혹은 라이브 데모가 가능하도록 샘플API를 첨부할 수도 있습니다.

+α Checklist

  • 자바 코드 컨벤션을 지키면서 프로그래밍했는가?
  • 한 메서드에 오직 한 단계의 들여쓰기(indent)만 허용했는가?
  • else 예약어를 쓰지 않았는가?
  • 모든 원시값과 문자열을 포장했는가?
  • 콜렉션에 대해 일급 콜렉션을 적용했는가?
  • 3개 이상의 인스턴스 변수를 가진 클래스를 구현하지 않았는가? (가능하면 인스턴스 변수의 수를 줄이기 위해 노력한다.)
  • getter/setter 없이 구현했는가? (단, DTO는 허용한다.)
  • 메소드의 인자 수를 제한했는가? (4개 이상의 인자는 허용하지 않는다. 3개도 가능하면 줄이기 위해 노력해 본다.)
  • 코드 한 줄에 점(.)을 하나만 허용했는가?
  • 메소드가 한가지 일만 담당하도록 구현했는가?
  • 클래스를 작게 유지하기 위해 노력했는가?

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Code review by ChatGPT

@BellYun BellYun self-assigned this Jan 1, 2025
@BellYun BellYun added 📁 Test 테스트 관련 코드 Frontend labels Jan 1, 2025
Copy link
Collaborator

@jung2941 jung2941 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend 📁 Test 테스트 관련 코드
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants