From ee7fa42171c628c71bf072131b4b67b79316129c Mon Sep 17 00:00:00 2001
From: Erik Brakkee <erik@brakkee.org>
Date: Thu, 22 Jul 2010 17:29:56 +0000
Subject: [PATCH]

---
 .../AutoCreateUserTransaction.java            | 76 +++++++++++++++++++
 .../SimpleTransactionManager.java             | 10 ++-
 .../transactions/SimpleUserTransaction.java   |  2 +-
 3 files changed, 84 insertions(+), 4 deletions(-)
 create mode 100644 test/enterprise/src/main/java/org/wamblee/test/transactions/AutoCreateUserTransaction.java

diff --git a/test/enterprise/src/main/java/org/wamblee/test/transactions/AutoCreateUserTransaction.java b/test/enterprise/src/main/java/org/wamblee/test/transactions/AutoCreateUserTransaction.java
new file mode 100644
index 00000000..7b5124be
--- /dev/null
+++ b/test/enterprise/src/main/java/org/wamblee/test/transactions/AutoCreateUserTransaction.java
@@ -0,0 +1,76 @@
+/*
+ * Copyright 2005-2010 the original author or authors.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.wamblee.test.transactions;
+
+import javax.transaction.HeuristicMixedException;
+import javax.transaction.HeuristicRollbackException;
+import javax.transaction.NotSupportedException;
+import javax.transaction.RollbackException;
+import javax.transaction.SystemException;
+import javax.transaction.UserTransaction;
+
+/**
+ * User transaction implemenetation that automatically creates thread specific transaction if needed.
+ * 
+ * @author Erik Brakkee
+ *
+ */
+public class AutoCreateUserTransaction implements UserTransaction {
+    
+    private SimpleTransactionManager manager; 
+    
+    public AutoCreateUserTransaction(SimpleTransactionManager aManager) {
+        manager = aManager; 
+    }
+    
+    private UserTransaction getTransaction() { 
+        return manager.getOrCreateThreadSpecificTransaction(); 
+    }
+    
+    @Override
+    public void begin() throws NotSupportedException, SystemException {
+        getTransaction().begin();
+    }
+    
+    @Override
+    public void commit() throws RollbackException, HeuristicMixedException,
+        HeuristicRollbackException, SecurityException, IllegalStateException,
+        SystemException {
+        getTransaction().commit();
+    }
+    
+    @Override
+    public int getStatus() throws SystemException {
+        return getTransaction().getStatus();
+    }
+    
+    @Override
+    public void rollback() throws IllegalStateException, SecurityException,
+        SystemException {
+        getTransaction().rollback();
+    }
+    
+    @Override
+    public void setRollbackOnly() throws IllegalStateException, SystemException {
+        getTransaction().setRollbackOnly();
+    }
+    
+    @Override
+    public void setTransactionTimeout(int aSeconds) throws SystemException {
+        getTransaction().setTransactionTimeout(aSeconds);   
+    }
+
+}
diff --git a/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleTransactionManager.java b/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleTransactionManager.java
index 81a5bfad..3ba4e814 100644
--- a/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleTransactionManager.java
+++ b/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleTransactionManager.java
@@ -64,7 +64,7 @@ import org.wamblee.test.persistence.JpaBuilder;
 public class SimpleTransactionManager {
 
     private UserTransactionFactory factory;
-    private ThreadSpecificProxyFactory<UserTransaction> transaction; 
+    private ThreadLocal<UserTransaction> transaction; 
     private UserTransactionCallback callback;
     private List<TransactionResource> resources;
 
@@ -76,7 +76,7 @@ public class SimpleTransactionManager {
      */
     public SimpleTransactionManager(UserTransactionFactory aFactory) {
         factory = aFactory;
-        transaction = new ThreadSpecificProxyFactory<UserTransaction>(UserTransaction.class);
+        transaction = new ThreadLocal<UserTransaction>();
         callback = new UserTransactionCallback() {
 
             @Override
@@ -106,12 +106,16 @@ public class SimpleTransactionManager {
      * @return User transaction.
      */
     public UserTransaction getTransaction() {
+        return new AutoCreateUserTransaction(this);
+    }
+    
+    UserTransaction getOrCreateThreadSpecificTransaction() { 
         UserTransaction tx = transaction.get();
         if (tx == null) {
             tx = factory.create(callback, resources);
             transaction.set(tx);
         }
-        return transaction.getProxy();
+        return tx;
     }
     
     /**
diff --git a/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleUserTransaction.java b/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleUserTransaction.java
index 1b9b59ac..ffa8925a 100644
--- a/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleUserTransaction.java
+++ b/test/enterprise/src/main/java/org/wamblee/test/transactions/SimpleUserTransaction.java
@@ -34,7 +34,7 @@ public class SimpleUserTransaction implements UserTransaction {
     private int status;
 
     /**
-     * Construcst the transaction.
+     * Constructs the transaction.
      * 
      * @param aCallback
      *            Callback to use.
-- 
2.31.1