Skip to content

test_workflow_guard: test_split_finishes_quickly_on_a_long_body が 0.1 秒の壁時計の上限で継続的統合の上で揺れる #922

Description

@takemi-ohama

何を見つけたか

plugins/ndf/skills/development-workflow/tests/test_workflow_guard.py::test_split_finishes_quickly_on_a_long_body が、継続的統合の上で時間の上限(0.1 秒)を超えて落ちた。

>       assert time.monotonic() - started < 0.1
E       assert (89.710371028 - 89.583896483) < 0.1
plugins/ndf/skills/development-workflow/tests/test_workflow_guard.py:535: AssertionError

実測は 0.126 秒。変更は issues/ の Markdown 3 本だけで、このテストの対象のコードには触れていない。失敗したジョブの再実行で結果を確かめる。

どこで見つけたか

PR #920 の継続的統合(https://github.com/devbasex/ai-plugins/actions/runs/35872253492pytest (1/2)

なぜこの変更の範囲外なのか

PR #920 は cross-review のラウンドを減らす設計(issue #542 / issue #786)で、文書だけを変える。development-workflow のテストは受け入れ条件にも直す対象にも含まれない。

直さないと何が起きるか

壁時計の時間を 0.1 秒で切る検査は、継続的統合の実行機の負荷で揺れる。関係の無い Pull Request の検査が落ち、cross-review の merge-fix がコードの失敗として中断する(PR #920 で発生)。

由来

PR #920

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions