[monotouch-test] Assert instead of plain exception.#26247
Conversation
Much easier to diagnose.
There was a problem hiding this comment.
Pull request overview
Updates a macOS-only SearchKit monotouch test to report failures using NUnit assertions instead of throwing a plain exception, improving diagnostics when index creation fails.
Changes:
- Replaced a manual
if (idx is null) throw new Exception ();check withAssert.That (idx, Is.Not.Null, ...)inTestCreate.
✅ 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 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
🚀 [CI Build #034423c] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 203 tests passed 🎉 Tests counts✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Monterey (12): All 5 tests passed. [attempt 2] Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
Much easier to diagnose.