Skip to content

test/api-w0-contracts - #101

Merged
kimwuju merged 1 commit into
mainfrom
test/api-w0-contracts
Oct 15, 2026
Merged

kimwuju merged 1 commit into
mainfrom
test/api-w0-contracts

Conversation

@danjee2009

Copy link
Copy Markdown
Member

FAILED 피드백 재채점과 영상 분석 비활성 응답의 상태별 동작을 회귀 테스트로 검증합니다. W0 명세의 확정된 API 기능은 기존 코드에 구현되어 있어, 계획서와 테스트를 보강했습니다.

관련 이슈

별도 연결 이슈 없음. 첨부된 W0 구현 명세를 기준으로 검증했습니다.

Task

  • 피드백 요청 6개, 조회 4개, 영상 분석 비활성 상태 1개 등 회귀 테스트 11개 추가
  • FAILED·시간 초과 PENDING의 재접수, PENDING·SUCCEEDED의 409, 조회 시 만료 처리 및 소유자 검증
  • 영상 분석 비활성 시 WAITING·결과 없음·재시도 불가 검증
  • API 요구사항 추출, 구현 계획, 보류 항목의 착수 조건 및 검증 결과 문서화

검증

  • 관련 테스트 149개 통과
  • JDK 21 및 임시 PostgreSQL 16 환경에서 ./gradlew test --offline --no-daemon: 전체 736개 통과, 실패·오류·건너뜀 0개
  • git diff --check 통과

ETC

API의 현재 피드백 대기 제한은 설정 파일 기준 10분이며 Web 폴링 마감은 5분입니다. 이 차이를 계획서에 기록했습니다. 운영 코드와 기존 대기 제한 정책의 변경은 없습니다.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-15T11:49:45.480002Z 0a9eb77 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@danjee2009 danjee2009 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

재채점의 상태별 접수·조회 동작과 영상 분석 비활성 응답에 추가된 테스트 및 W0 구현 계획 문서를 검토했습니다. 확인된 실질적인 문제는 없습니다. GitHub CI 테스트가 통과했습니다. 로컬 테스트는 별도로 실행하지 않았습니다.\n\n

@kimwuju
kimwuju merged commit 8623c15 into main Oct 15, 2026
1 check passed
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.

2 participants