test(gui): update client inventory assertions for mcode (#1744 follow-up) - #1749
Conversation
… mcode MiniMax Code (#1664) legitimately added a ninth integration client; the inventory tests and locale parity allowlist now reflect it (product proper noun kept in English, matching Kimi Code precedent).
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
This pull request was already a draft. Its draft status will be preserved after every issue above is resolved. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThe GUI test suite now treats MiniMax Code ( ChangesMiniMax Code integration test coverage
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
Summary\n\nFixes the 4 red GUI gate tests on dev after #1744: MiniMax Code (mcode) legitimately became the ninth integration client, and the hardcoded client-inventory assertions (integrations-api, client-config-panel, integrations-overview-rows) plus the zh-TW locale-parity allowlist needed to catch up. Test-only change; no product code touched. zh-TW keeps the product proper noun in English, matching the Kimi Code precedent.\n\n## Verification\n\n- cd gui && bun test --isolate tests: 854 pass / 0 fail (146 files) on this branch — previously 4 fail on dev.\n\n## Checklist\n\n- [x] Targets dev\n- [x] Test-only
Summary by CodeRabbit