issue#293 : Always need to run CountDownLatch.countDown in finally block

The endLatch.countDown was only being run if exceptions were not thrown, which deadlocks tests that fail.

Signed-off-by: Peter Ansell <p_ansell@yahoo.com>

testsuites/lucene/src/main/java/org/eclipse/rdf4j/sail/lucene/AbstractLuceneSailTest.java

@@ -27,6 +27,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
+import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 
@@ -1051,6 +1052,7 @@ public void testMultithreadedAdd()
 		int numThreads = 3;
 		final CountDownLatch startLatch = new CountDownLatch(1);
 		final CountDownLatch endLatch = new CountDownLatch(numThreads);
+		final Set<Throwable> exceptions = ConcurrentHashMap.newKeySet();
 		for (int i = 0; i < numThreads; i++) {
 			new Thread(new Runnable() {
 				public void run() {
@@ -1062,15 +1064,22 @@ public void run() {
 									vf.createLiteral(i));
 						}
 					}
-					catch (InterruptedException e) {
+					catch (Throwable e) {
+						exceptions.add(e);
 						throw new AssertionError(e);
 					}
-					endLatch.countDown();
+					finally {
+						endLatch.countDown();
+					}
 				}
 			}).start();
 		}
 		startLatch.countDown();
 		endLatch.await();
+		assertEquals(0, exceptions.size());
+		for(Throwable e : exceptions) {
+			throw new AssertionError(e);
+		}
 	}
 
 	protected void assertQueryResult(String literal, IRI predicate, Resource resultUri)