CI: unify L4 pod selection on scene-test levels - #1775
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThe pull request introduces normalized scene-test levels for chip, host, and pod tests. Pytest supports level inclusion and exclusion, dispatch handles inherited filters, pod tests use level metadata, and CI workflows and documentation use the new options. ChangesScene-level test filtering
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant PytestCLI
participant conftestpy
participant SceneTestDispatcher
PytestCLI->>conftestpy: provide scene-level selection
conftestpy->>conftestpy: normalize metadata and filter tests
conftestpy->>SceneTestDispatcher: remove inherited exclude-level option
SceneTestDispatcher->>conftestpy: dispatch eligible pod-level tests
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
- Add SceneTestLevel and scene_level for shared level metadata - Switch pod wrappers, CI, and docs to use level 4 selection - Add unit coverage for level filtering and pod log gating
5ee7759 to
b0990e9
Compare
b0990e9 to
dbdb63f
Compare
This removes @pytest.mark.pod as a selection mechanism and moves L4 pod wrappers onto the shared scene-test level axis with SceneTestLevel.POD and @scene_level(...).
Changes: