Merged HEAD-BUG-FIX (4.3/Cloud) to HEAD (4.3/Cloud)

59936: Added extra assertions (test pre-conditions) to help debug sporadic failure.


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@62234 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261

source/test-java/org/alfresco/repo/lock/mem/AbstractLockStoreTxTest.java

@@ -41,6 +41,7 @@ import org.junit.BeforeClass;
 import org.junit.Test;
 import org.springframework.context.ApplicationContext;
 import org.springframework.dao.ConcurrencyFailureException;
+import org.springframework.transaction.support.TransactionSynchronizationManager;
 
 /**
  * Integration tests that check transaction related functionality of {@link LockStore} implementations.
@@ -212,11 +213,17 @@ public abstract class AbstractLockStoreTxTest<T extends LockStore>
                     "jbloggs", expires, Lifetime.EPHEMERAL, null);
         
 
+        assertFalse("Transaction present, but should not be. Leak?",
+                    TransactionSynchronizationManager.isSynchronizationActive());
+        
         Thread thread2 = new Thread("Thread2")
         {
             @Override
             public void run()
             {
+                assertFalse("Transaction present, but should not be. Leak?",
+                            TransactionSynchronizationManager.isSynchronizationActive());
+                
                 Object main = AbstractLockStoreTxTest.this;
                 try
                 {