[BUG] Completed and canceled progress states are not treated as terminal
メンテナーはふだん 6 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
progress_type_request から始めて、TestStatus.progress_step() と保存されている TestStatus の値がどのように比較されているかを確認します。issue で言及されているリグレッションテストを実行し、完了済みとキャンセル済みの両方の後の更新が拒否され、進捗の履歴が変更されないことを確認できたら、作業完了とします。
索引モデルが issue の本文から書いたものです。
説明
Description
progress_type_request does not correctly reject progress updates after a test has reached completed or canceled.
TestStatus.progress_step() returns an integer progress index, but the returned integer was compared directly against TestStatus.completed and TestStatus.canceled enum symbols.
As a result, terminal states could be incorrectly treated as active progress states.
Expected behavior
If the latest progress status is completed or canceled, a subsequent progress update should be rejected and the progress trail should remain unchanged.
Actual behavior
The terminal-state guard was bypassed, allowing additional progress entries after completed or canceled.
Reproduction
Regression tests demonstrate that:
- completed -> completed was accepted
- canceled -> completed was accepted
Fix
Compare the stored TestStatus enum directly against the terminal enum values, while retaining progress_step() for numeric progress ordering.
Tests
Regression tests cover both completed and canceled terminal states.
- 主要言語
- Python
- スター
- 30
- フォーク
- 87
- 平均マージ
- 5日 10時間
- マージ済み PR(30日)
- 21
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CCExtractor/sample-platform のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 84/100
CCExtractor/sample-platform#1192 · コメント 1 件 ·
メンテナーはふだん 6 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
CCExtractor/sample-platform#1082 ·
メンテナーはふだん 6 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
CCExtractor/sample-platform#1079 ·
メンテナーはふだん 6 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
CCExtractor/sample-platform#1062 · コメント 2 件 ·
メンテナーはふだん 6 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 42/100
CCExtractor/sample-platform#1057 ·
メンテナーはふだん 6 日以内に返信
CCExtractor/sample-platform の issue をすべて見る
似ている issue
-
adr
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
kristofdegrave/homeassistant-smart-charging#1607 ·
メンテナーはふだん 1 日以内に返信
-
namespace operations
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
EclipseFdn/open-vsx.org#13665 ·
メンテナーはふだん 1 日以内に返信
-
doc good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
collective/icalendar#1865 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
canonical/opentelemetry-collector-operator#409 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
mozilla/addons-release-tests#1243 ·
メンテナーはふだん 1 日以内に返信