[xharness] Bump assembly-preparer test timeout to 30 minutes. - #26532
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
The change is low-risk and directly addresses the reported harness timeout flake, with only an optional documentation nit noted.
Pull request overview
This PR increases the xharness timeout for the assembly-preparer unit test project to reduce CI flakiness where the harness hits its 10-minute deadline even after NUnit finishes successfully (per issue #26528).
Changes:
- Bump
Assembly processing teststimeout from 10 minutes to 30 minutes in xharness project registration.
File summaries
| File | Description |
|---|---|
| tests/xharness/Harness.cs | Increases the harness timeout for assembly-preparer tests to reduce intermittent CI timeouts. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
🚀 [CI Build #0790065] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 264 tests passed 🎉 Tests counts✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Sonoma (14): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
Increase the xharness timeout for assembly-preparer tests from 10 to 30 minutes to accommodate longer-running test execution.
Fixes #26528.
🤖 Pull request created by Copilot