テストの待ち時間を、確かめたいことに要る長さまで縮める

43 秒のうち 19 秒が待ち時間だった。1 本ごとは小さく見えても、本数を掛けると
走らせる気が失せる長さになる。

直したのは 4 か所。後片付けの見回り（HTTPServer.serve_forever の既定は 0.5 秒で、
テスト 1 本ごとに払っていた。15 本で 7.5 秒）、繋がらない相手を待つ既定（5 秒）、
1 行ずつ眠らせる読み手（2 秒）、締め切りより長く眠る子プロセス。

確かめたいのは「その扱いになること」であって、何秒かかるかではない。締め切りを
超えたことが分かる長さがあれば足りる。

43 秒 → 27 秒。テストは 1 本も減らしていない。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: claude

CLAUDE.md

@@ -2818,6 +2818,13 @@ GeoNames 全世界地名辞典 = `geonames`(いずれも 348 言語版・195 か
     「Ukraine (with Crimea)」や複数国をまとめた抽出)は `OVERRIDES` に人手で書く
     — Geofabrik の ISO コードには取り違え(トケラウに `VU` 等)があり、そのまま引くと
     「トケラウ = バヌアツ」のような誤表示になるため
+  - **テストで待つときは、確かめたいことに要る長さだけ待つ。** 1 本ごとの待ちは
+    小さく見えても、本数を掛けると走らせる気が失せる長さになる。実測で直した 4 か所:
+    後片付けの見回り(`HTTPServer.serve_forever` の既定は 0.5 秒で、**テスト 1 本ごと**に
+    払っていた)、繋がらない相手を待つ既定(5 秒)、1 行ずつ眠らせる読み手(2 秒)、
+    締め切りより長く眠る子プロセス。**確かめたいのは「その扱いになること」で、
+    何秒かかるかではない** —— 締め切りを超えたことが分かる長さがあれば足りる
+    (43 秒 → 27 秒。テストは 1 本も減らしていない)
   - `run_tests.sh` — テスト実行のラッパー(引数はそのまま pytest へ)。手元に依存が
     揃っていればそれで、無ければ **CI と同じ Python 3.12 のイメージを組み立てて** Docker で
     回す。ホストの python が 3.12 でない環境(依存に C 拡張があるので import から落ちる)で

tests/test_bridge.py

@@ -898,7 +898,8 @@ def test_a_timeout_carries_what_the_cli_said(self, bridge, monkeypatch):
         monkeypatch.setattr(server, "IMAGE_TIMEOUT", 0.5)
         monkeypatch.setattr(
             server, "image_command",
-            lambda out_dir, model="": ["sh", "-c", "echo 権限の確認を待っています; sleep 2"],
+            # 締め切り(0.5 秒)より長ければよい。長くするほどこの 1 本が待つ
+            lambda out_dir, model="": ["sh", "-c", "echo 権限の確認を待っています; sleep 1"],
         )
 
         with pytest.raises(fastapi.HTTPException) as got:

tests/test_extract.py

@@ -1065,8 +1065,11 @@ def test_a_slow_reader_does_not_run_out_the_clock(self, tmp_path):
 
         seen = 0
         for _row in db.stream(path, "SELECT n FROM t ORDER BY n", timeout=0.05):
-            # 1 行の手間は小さいが、行数を掛けると締め切りをはるかに超える
-            time.sleep(0.0001)
+            # **合計で締め切りを超えれば足りる。** 1 行ずつ眠らせると 2 秒かかるので、
+            # まとめて休む —— 確かめたいのは「読み手の手間を締め切りに数えない」ことで、
+            # どれだけ細かく休むかではない
+            if seen % 100 == 0:
+                time.sleep(0.002)
             seen += 1
 
         assert seen == self.ROWS

tests/test_remote_plugins.py

@@ -74,7 +74,11 @@ def plugin():
     _Handler.docs = DOCS
     _Handler.refuse = None
     server = HTTPServer(("127.0.0.1", 0), _Handler)
-    thread = threading.Thread(target=server.serve_forever, daemon=True)
+    # **見に来る間隔を詰める。** `shutdown()` は次の見回りまで戻らないので、
+    # 既定(0.5 秒)のままだと**テスト 1 本ごとに 0.5 秒**を後片付けに払う
+    thread = threading.Thread(
+        target=lambda: server.serve_forever(poll_interval=0.02), daemon=True,
+    )
     thread.start()
     try:
         yield f"http://127.0.0.1:{server.server_port}"
@@ -94,8 +98,12 @@ def test_empty_setting_means_no_plugins(self):
         assert remote.catalog("") == []
         assert remote.catalog("  ,  ") == []
 
-    def test_unreachable_plugin_is_skipped(self, caplog):
+    def test_unreachable_plugin_is_skipped(self, caplog, monkeypatch):
         """落ちていても本体は動く(警告のみ)。別コンテナなので一時的な不通は正常。"""
+        # **待つ時間は縮める。** 確かめたいのは「繋がらなければ飛ばす」ことで、
+        # 何秒待つかではない —— 既定のまま待つと、この 1 本で 5 秒かかる
+        monkeypatch.setattr(remote, "CATALOG_TIMEOUT", 0.2)
+
         assert remote.catalog("http://127.0.0.1:9") == []
         assert "catalog unavailable" in caplog.text