kookmin-sw / capstone-2023-05

capstone-2023-05 created by GitHub Classroom
https://capstone-2023-05.vercel.app
0 stars 0 forks source link

feat: finish battle #66

Closed Jaewook-Lee closed 1 year ago

Jaewook-Lee commented 1 year ago

이 PR은 #65 이슈를 구현했습니다.
WebSocket을 통해 토론의 최종 결과를 얻는 기능을 수행할 수 있습니다.

테스트 전

Support 테이블에 충분한 양의 투표 기록들이 있어야 합니다. 로컬 테스트에서 사용했던 테이블 내용은 아래의 csv 파일로 공유합니다.
Support_Table.csv(23.05.11에 파일이 수정됐습니다.)

그리고 테스트를 위해 최소 1명의 토론 참가자가 initJoin 과정을 무사히 수행해야 합니다.

테스트 방법

  1. 연결한 connection에서 finishBattle 액션을 요청합니다.
    스크린샷 2023-05-10 23-49-28
  2. 연결된 모든 참가자에게 websocket 메세지가 전송되는지 확인합니다.
    테스트가 성공적이라면 아래와 같은 메세지들을 받아야 합니다.
    {
    "action": "getFinalResult",
    "result": {
        "1": 1,
        "2": 3
    }
    }

    위에 작성한 예시 결과는 위에 공유한 파일을 바탕으로 나오는 결과입니다.

Binsk-dev commented 1 year ago

일단 docker-compose.yml 파일에 db 컨테이너의 user 항목이 2번 들어갑니다. 제거해주세요. 추가로 CSV 파일에 order column이 들어가는데 이건 왜 들어간 거죠? 그냥 import해서 쓰려고 하니깐 테이블 스펙이랑 맞지 않아서 일단 제거해서 불러오긴 했습니다.

Binsk-dev commented 1 year ago

일단 docker-compose.yml 파일에 db 컨테이너의 user 항목이 2번 들어갑니다. 제거해주세요.

Binsk-dev commented 1 year ago

일단 docker-compose.yml 파일에 db 컨테이너의 user 항목이 2번 들어갑니다. 제거해주세요.

Jaewook-Lee commented 1 year ago

@Binsk-dev

CSV 파일에 order column이 들어가는데 이건 왜 들어간 거죠?

아 이건.... Opinion 테이블에 썼던걸 재활용하다가 까먹고 못 지웠나봅니다... 하하....

일단 docker-compose.yml 파일에 db 컨테이너의 user 항목이 2번 들어갑니다. 제거해주세요.

여러번 강조해준만큼 바로 수정할게요!

동작합니다!

고마워요!

seungholee-dev commented 1 year ago

동작 확인했습니다!! 감사해요!!