[GitHub] [dubbo] pinxiong commented on a change in pull request #9138: [Feature] Support the global registry center in unit tests

[GitBox <gi...@apache.org>]

pinxiong commented on a change in pull request #9138:
URL: https://github.com/apache/dubbo/pull/9138#discussion_r737207901



##########
File path: dubbo-test/dubbo-test-check/src/main/resources/META-INF/services/org.junit.platform.launcher.TestExecutionListener
##########
@@ -1 +1,5 @@
+# DubboRegistryCenterStarted should be the first one because of the startup of zookeeper
+org.apache.dubbo.test.check.RegistryCenterStarted

Review comment:
       All of implemenations of `TestExecutionListener` will be loaded, which are in `resources/META-INF/services/org.junit.platform.launcher.TestExecutionListener`.  The order of loading implemenations is as listed in the configuration file.
   
   + The first implemenation is `RegistryCenterStarted`, which is to handle the startup of zookeeper
   + The second implemenation is `DubboTestChecker`, which is to destroy resources after ut finished
   + The last implemenation is `RegistryCenterFinished `, which is to handle the shutdown of zookeeper
   
   The purpose of splitting into two implemenations is to make sure that `zookeeper` is started before all operations and destroyed after all operations.




-- 
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-unsubscribe@dubbo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org