This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-dbutils.git
The following commit(s) were added to refs/heads/master by this push: new 67ce356 Format: "catch(" -> "catch (" 67ce356 is described below commit 67ce35692d06d98921a85314085e9981fedbb3fe Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Tue Apr 18 10:36:21 2023 -0400 Format: "catch(" -> "catch (" --- .../java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java | 6 +++--- src/test/java/org/apache/commons/dbutils/QueryLoaderTest.java | 2 +- src/test/java/org/apache/commons/dbutils/QueryRunnerTest.java | 10 +++++----- .../java/org/apache/commons/dbutils/ResultSetIteratorTest.java | 2 +- .../commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java | 2 +- 5 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java b/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java index 3272ceb..f890612 100644 --- a/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java +++ b/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java @@ -149,7 +149,7 @@ public class AsyncQueryRunnerTest { verify(prepStmt, times(1)).executeBatch(); verify(prepStmt, times(1)).close(); // make sure the statement is closed verify(conn, times(1)).close(); // make sure the connection is closed - } catch(final Exception e) { + } catch (final Exception e) { caught = true; } @@ -282,7 +282,7 @@ public class AsyncQueryRunnerTest { verify(results, times(1)).close(); verify(prepStmt, times(1)).close(); // make sure we closed the statement verify(conn, times(1)).close(); // make sure we closed the connection - } catch(final Exception e) { + } catch (final Exception e) { caught = true; } @@ -416,7 +416,7 @@ public class AsyncQueryRunnerTest { verify(prepStmt, times(1)).executeUpdate(); verify(prepStmt, times(1)).close(); // make sure we closed the statement verify(conn, times(1)).close(); // make sure we closed the connection - } catch(final Exception e) { + } catch (final Exception e) { caught = true; } diff --git a/src/test/java/org/apache/commons/dbutils/QueryLoaderTest.java b/src/test/java/org/apache/commons/dbutils/QueryLoaderTest.java index 49901b0..47f488c 100644 --- a/src/test/java/org/apache/commons/dbutils/QueryLoaderTest.java +++ b/src/test/java/org/apache/commons/dbutils/QueryLoaderTest.java @@ -46,7 +46,7 @@ public class QueryLoaderTest extends BaseTestCase { try { queryLoader.load("e"); fail("Expecting exception: IllegalArgumentException"); - } catch(final IllegalArgumentException e) { + } catch (final IllegalArgumentException e) { assertEquals("e not found.",e.getMessage()); assertEquals(QueryLoader.class.getName(), e.getStackTrace()[0].getClassName()); } diff --git a/src/test/java/org/apache/commons/dbutils/QueryRunnerTest.java b/src/test/java/org/apache/commons/dbutils/QueryRunnerTest.java index d408398..31f5682 100644 --- a/src/test/java/org/apache/commons/dbutils/QueryRunnerTest.java +++ b/src/test/java/org/apache/commons/dbutils/QueryRunnerTest.java @@ -163,7 +163,7 @@ public class QueryRunnerTest { verify(prepStmt, times(1)).executeBatch(); verify(prepStmt, times(1)).close(); // make sure the statement is closed verify(conn, times(1)).close(); // make sure the connection is closed - } catch(final SQLException e) { + } catch (final SQLException e) { System.out.println("[TEST] The following exception is expected:"); System.out.println(e); caught = true; @@ -306,7 +306,7 @@ public class QueryRunnerTest { verify(prepStmt, times(1)).close(); // make sure we closed the statement verify(results, times(1)).close(); verify(conn, times(1)).close(); // make sure we closed the connection - } catch(final SQLException e) { + } catch (final SQLException e) { caught = true; } @@ -498,7 +498,7 @@ public class QueryRunnerTest { verify(prepStmt, times(1)).executeUpdate(); verify(prepStmt, times(1)).close(); // make sure we closed the statement verify(conn, times(1)).close(); // make sure we closed the connection - } catch(final SQLException e) { + } catch (final SQLException e) { caught = true; } @@ -718,7 +718,7 @@ public class QueryRunnerTest { when(meta.getParameterCount()).thenReturn(2); runner.query("{call my_proc(?, ?)}", handler, params); - } catch(final SQLException e) { + } catch (final SQLException e) { caught = true; } @@ -948,7 +948,7 @@ public class QueryRunnerTest { when(meta.getParameterCount()).thenReturn(2); runner.execute("{call my_proc(?, ?)}", handler, params); - } catch(final SQLException e) { + } catch (final SQLException e) { caught = true; } diff --git a/src/test/java/org/apache/commons/dbutils/ResultSetIteratorTest.java b/src/test/java/org/apache/commons/dbutils/ResultSetIteratorTest.java index 491a378..d73c58e 100644 --- a/src/test/java/org/apache/commons/dbutils/ResultSetIteratorTest.java +++ b/src/test/java/org/apache/commons/dbutils/ResultSetIteratorTest.java @@ -61,7 +61,7 @@ public class ResultSetIteratorTest extends BaseTestCase { try { resultSetIterator.rethrow(sQLException); fail("Expecting exception: RuntimeException"); - } catch(final RuntimeException e) { + } catch (final RuntimeException e) { assertEquals(ResultSetIterator.class.getName(), e.getStackTrace()[0].getClassName()); } diff --git a/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java b/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java index 600b3e0..1aafa2e 100644 --- a/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java +++ b/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java @@ -482,7 +482,7 @@ public class SqlNullCheckedResultSetTest extends BaseTestCase { try { getUrlInt = ResultSet.class.getMethod("getURL", Integer.TYPE); getUrlString = ResultSet.class.getMethod("getURL", String.class); - } catch(final NoSuchMethodException | SecurityException e) { + } catch (final NoSuchMethodException | SecurityException e) { // ignore } if (getUrlInt != null && getUrlString != null) {