tighten test conditions - do not ignore any exceptions

h2/src/test/org/h2/test/synth/TestConcurrentUpdate.java

@@ -8,10 +8,8 @@ package org.h2.test.synth;
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
-import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.Random;
-import org.h2.api.ErrorCode;
 import org.h2.test.TestBase;
 import org.h2.util.Task;
 
@@ -29,19 +27,26 @@ public class TestConcurrentUpdate extends TestBase {
      * @param a ignored
      */
     public static void main(String... a) throws Exception {
-        TestBase t = TestBase.createCaller().init();
-        t.config.memory = true;
-        t.test();
+        org.h2.test.TestAll config = new org.h2.test.TestAll();
+        config.memory = true;
+        config.multiThreaded = true;
+//        config.mvStore = false;
+//        config.mvcc = false;
+        System.out.println(config);
+        TestBase test = createCaller().init(config);
+        for (int i = 0; i < 10; i++) {
+            System.out.println("Pass #" + i);
+            test.config.beforeTest();
+            test.test();
+            test.config.afterTest();
+        }
     }
 
     @Override
     public void test() throws Exception {
-        if (!config.multiThreaded) {
-            return;
-        }
         deleteDb("concurrent");
         final String url = getURL("concurrent", true);
-        Connection conn = getConnection(url);
+        try (Connection conn = getConnection(url)) {
             Statement stat = conn.createStatement();
             stat.execute("create table test(id int primary key, name varchar)");
 
@@ -52,9 +57,9 @@ public class TestConcurrentUpdate extends TestBase {
                     @Override
                     public void call() throws Exception {
                         Random r = new Random(threadId);
-                    Connection conn = getConnection(url);
+                        try (Connection conn = getConnection(url)) {
                             PreparedStatement insert = conn.prepareStatement(
-                            "insert into test values(?, ?)");
+                                    "merge into test values(?, ?)");
                             PreparedStatement update = conn.prepareStatement(
                                     "update test set name = ? where id = ?");
                             PreparedStatement delete = conn.prepareStatement(
@@ -62,7 +67,6 @@ public class TestConcurrentUpdate extends TestBase {
                             PreparedStatement select = conn.prepareStatement(
                                     "select * from test where id = ?");
                             while (!stop) {
-                        try {
                                 int x = r.nextInt(ROW_COUNT);
                                 String data = "x" + r.nextInt(ROW_COUNT);
                                 switch (r.nextInt(3)) {
@@ -88,13 +92,9 @@ public class TestConcurrentUpdate extends TestBase {
                                         }
                                         break;
                                 }
-                        } catch (SQLException e) {
-                            handleException(e);
                             }
                         }
-                    conn.close();
                     }
-
                 };
                 tasks[i] = t;
                 t.execute();
@@ -112,24 +112,6 @@ public class TestConcurrentUpdate extends TestBase {
             for (Task t : tasks) {
                 t.get();
             }
-        conn.close();
-    }
-
-    /**
-     * Handle or ignore the exception.
-     *
-     * @param e the exception
-     */
-    void handleException(SQLException e) throws SQLException {
-        switch (e.getErrorCode()) {
-        case ErrorCode.CONCURRENT_UPDATE_1:
-        case ErrorCode.DUPLICATE_KEY_1:
-        case ErrorCode.ROW_NOT_FOUND_WHEN_DELETING_1:
-        case ErrorCode.LOCK_TIMEOUT_1:
-            break;
-        default:
-            throw e;
         }
     }
-
 }