Skip to content

✅ 공지사항 테스트 추가 #416

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

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

✅ 공지사항 테스트 추가 #416

wants to merge 2 commits into from

Conversation

yeolyi
Copy link
Collaborator

@yeolyi yeolyi commented Aug 5, 2025

close #

작업 내용

공지사항 테스트를 작성합니다. 엄청 많아보이는데 커서로 테스트 틀을 짜서 그렇고, test.skip으로 스킵처리하지 않은 테스트들은 직접 구현한 것이니 그거랑 page fixture 위주로 확인해주시면 됩니다.

이상하게 공지사항 제목을 수정할 수 있다 테스트에서 clickNoticeByTitle이 안되는데... 내일 다시 확인해보겠습니다.

테스트 방법

bash ./scripts/start-backend.sh
pnpm dev:devlocal
pnpm test --ui

이후 playwright 테스트 정상동작 확인

Copy link

github-actions bot commented Aug 5, 2025

@goranikin님 2025. 8. 8.까지 리뷰 부탁드립니다 🎉

@github-actions github-actions bot requested a review from goranikin August 5, 2025 16:24
@yeolyi yeolyi changed the title Feat/notice test ✅ 공지사항 테스트 추가 Aug 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant