bugfix: ensure visibility of rm and The methods in MockTest are executed in order

Author: maple

test-old-version/src/test/java/io/seata/MockTest.java

@@ -29,16 +29,14 @@
 import io.seata.rm.DefaultResourceManager;
 import org.apache.seata.mockserver.MockCoordinator;
 import org.apache.seata.mockserver.MockServer;
-import org.junit.jupiter.api.AfterAll;
-import org.junit.jupiter.api.Assertions;
-import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.*;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 /**
  * the type MockServerTest
  */
+@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
 public class MockTest {
 
     static String RESOURCE_ID = "mock-action-061";
@@ -57,39 +55,45 @@ public static void after() {
     }
 
     @Test
+    @Order(1)
     public void testCommit() throws Exception {
         String xid = doTestCommit(0);
         Assertions.assertEquals(1, Action1Impl.getCommitTimes(xid));
         Assertions.assertEquals(0, Action1Impl.getRollbackTimes(xid));
     }
 
     @Test
+    @Order(2)
     public void testCommitRetry() throws Exception {
         String xid = doTestCommit(2);
         Assertions.assertEquals(3, Action1Impl.getCommitTimes(xid));
         Assertions.assertEquals(0, Action1Impl.getRollbackTimes(xid));
     }
 
     @Test
+    @Order(3)
     public void testRollback() throws Exception {
         String xid = doTestRollback(0);
         Assertions.assertEquals(0, Action1Impl.getCommitTimes(xid));
         Assertions.assertEquals(1, Action1Impl.getRollbackTimes(xid));
     }
 
     @Test
+    @Order(4)
     public void testRollbackRetry() throws Exception {
         String xid = doTestRollback(2);
         Assertions.assertEquals(0, Action1Impl.getCommitTimes(xid));
         Assertions.assertEquals(3, Action1Impl.getRollbackTimes(xid));
     }
 
     @Test
+    @Order(5)
     public void testTm() throws Exception {
         TmClientTest.testTm();
     }
 
     @Test
+    @Order(6)
     public void testRm() throws Exception {
         RmClientTest.testRm("testRM01");
     }

test-old-version/src/test/java/io/seata/core/rpc/netty/RmClientTest.java

@@ -35,7 +35,7 @@
 public class RmClientTest {
 
     protected static final Logger LOGGER = LoggerFactory.getLogger(RmClientTest.class);
-    private static DefaultResourceManager rm = null;
+    private static volatile DefaultResourceManager rm = null;
 
     public static void testRm(String resourceId) throws TransactionException, NoSuchMethodException {
         String xid = "1111";