Skip to content

0040 wait_notify のタイムアウトがデフォルト 5 秒に張り付き flake する問題を修正する - #364

Open
voluntas wants to merge 3 commits into
developfrom
feature/fix-wait-notify-timeout-too-short
Open

voluntas wants to merge 3 commits into
developfrom
feature/fix-wait-notify-timeout-too-short

Conversation

@voluntas

@voluntas voluntas commented Sep 1, 2026

Copy link
Copy Markdown
Member

概要

tests/test_signaling_notify.pywait_notify のタイムアウトがデフォルト 5 秒に張り付いて flake する問題を修正する。タイムアウトをテスト側で明示し、タイムアウト時に受信済み notify の内訳が分かる assert メッセージにする。

変更内容

  • tests/client.pySoraClient.wait_notifylabel パラメータを追加し、タイムアウト時に label・timeout・受信済みの event_type 一覧を含む AssertionError を raise するように変更
  • timeout の型を float に変更(デフォルトは従来どおり 5 秒)
  • tests/test_signaling_notify.py の全 4 箇所に明示的なタイムアウト(connection.created = 10 秒、connection.destroyed = 15 秒)と日本語の label を指定
  • tests/test_wait_notify.py を新規追加し、タイムアウト時のエラーメッセージ(label・timeout・受信済み event_type 一覧・空一覧)を検証

完了条件

  • tests/test_signaling_notify.pywait_notify 呼び出しすべてに明示的なタイムアウトと用途が分かる label が付くこと
  • wait_notify がタイムアウトしたとき、label と、それまでに受信した event_type のリストが assert メッセージに含まれること
  • 既存のテストがすべて pass すること(全スイートは本 PR の CI で確認)

関連 issue

  • issues/closed/0040-bug-fix-wait-notify-timeout-too-short.md

- タイムアウト時に label と受信済み event_type 一覧を AssertionError に含める
- `test_signaling_notify.py` の全呼び出しに明示タイムアウトと label を指定する
- タイムアウト動作の単体テストを追加する
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.

1 participant