fix unit test action (sofastack#228)

.github/workflows/integration-test.yml

@@ -20,7 +20,13 @@ jobs:
         uses: actions/setup-java@v1
         with:
           java-version: 1.8
-      - name: ut
+      - name: Integration Test
         run: mvn compile -B
-          && sh ./tools/check_format.sh
-          && mvn clean test -DisSkipUnitTest=true "-Dtest.logging.level=ERROR"
+          && mvn clean test -DisSkipUnitTest=true "-Dtest.logging.level=ERROR"
+      - name: Publish Test Report
+        if: ${{ always() }}
+        uses: ScaCap/action-surefire-report@v1
+        with:
+          fail_on_test_failures: true
+          skip_publishing: true
+

.github/workflows/unit-test.yml

@@ -20,10 +20,15 @@ jobs:
         uses: actions/setup-java@v1
         with:
           java-version: 1.8
-      - name: ut
+      - name: Unit Testt
         run: mvn compile -B
-          && sh ./tools/check_format.sh
-          && mvn clean test -DisSkipIntegrationTest=true "-Dtest.logging.level=ERROR"
+          && mvn clean test -DisSkipIntegrationTest=true "-Dtest.logging.level=ERROR" --fail-at-end --batch-mode
+      - name: Publish Test Report
+        if: ${{ always() }}
+        uses: ScaCap/action-surefire-report@v1
+        with:
+          fail_on_test_failures: true
+          skip_publishing: true
       - name: Codecov
         uses: codecov/codecov-action@v1
         with:

client/impl/src/main/java/com/alipay/sofa/registry/client/provider/DirectServerManager.java

@@ -36,10 +36,12 @@ public class DirectServerManager implements ServerManager {
 
   public DirectServerManager(RegistryClientConfig config) {
     this.serverNodes = new ArrayList<ServerNode>();
+    int port = config.getRegistryEndpointPort();
+    if (port == 0) {
+      port = 9600;
+    }
     this.serverNodes.add(
-        ServerNodeParser.parse(
-            String.format(
-                "%s:%s", config.getRegistryEndpoint(), config.getRegistryEndpointPort())));
+        ServerNodeParser.parse(String.format("%s:%s", config.getRegistryEndpoint(), port)));
   }
 
   @Override

client/impl/src/test/java/com/alipay/sofa/registry/client/provider/DirectServerManagerTest.java

@@ -47,7 +47,6 @@ public void initServerList() {
 
     // when
     when(config.getRegistryEndpoint()).thenReturn("127.0.0.1");
-    when(config.getRegistryEndpointPort()).thenReturn(9600);
 
     // then
     ServerManager serverManager = new DirectServerManager(config);

server/remoting/bolt/src/test/java/com/alipay/sofa/registry/remoting/bolt/BoltServerTest.java

@@ -32,8 +32,9 @@
  * @version $Id: BoltServerTest.java, v 0.1 2018-05-14 19:34 shangyu.wh Exp $
  */
 public class BoltServerTest {
-  private static URL url = new URL("0.0.0.0", 12345);
-  private static BoltServer server = new BoltServer(url, Collections.emptyList());
+  private static URL url = new URL("127.0.0.1", 12345);
+  private static URL listenURL = new URL("0.0.0.0", 12345);
+  private static BoltServer server = new BoltServer(listenURL, Collections.emptyList());
 
   @BeforeClass
   public static void before() {