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

사용하지 않는 파일 수정 및 삭제 #288

Closed
takoyakimchi opened this issue Aug 2, 2024 · 3 comments · Fixed by #363
Closed

사용하지 않는 파일 수정 및 삭제 #288

takoyakimchi opened this issue Aug 2, 2024 · 3 comments · Fixed by #363
Assignees
Labels
Milestone

Comments

@takoyakimchi
Copy link
Contributor

takoyakimchi commented Aug 2, 2024

요구 사항

  • image/ImageController.java는 업로드 테스트용 코드이므로, dev, local 환경에서만 작동하도록 한다.
  • test/TestController.java는 사용하지 않는 코드이므로 삭제한다.
@ehtjsv2
Copy link
Contributor

ehtjsv2 commented Aug 3, 2024

image/ImageController.java는 예제 코드가 아니라 이미지 put을 간편하게 하기 위한 코드입니다.

만약 저 컨트롤러가 없다면 image관련된 기능 관련된 테스트를 다른 도메인을 걸쳐서 해야해서 불편할 것 같습니다.

예를들어 삭제 했다면, jpeg도 들어가게 기능을 변경->(api테스트)->강아지등록api로 테스트->불편(붉은색)
삭제 안했다면 jpegd도 들어가게 기능 변경 -> image컨트롤러로 테스트-> 편안(파란색)

@takoyakimchi
Copy link
Contributor Author

오 굿입니다 👍

@takoyakimchi takoyakimchi changed the title 사용하지 않는 파일을 삭제한다. 사용하지 않는 파일을 수정 및 삭제한다. Aug 4, 2024
@takoyakimchi takoyakimchi changed the title 사용하지 않는 파일을 수정 및 삭제한다. 사용하지 않는 파일 수정 및 삭제 Aug 4, 2024
@jimi567
Copy link
Member

jimi567 commented Aug 11, 2024

이 이슈에서 패키지도 수정하면 좋을 것 같아요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants