Fix parts integration tests
Metadata
Current evaluation
Merged a fix for parts integration tests failing due to YAML dictionary ordering differences. Tests now load and compare dictionaries directly instead of relying on serialized output order. Approved by two reviewers and passed CI.
Suggested action: —
No scores available.
Issue body
Insulate the parts integration tests against trivial differences in YAML
dump ordering of dictionaries.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix for parts integration tests failing due to YAML dictionary ordering differences. Tests now load and compare dictionaries directly instead of relying on serialized output order. Approved by two reviewers and passed CI. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for parts integration tests. The change replaces serialized YAML comparison with direct dictionary comparison, insulating tests from non-deterministic YAML dump ordering. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updated parts integration tests to compare parsed dictionaries instead of serialized YAML output, resolving failures caused by non-deterministic dictionary ordering. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3669 parts: raise error when duplicate keys are used | snapcraft | merged | Merged. Refactored YAML loading into a new yaml_utils module and forward-ported safe loading to raise errors for duplicate keys in parts configuration. Approved by reviewer and passed CI. |