ZinnaChoi / Study-Management

온라인 스터디 관리 시스템: 부재 일정 캘린더, 토론 게시판 및 알림 기능 제공
8 stars 0 forks source link

Task add check wakeup popup #89

Closed dayeon-dayeon closed 6 months ago

dayeon-dayeon commented 6 months ago

안녕하세요! 기상 체크 popup입니다.

image

위와 같은 팝업에 확인을 클릭한 시간으로 기상 성공 여부를 저장합니다.

image

이미 그 날의 기상 기록이 있는 경우는 다시 기상 체크를 해도 저장되지 않습니다.

또한 23시 55분에 그 날의 기상 체크가 되어있지 않은 멤버는 실패로 저장되도록 백엔드에 스케줄링 기능 추가했습니다.

MeMyself-And-I commented 6 months ago

StatControllerTest.java 테스트 코드 중 getStatListSuccessPage1Size2() 테스트 함수에서 기상 로그를 등록 후 테스트를 실행하면 결과값이 2개로 나와 assertTrue(contentCount == 1);에서 에러가 발생합니다. 제 생각이 맞다면 sortable size를 2로 설정하여 결과값이 두 개 이상일 경우 2개로 나오게 되는 것 같은데 DB에 다른 통계 데이터가 존재할 경우를 고려해야 한다면 assertTrue(contentCount >= 1); 등과 같이 변경을 해야할 것 같습니다.

바로 위에 있는 getStatListSuccessPage0Size1()함수의 경우 sortable size가 1로 설정되어 있어 데이터가 여러개 있어도 1개만 리턴하게 되어 테스트가 성공하는 것 같습니다!

MeMyself-And-I commented 6 months ago

수고 많으셨습니다! 기상 체크 버튼 클릭 시 팝업을 통해 클릭한 당시의 현재 시간을 사용자에게 보여주고, 수정이 불가능하게 readonly 설정으로 두는 아이디어 정말 좋은 것 같습니다! 코멘트 확인 부탁 드립니다!

dayeon-dayeon commented 6 months ago

StatControllerTest.java 는 통계 화면 PR올릴 때 함께 백엔드 코드도 수정한 부분이 있어서(파라미터에서 page 없애고 시작일자와 종료일자 추가) 그 PR 에서 반영하면 될 것 같습니당. 일단 assertTrue(contentCount >= 1)로 수정해두었습니다!

MeMyself-And-I commented 6 months ago

코멘트 반영해주신 부분 확인하고, UI 테스트를 진행하는데 오류가 발생합니다. 테스트 방법은 아래와 같습니다.

  1. 회원가입

  2. 기상 체크

    image
  3. 기상 체크 실패 alert 발생

    image
  4. daily_log 테이블에는 저장됨

MeMyself-And-I commented 6 months ago

정상동작 확인했습니다!