zjncs opened a new pull request, #5856: URL: https://github.com/apache/rocketmq-dashboard/pull/5856
### What `UpdateAclUserDTO` carries an ACL user update. It had no test. This adds 4 tests. ### What is pinned - the **id is required**, and a numeric id **round-trips as a `Long`** (trimmed; non-numeric or blank maps to `null`); - **`toAclUserVO()` carries every field**; - the **nullable `permRead`/`permWrite` flags pass through unchanged**, while an absent `admin` flag reads as `false` in the view — preserving the stored value for a partial update is the caller's job before mapping. ### Verification - New tests on unmodified code: **4/4 green**, checkstyle clean. - Discriminating mutation: the admin null-handling replaced with a constant `true` (red). - Restored: 4/4 green. Test-only change; no product code touched. -- 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]
