svn commit: r1701596 - in /jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr: CompatibilityIssuesTest.java RepositoryTest.java query/QueryTest.java

[mr...@apache.org]

Author: mreutegg
Date: Mon Sep  7 10:17:09 2015
New Revision: 1701596

URL: http://svn.apache.org/r1701596
Log:
OAK-3356: Unclosed sessions in oak-jcr tests

Modified:
    jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/CompatibilityIssuesTest.java
    jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/RepositoryTest.java
    jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryTest.java

Modified: jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/CompatibilityIssuesTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/CompatibilityIssuesTest.java?rev=1701596&r1=1701595&r2=1701596&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/CompatibilityIssuesTest.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/CompatibilityIssuesTest.java Mon Sep  7 10:17:09 2015
@@ -273,6 +273,8 @@ public class CompatibilityIssuesTest ext
         } catch (InvalidItemStateException e) {
             assertTrue(e.getCause() instanceof CommitFailedException);
         }
+        s2.logout();
+        s3.logout();
     }
 
     private Session newSession(boolean refreshIntervalZero) throws RepositoryException {
@@ -448,6 +450,7 @@ public class CompatibilityIssuesTest ext
         QueryResult r = q.execute();
         RowIterator it = r.getRows();
         Assert.assertTrue(it.hasNext());
+        anonymousSession.logout();
     }
 
 }

Modified: jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/RepositoryTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/RepositoryTest.java?rev=1701596&r1=1701595&r2=1701596&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/RepositoryTest.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/RepositoryTest.java Mon Sep  7 10:17:09 2015
@@ -143,6 +143,7 @@ public class RepositoryTest extends Abst
         assertEquals(1, attributeNames.length);
         assertEquals(RepositoryImpl.REFRESH_INTERVAL, attributeNames[0]);
         assertEquals(42L, session.getAttribute(RepositoryImpl.REFRESH_INTERVAL));
+        session.logout();
     }
 
     @Test

Modified: jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryTest.java?rev=1701596&r1=1701595&r2=1701596&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryTest.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryTest.java Mon Sep  7 10:17:09 2015
@@ -727,6 +727,7 @@ public class QueryTest extends AbstractR
         NodeIterator ni = qr.getNodes();
         Node n = ni.nextNode();
         assertEquals("/etc/p2/r", n.getPath());
+        session.logout();
     }
 
     @Test
@@ -746,6 +747,7 @@ public class QueryTest extends AbstractR
         Node n = ni.nextNode();
         assertEquals("/etc/p1", n.getPath());
         assertFalse(ni.hasNext());
+        session.logout();
     }
 
     @Test
@@ -773,6 +775,7 @@ public class QueryTest extends AbstractR
         Node n = ni.nextNode();
         assertEquals("/one", n.getPath());
         assertFalse(ni.hasNext());
+        session.logout();
     }
 
 }