You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pp...@apache.org on 2010/02/18 02:26:04 UTC
svn commit: r911240 -
/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/TestUnwrap.java
Author: ppoddar
Date: Thu Feb 18 01:26:04 2010
New Revision: 911240
URL: http://svn.apache.org/viewvc?rev=911240&view=rev
Log:
OPENJPA-1503: Set txn for rollback directly on illegal argument
Modified:
openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/TestUnwrap.java
Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/TestUnwrap.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/TestUnwrap.java?rev=911240&r1=911239&r2=911240&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/TestUnwrap.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/TestUnwrap.java Thu Feb 18 01:26:04 2010
@@ -76,21 +76,18 @@
public void testInvalidEntityManagerUnwrap() {
EntityManager em = emf.createEntityManager();
- Class<?>[] validCasts = new Class[] {
+ Class<?>[] invalidCasts = new Class[] {
Object.class,
Properties.class,
null,
};
- for (Class<?> c : validCasts) {
+ for (Class<?> c : invalidCasts) {
try {
em.unwrap(c);
fail("Expected to fail to unwarp with " + c);
- } catch (PersistenceException e) {
+ } catch (Exception e) {
EntityTransaction txn = em.getTransaction();
assertFalse(txn.isActive());
- } catch (Exception ex) {
- ex.printStackTrace();
- fail("Unexpected exception while unwrapping " + c);
}
}
}