Remove dead code in DomaPersistenceExceptionTranslator

gh-12

Author: kazuki43zoo

doma-spring-boot-autoconfigure/src/main/java/org/seasar/doma/boot/autoconfigure/DomaPersistenceExceptionTranslator.java

@@ -43,17 +43,6 @@ public DataAccessException translateExceptionIfPossible(RuntimeException ex) {
 			return null;
 		}
 
-		if (ex.getCause() instanceof SQLException) {
-			SQLException e = (SQLException) ex.getCause();
-			String sql = null;
-			if (ex instanceof SqlExecutionException) {
-				sql = ((SqlExecutionException) ex).getRawSql();
-			}
-			else if (ex instanceof UniqueConstraintException) {
-				sql = ((UniqueConstraintException) ex).getRawSql();
-			}
-			return translator.translate(ex.getMessage(), sql, e);
-		}
 		if (ex instanceof OptimisticLockException) {
 			return new OptimisticLockingFailureException(ex.getMessage(), ex);
 		}
@@ -72,6 +61,15 @@ else if (ex instanceof UnknownColumnException
 			return new TypeMismatchDataAccessException(ex.getMessage(), ex);
 		}
 
+		if (ex.getCause() instanceof SQLException) {
+			SQLException e = (SQLException) ex.getCause();
+			String sql = null;
+			if (ex instanceof SqlExecutionException) {
+				sql = ((SqlExecutionException) ex).getRawSql();
+			}
+			return translator.translate(ex.getMessage(), sql, e);
+		}
+
 		return new UncategorizedDataAccessException(ex.getMessage(), ex) {
 		};
 	}

doma-spring-boot-autoconfigure/src/test/java/org/seasar/doma/boot/autoconfigure/DomaPersistenceExceptionTranslatorTest.java

@@ -53,18 +53,6 @@ public void testOccurSqlExecutionException() {
 				is("select * from todo where todo_id = ?"));
 	}
 
-	@Test
-	public void testOccurUniqueConstraintException() {
-		DataAccessException dataAccessException = translator.translateExceptionIfPossible(
-				new UniqueConstraintException(SqlLogType.FORMATTED, SqlKind.INSERT,
-						"insert into todo (todo_id, title) values (?, ?)",
-						"insert into todo (todo_id, title) values ('000000001', 'Title')",
-						"TodoDao/insert.sql", new SQLException()));
-		assertThat(dataAccessException, is(instanceOf(UncategorizedSQLException.class)));
-		assertThat(UncategorizedSQLException.class.cast(dataAccessException).getSql(),
-				is("insert into todo (todo_id, title) values (?, ?)"));
-	}
-
 	@Test
 	public void testThrowOptimisticLockingFailureException() {
 		DataAccessException dataAccessException = translator.translateExceptionIfPossible(
@@ -82,7 +70,7 @@ public void testThrowDuplicateKeyException() {
 				new UniqueConstraintException(SqlLogType.FORMATTED, SqlKind.INSERT,
 						"insert into todo (todo_id, title) values (?, ?)",
 						"insert into todo (todo_id, title) values ('000000001', 'Title')",
-						"TodoDao/insert.sql", null));
+						"TodoDao/insert.sql", new SQLException()));
 		assertThat(dataAccessException, is(instanceOf(DuplicateKeyException.class)));
 	}