RockteMQ-AI commented on issue #4788: URL: https://github.com/apache/rocketmq-dashboard/issues/4788#issuecomment-5765170874
## Evaluation **Classification:** Bug — test fixtures out of sync with backend contract **Severity:** LOW-MEDIUM **Status:** ✅ Confirmed ### Verification Code references verified against `rocketmq-studio @ 4c697f07`: 1. **`RESTART_BROKER`** in `web/src/mock/audit.ts` (lines 83, 157, 230) — `grep -rn RESTART_BROKER server/src/main/java` returns zero hits. The operation does not exist in `OperationAuditConstants` or any `auditService.record(...)` call. 2. **`UPDATE_ACL`** in mock (lines 71, 132, 206) — backend records as `UPDATE_ACL_RULE` (`AclService.java:158`) and `UPDATE_ACL_USER` (`AclService.java:258`), not `UPDATE_ACL`. Resource type mismatch: mock uses `TOPIC`, backend uses `ACL_RULE`. 3. **`RUNNING`** in `web/src/api/clusterContract.test.ts` (lines 31, 37, 47, 53) — backend `ClusterStatus` serializes as `healthy/warning/error/offline`, `BrokerStatus` as `running/readonly/maintenance`. `RUNNING` is never serialized. ### Impact - Mock mode offers filter options for operations that cannot exist, misleading UI testing. - Contract test asserts values the API never produces, giving false confidence in serialization correctness. ### Associated PR PR #4787 (by @Frun1na) — test fix: pin mock and contract values to what the backend actually records. --- *Evaluated by github-manager bot* -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
