[owayo]

概要

orchestrator::lock_age_tests::final_verification_has_an_independent_time_budget が、ビルドとテストを並行して動かしている負荷の高いときに失敗することがありました (照会の回数が 2 にならない)。単独で流すと通ります。負荷によらずに通るよう、テストの時間の枠を広げます。

原因

このテストは、差し戻しの照会と最終の検証のそれぞれに 100 ms の時間の枠を渡し、最初の照会だけを 1 秒待たせて枠を使い切らせます。最終の検証は独立した 100 ms の枠で 2 回目の照会をすることを確かめますが、時間は std::time::Instant で測るので、負荷が高いと照会を始める前に 100 ms が過ぎ、照会しないまま枠を使い切っていました。

変更内容

• 枠を 1 秒にし、最初の照会は 60 秒待たせる (枠で打ち切られるので、テストの時間は枠の分だけ延びる)。確かめる内容 (差し戻しの枠を使い切っても最終の検証は独立した枠で照会する) は変えていない。

製品のコードは変えていません。tokio の時間の停止 (start_paused) は、監査が std::time::Instant で時間を測るので使えません。

検証

• cargo test --locked --lib final_verification_has_an_independent_time_budget: pass
• cargo fmt --all -- --check: pass
• cargo clippy --locked --all-targets -- -D warnings: pass
• cargo test --locked (Windows): pass

Closes #128

🤖 Generated with Claude Code

[coderabbitai]

Important

• Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 3115946c-8a7a-46c7-9f95-eef586fddae1

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn