refactor: e2e 테스트 제거 #278
Merged
refactor: e2e 테스트 제거 #278
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Walkthrough
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (12)
💤 Files with no reviewable changes (11)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/test/java/com/example/solidconnection/e2e/DynamicFixture.java (1)
8-8: TODO 주석의 추가와 클래스 단순화에 대한 피드백
클래스 용도 변경 사항
- 기존 다양한 테스트 픽스처 생성 메서드가 제거되어 단일 메서드만 남게 되었습니다.
- 향후 추가 테스트 픽스처 개선 작업 후 이 클래스를 완전히 제거할 계획이 명확히 표현되었습니다.
코드 정리 제안
- 클래스가 단일 메서드만 포함하고 있고, 향후 삭제 예정이라면 이 메서드를 필요한 테스트 클래스로 직접 이동하는 것도 고려해볼 수 있을 것 같습니다.
- 이렇게 하면 현재의 중간 단계를 거치지 않고 코드를 더 깔끔하게 유지할 수 있습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (12)
build.gradle(0 hunks)src/test/java/com/example/solidconnection/e2e/ApplicantsQueryTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/BaseEndToEndTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/DynamicFixture.java(1 hunks)src/test/java/com/example/solidconnection/e2e/MyPageTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/SignInTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/SignUpTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/UniversityDataSetUpEndToEndTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/UniversityDetailTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/UniversityLikeTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/UniversityRecommendTest.java(0 hunks)src/test/java/com/example/solidconnection/e2e/UniversitySearchTest.java(0 hunks)
💤 Files with no reviewable changes (11)
- build.gradle
- src/test/java/com/example/solidconnection/e2e/ApplicantsQueryTest.java
- src/test/java/com/example/solidconnection/e2e/BaseEndToEndTest.java
- src/test/java/com/example/solidconnection/e2e/SignUpTest.java
- src/test/java/com/example/solidconnection/e2e/UniversitySearchTest.java
- src/test/java/com/example/solidconnection/e2e/SignInTest.java
- src/test/java/com/example/solidconnection/e2e/UniversityDataSetUpEndToEndTest.java
- src/test/java/com/example/solidconnection/e2e/MyPageTest.java
- src/test/java/com/example/solidconnection/e2e/UniversityRecommendTest.java
- src/test/java/com/example/solidconnection/e2e/UniversityDetailTest.java
- src/test/java/com/example/solidconnection/e2e/UniversityLikeTest.java
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
🔇 Additional comments (2)
src/test/java/com/example/solidconnection/e2e/DynamicFixture.java (2)
1-22: e2e 테스트 제거 맥락에서의 전체 클래스 검토
변경 범위 파악
- AI 요약에 따르면 여러 e2e 테스트 클래스들과 함께 이 클래스에서도 많은 메서드가 제거되었습니다.
- 이는 PR 목표인 "e2e 테스트 제거"와 일치합니다.
클래스 유지 이유 고려
- 대부분의 e2e 테스트 클래스가 완전히 제거된 반면, 이 클래스는 부분적으로 남아있습니다.
- 남아있는
createSiteUserByEmail메서드가 다른 종류의 테스트에서 여전히 필요한 것으로 보입니다.문서화 제안
- PR 설명에서 언급된 대로 Wiki에 e2e 테스트 제거 이유를 문서화하는 것은 좋은 접근입니다.
- 이 클래스의 역할 변화와 향후 계획도 함께 문서화하면 더욱 명확할 것 같습니다.
10-20:Details
❓ Verification inconclusive
유일하게 남은 createSiteUserByEmail 메서드에 대한 검토
메서드 보존 이유 확인
- 다른 여러 메서드가 제거된 반면 이 메서드만 남아있는 이유가 명확하지 않습니다.
- 현재 이 메서드가 어디서 사용되고 있는지 확인하시면 좋겠습니다.
하드코딩된 값들에 대한 생각
- 일관된 테스트 데이터를 위해 하드코딩된 값들을 사용하는 것은 이해되지만,
- 추후 테스트 픽스처 개선 작업 시에는 더 유연한 방식으로 개선하면 좋을 것 같습니다.
현재 이 메서드가 어디서 사용되고 있는지 확인하려면 다음 명령을 실행해보세요:
🏁 Script executed:
#!/bin/bash # 프로젝트 내에서 createSiteUserByEmail 메서드의 사용처 찾기 rg "createSiteUserByEmail" --type javaLength of output: 519
검증 완료: createSiteUserByEmail 사용처 확인 및 테스트 픽스처 개선 제안
메서드 사용처 확인
- 현재createSiteUserByEmail메서드는 다른 테스트 클래스인PostLikeCountConcurrencyTest.java에서 사용되고 있습니다.
- 이로 인해 메서드가 남아있는 이유는 테스트 환경의 일관성을 위한 픽스처로 의도된 것으로 보입니다.하드코딩된 값에 대한 개선 제안
- 테스트 데이터의 일관성을 위해 하드코딩된 값들을 사용하는 것은 당장의 목적으로는 문제가 없으나,
- 추후 테스트 픽스처 개선 작업 시 데이터를 보다 유연하게 관리할 수 있도록 개선하는 방안을 고려해 보시기 바랍니다.
…e-e2e-test # Conflicts: # src/test/java/com/example/solidconnection/e2e/DynamicFixture.java # src/test/java/com/example/solidconnection/e2e/MyPageTest.java # src/test/java/com/example/solidconnection/e2e/SignUpTest.java
Member
|
창 켜두었더니 소켓 연결되어 있는지 계속 실시간 갱신되는거 신기하네요 😮 |
wibaek
approved these changes
Apr 9, 2025
Collaborator
Author
|
@Gyuhyeok99 간단한 PR이라 빠르게 머지하겠습니다! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 이슈
작업 내용
link: #266
e2e 테스트를 제거합니다.
특이 사항
e2e 테스트를 도입했던 이유 & 지금은 삭제한 이유를 위키로 정리해보려 합니다!
(위키 쓸게 많네요 🏃♀️🏃♀️)
리뷰 요구사항 (선택)