Transaction logic in WebJPATestCase servlet masks root cause of test failures

The logic within the servlet used by this test case calls tx.commit() within a finally block. However, an exception is thrown from the try block, its exception handler triggers tx.setRollbackOnly(), which will cause the subsequent tx.commit() to fail, thus masking the original exception.