zjncs opened a new pull request, #5826: URL: https://github.com/apache/rocketmq-dashboard/pull/5826
### What `MetricCollectionScope` identifies one successful native metric collection scope. It had no test. This adds 7 tests. ### What is pinned - validation: a **null domain**, a **missing or blank instanceId**, an **empty key set** and a **blank metric key** are all rejected; - **the key set is defensively copied** — mutating the source set after construction does not widen the scope; - **`contains` matches only when domain, instance and key all agree**, and rejects a null sample. ### Verification - New tests on unmodified code: **7/7 green**, checkstyle clean. - Discriminating mutation: the domain check dropped from `contains` (red). - Restored: 7/7 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]
