test(st): drop a5 from runtime_fatal_codes onboard coverage - #1206
Conversation
The onboard device-error test now runs on a2a3 only; a5 is taken offline. The sim test (a5sim/a2a3sim) is unchanged.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe ChangesPlatform Restriction for Hardware Fatal-Code Test
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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. Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the test suite in test_runtime_fatal_codes.py by removing the 'a5' platform from the @pytest.mark.platforms decorator for a specific test case, limiting its execution to the 'a2a3' platform. There are no review comments provided, and I have no additional feedback on this change.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
The onboard device-error test (
test_device_error_class_reaches_host_log) now runs ona2a3only;a5is taken offline.The sim test (
a5sim/a2a3sim) is unchanged.