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

강의 이수 변경사항 테스트코드 작성 #524

Merged
merged 4 commits into from
Jul 17, 2024

Conversation

ani2689
Copy link
Contributor

@ani2689 ani2689 commented Jul 17, 2024

💡 배경 및 개요

강의 이수 변경사항 테스트코드 작성

Resolves: #523

📃 작업내용

  • 강의 이수중인 학생 상세조회 테스트코드 작성
  • 강의 이수 여부 확인 테스트 코드에 save -> saveAll로 변경
  • 반환 값 잘못 들어가있던거 수정

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

@ani2689 ani2689 added the 1️⃣ Priority: High 우선순위 - 상 label Jul 17, 2024
@ani2689 ani2689 self-assigned this Jul 17, 2024
@ani2689 ani2689 linked an issue Jul 17, 2024 that may be closed by this pull request
@@ -587,7 +590,7 @@ class LectureServiceImplTest : BehaviorSpec({
}

//signUpLecture 테스트 코드
Given("Lecture id가 주어질 때") {
Given("Lecture id가 주어질 때 ") {
Copy link
Collaborator

Choose a reason for hiding this comment

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

공백 잘못 붙이신 것 같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ec570ba 삭제했습니다~~

@ani2689 ani2689 merged commit 20e260d into master Jul 17, 2024
1 check passed
@ani2689 ani2689 deleted the 523-feature/registered-lecture-test branch July 17, 2024 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1️⃣ Priority: High 우선순위 - 상
Projects
None yet
Development

Successfully merging this pull request may close these issues.

강의 이수 변경사항 테스트코드
3 participants