You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by zh...@apache.org on 2022/11/03 03:07:19 UTC

[shardingsphere] branch master updated: [Issue #20380]-Improve coverage of AtomikosTransactionManagerProvider (#21334)

This is an automated email from the ASF dual-hosted git repository.

zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 2ded81ddda7 [Issue #20380]-Improve coverage of AtomikosTransactionManagerProvider (#21334)
2ded81ddda7 is described below

commit 2ded81ddda792cd89bc97156aa81a48f336c3d25
Author: Abhinav Koppula <ab...@gmail.com>
AuthorDate: Thu Nov 3 08:37:11 2022 +0530

    [Issue #20380]-Improve coverage of AtomikosTransactionManagerProvider (#21334)
---
 .../AtomikosTransactionManagerProviderTest.java     | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/kernel/transaction/type/xa/provider/atomikos/src/test/java/org/apache/shardingsphere/transaction/xa/atomikos/manager/AtomikosTransactionManagerProviderTest.java b/kernel/transaction/type/xa/provider/atomikos/src/test/java/org/apache/shardingsphere/transaction/xa/atomikos/manager/AtomikosTransactionManagerProviderTest.java
index c79a4aa4c3d..75cb2b7cf13 100644
--- a/kernel/transaction/type/xa/provider/atomikos/src/test/java/org/apache/shardingsphere/transaction/xa/atomikos/manager/AtomikosTransactionManagerProviderTest.java
+++ b/kernel/transaction/type/xa/provider/atomikos/src/test/java/org/apache/shardingsphere/transaction/xa/atomikos/manager/AtomikosTransactionManagerProviderTest.java
@@ -34,6 +34,9 @@ import javax.transaction.Transaction;
 
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.MatcherAssert.assertThat;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
@@ -90,4 +93,22 @@ public final class AtomikosTransactionManagerProviderTest {
         transactionManagerProvider.close();
         verify(userTransactionService).shutdown(true);
     }
+    
+    @Test
+    public void assertInit() throws Exception {
+        transactionManagerProvider.init();
+        assertNull(transactionManagerProvider.getTransactionManager().getTransaction());
+        assertFalse(transactionManagerProvider.getTransactionManager().getForceShutdown());
+        assertTrue(transactionManagerProvider.getTransactionManager().getStartupTransactionService());
+    }
+    
+    @Test
+    public void assertGetType() {
+        assertThat(transactionManagerProvider.getType(), is("Atomikos"));
+    }
+    
+    @Test
+    public void assertIsDefault() {
+        assertTrue(transactionManagerProvider.isDefault());
+    }
 }