yyqdbngt opened a new pull request, #3804: URL: https://github.com/apache/rocketmq-dashboard/pull/3804
### Motivation The existing `SystemTopicFilterTest` covers the canonical system topics and broker-name matching in one test, but the null/empty broker set, the no-broker convenience overload and the bare retry/dlq markers were untested. ### Changes - `shouldTreatMissingOrEmptyBrokerNamesAsNoMatch`: a broker-named topic is not system when the broker set is null or empty, while real system topics stay recognised. - `convenienceOverloadShouldClassifyByNameOnly`: the single-argument form classifies system topics and retry/dlq names without any broker context. - `shouldTreatBareRetryAndDlqMarkersAsSystemTopics`: the bare `%RETRY%`/`%DLQ%` markers and their suffixed forms count as system topics. ### Verification ``` mvn -B test -Dtest=SystemTopicFilterTest [INFO] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 [INFO] BUILD SUCCESS ``` -- 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]
