Use assertThrows() in trivial cases in tests

h2/src/test/org/h2/test/db/TestCases.java

@@ -158,12 +158,7 @@ public class TestCases extends TestBase {
     private void testClearSyntaxException() throws SQLException {
         Connection conn = getConnection("cases");
         Statement stat = conn.createStatement();
-        try {
-            stat.execute("select t.x, t.x t.y from dual t");
-            fail();
-        } catch (SQLException e) {
-            assertEquals("42000", e.getSQLState());
-        }
+        assertThrows(42000, stat).execute("select t.x, t.x t.y from dual t");
         conn.close();
     }
 
@@ -240,24 +235,8 @@ public class TestCases extends TestBase {
         Connection conn = getConnection("selfreferential");
         Statement stat = conn.createStatement();
         stat.execute("create table sr(id integer, usecount integer as usecount + 1)");
-        check: {
-            try {
-                stat.execute("insert into sr(id) values (1)");
-            } catch (SQLException ex) {
-                assertEquals(ErrorCode.getState(ErrorCode.NULL_NOT_ALLOWED), ex.getSQLState());
-                break check;
-            }
-            fail("Exception expected");
-        }
-        check: {
-            try {
-                stat.execute("select max(id), usecount from sr");
-            } catch (SQLException ex) {
-                assertEquals(ErrorCode.getState(ErrorCode.MUST_GROUP_BY_COLUMN_1), ex.getSQLState());
-                break check;
-            }
-            fail("Exception expected");
-        }
+        assertThrows(ErrorCode.NULL_NOT_ALLOWED, stat).execute("insert into sr(id) values (1)");
+        assertThrows(ErrorCode.MUST_GROUP_BY_COLUMN_1, stat).execute("select max(id), usecount from sr");
         conn.close();
     }
 

h2/src/test/org/h2/test/db/TestFunctions.java

@@ -1211,28 +1211,13 @@ public class TestFunctions extends TestBase implements AggregateFunction {
         java.util.Date nowDate = c.getTime();
         assertEquals(nowDate, rs.getTimestamp(2));
 
-        try {
-            rs = stat.executeQuery("SELECT TRUNCATE('bad', 1) FROM dual");
-            fail("expected exception");
-        } catch (SQLException ex) {
-            // expected
-        }
+        assertThrows(SQLException.class, stat).executeQuery("SELECT TRUNCATE('bad', 1) FROM dual");
 
         // check for passing wrong data type
-        try {
-            rs = stat.executeQuery("SELECT TRUNCATE('bad') FROM dual");
-            fail("expected exception");
-        } catch (SQLException ex) {
-            // expected
-        }
+        rs = assertThrows(SQLException.class, stat).executeQuery("SELECT TRUNCATE('bad') FROM dual");
 
         // check for too many parameters
-        try {
-            rs = stat.executeQuery("SELECT TRUNCATE(1,2,3) FROM dual");
-            fail("expected exception");
-        } catch (SQLException ex) {
-            // expected
-        }
+        rs = assertThrows(SQLException.class, stat).executeQuery("SELECT TRUNCATE(1,2,3) FROM dual");
 
         conn.close();
     }

h2/src/test/org/h2/test/db/TestIndexHints.java

@@ -128,14 +128,8 @@ public class TestIndexHints extends TestBase {
 
     private void testWithInvalidIndexName() throws SQLException {
         Statement stat = conn.createStatement();
-        try {
-            stat.executeQuery("explain analyze select * " +
+        assertThrows(ErrorCode.INDEX_NOT_FOUND_1, stat).executeQuery("explain analyze select * " +
                 "from test use index(idx_doesnt_exist) where x=1 and y=1");
-            fail("Expected exception: "
-                    + "Index \"IDX_DOESNT_EXIST\" not found");
-        } catch (SQLException e) {
-            assertEquals(ErrorCode.INDEX_NOT_FOUND_1, e.getErrorCode());
-        }
     }
 
 }

h2/src/test/org/h2/test/db/TestLob.java

@@ -192,12 +192,7 @@ public class TestLob extends TestBase {
         Thread.sleep(100);
         // start a new transaction, to be sure
         stat.execute("delete from test");
-        try {
-            c1.getSubString(1, 3);
-            fail();
-        } catch (SQLException e) {
-            // expected
-        }
+        assertThrows(SQLException.class, c1).getSubString(1, 3);
         conn.close();
     }
 
@@ -641,12 +636,7 @@ public class TestLob extends TestBase {
         Statement stat;
         conn = getConnection("lob");
         stat = conn.createStatement();
-        try {
-            stat.execute("create memory table test(x clob unique)");
-            fail();
-        } catch (SQLException e) {
-            assertEquals(ErrorCode.FEATURE_NOT_SUPPORTED_1, e.getErrorCode());
-        }
+        assertThrows(ErrorCode.FEATURE_NOT_SUPPORTED_1, stat).execute("create memory table test(x clob unique)");
         conn.close();
     }
 

h2/src/test/org/h2/test/db/TestOutOfMemory.java

@@ -148,12 +148,7 @@ public class TestOutOfMemory extends TestBase {
         stat.execute("checkpoint");
         eatMemory(80);
         try {
-            try {
-                prep.execute();
-                fail();
-            } catch (SQLException e) {
-                assertEquals(ErrorCode.OUT_OF_MEMORY, e.getErrorCode());
-            }
+            assertThrows(ErrorCode.OUT_OF_MEMORY, prep).execute();
             assertThrows(ErrorCode.DATABASE_IS_CLOSED, conn).close();
             freeMemory();
             conn = null;

h2/src/test/org/h2/test/db/TestTriggersConstraints.java

@@ -165,14 +165,9 @@ public class TestTriggersConstraints extends TestBase implements Trigger {
         stat = conn.createStatement();
         stat.execute("drop table if exists test");
         stat.execute("create table test(id int)");
-        try {
-            stat.execute("create trigger test_insert before select on test " +
+        assertThrows(ErrorCode.TRIGGER_SELECT_AND_ROW_BASED_NOT_SUPPORTED, stat)
+                .execute("create trigger test_insert before select on test " +
                     "for each row call \"" + TestTriggerAdapter.class.getName() + "\"");
-            fail();
-        } catch (SQLException ex) {
-            assertEquals(ErrorCode.TRIGGER_SELECT_AND_ROW_BASED_NOT_SUPPORTED,
-                    ex.getErrorCode());
-        }
         conn.close();
     }
 

h2/src/test/org/h2/test/mvcc/TestMvcc3.java

@@ -79,12 +79,7 @@ public class TestMvcc3 extends TestBase {
         s1.execute("create unique index on test(name)");
         s1.executeUpdate("update test set name = 100 where id = 1");
 
-        try {
-            s2.executeUpdate("update test set name = 100 where id = 2");
-            fail();
-        } catch (SQLException e) {
-            // expected
-        }
+        assertThrows(SQLException.class, s2).executeUpdate("update test set name = 100 where id = 2");
 
         ResultSet rs = s1.executeQuery("select * from test order by id");
         assertTrue(rs.next());
@@ -268,7 +263,3 @@ public class TestMvcc3 extends TestBase {
     }
 
 }
-
-
-
-