Bubble up the exception rather than wrapping it in the unit test

Author: Dima1224

src/test/java/org/apache/commons/jxpath/util/ValueUtilsTest.java

@@ -26,6 +26,7 @@
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
+import java.util.concurrent.ExecutionException;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
@@ -96,27 +97,22 @@ void testGetValueFromSetTooSmall() {
     }
 
     @Test
-    void testGetDynamicPropertyHandlerConcurrently() throws InterruptedException {
+    void testGetDynamicPropertyHandlerConcurrently() throws InterruptedException, ExecutionException {
         // This test is to ensure that the dynamic property handler can be accessed concurrently
         // without throwing any exceptions. It does not assert any specific behavior, but rather
         // ensures that no exceptions are thrown during concurrent access.
         int nThreads = 200; // Number of threads to simulate concurrent access
         List<Future<?>> futures = new ArrayList<>();
         ExecutorService threadPool = Executors.newFixedThreadPool(nThreads);
         for (int i = 0; i < nThreads; i++) {
-           futures.add(threadPool.submit(() -> ValueUtils.getDynamicPropertyHandler(DummyHandler.class)));
+            futures.add(threadPool.submit(() -> ValueUtils.getDynamicPropertyHandler(DummyHandler.class)));
         }
 
         threadPool.shutdown();
         threadPool.awaitTermination(1, TimeUnit.SECONDS);
 
         for (Future<?> future : futures) {
-            try {
-                future.get(); // This will throw an exception if any thread encountered an issue
-            } catch (Exception e) {
-                // If an exception is thrown, the test fails
-                throw new AssertionError("Exception thrown during concurrent access", e);
-            }
+            future.get(); // This will throw an exception if any thread encountered an issue
         }
     }