test: port test_video_handlers.py coverage (handler dispatch, transcripts)
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 72/100
調査の方向性
video_handlers.py から始め、openedx-platform PR #38750 を通じて削除された lms/djangoapps/courseware/tests/test_video_handlers.py を調べ、元のケースを確認します。handler-dispatch、トランスクリプト、位置追跡、メタデータ保存のカバレッジを xblocks_contrib/tests/ に移植し、その後テストスイートを実行して、xmodule からインポートせずにパスすることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Context
As part of the Slash-n-Burn Built-In XBlocks epic (openedx-platform#37819), the file `lms/djangoapps/courseware/tests/test_video_handlers.py` was deleted from openedx-platform in PR #38750.
This test file covered `VideoBlock` handler logic that now lives in xblocks-contrib. The coverage was not ported before deletion.
What needs to be ported
`test_video_handlers.py` covered:
- Handler dispatch — `handle_ajax`, routing to sub-handlers
- Transcript download/upload/delete — `transcript`, `save_transcript`, `delete_transcript` handlers
- Transcript language/content negotiation — SRT vs SJSON format selection, language fallback
- Video position tracking — `save_user_state` handler
- Metadata save — `metadata_save` handler, field validation
The `video_handlers.py` module in xblocks-contrib is entirely untested at the handler level.
Acceptance Criteria
- Port equivalent tests into `xblocks_contrib/tests/` (or an appropriate subpackage)
- Tests should pass against the xblocks-contrib VideoBlock without importing from xmodule
- CI must remain green
Related
- openedx-platform PR: #38750
- Epic: openedx-platform#37819
- 主要言語
- Python
- スター
- 0
- フォーク
- 16
- 平均マージ
- 3時間 45分
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
openedx/xblocks-core のほかの issue
-
難易度 1/5 1〜3時間 初心者へのやさしさ 84/100
openedx/xblocks-core#305 ·
-
ChoiceTextResponse: a submission that omits a required numtolerance_input key is graded correctオープン
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
openedx/xblocks-core#306 ·
-
WCAG - Question Content Not Programmatically Associated With Answer Choices再び着手できるかも @jesusbalderramawgu が 51 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
openedx/xblocks-core#300 · 担当者 1 名 ·
-
flaky-test
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
openedx/xblocks-core#290 · コメント 2 件 ·
-
[DEPR]: Remove Video Bumper (pre-roll video) feature再び着手できるかも @feanil が 82 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンdepr
openedx/xblocks-core#284 · リアクション 1 件 · 担当者 2 名 ·
openedx/xblocks-core の issue をすべて見る
似ている issue
-
URLValidator ignores an explicit empty scheme allowlist対応中かも @IlyaKhalafi が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
tortoise/tortoise-orm#2393 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
frappe/frappe_docker#2046 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
complete_box_iou_loss / complete_box_iou: NaN gradients for zero-height boxes, NaN for point boxesオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
AXI SMU SPEC
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
tenstorrent/tt-oca-harness#3556 ·
メンテナーはふだん 1 日以内に返信