YongGoose commented on code in PR #7345:
URL: https://github.com/apache/incubator-seata/pull/7345#discussion_r2103731407
##########
discovery/seata-discovery-core/src/test/java/org/apache/seata/discovery/registry/MultiRegistryFactoryTest.java:
##########
Review Comment:
Since it’s been changed to a `TreeSet`, it might be good to add a few more
test cases
1. When the same `RegistryType` is provided with different casing (e.g.,
`File`, `FILE`), does it recognize them as the same?
- This could be verified by checking that no log is printed.
2. When more than one RegistryType is provided, does **the log output as
expected**?
3. When a value not defined in RegistryType is passed, is the appropriate
exception thrown?
--
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: notifications-unsubscr...@seata.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org
For additional commands, e-mail: notifications-h...@seata.apache.org
