You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by st...@apache.org on 2021/04/06 11:15:55 UTC

[openjpa] 01/02: clean up unit test

This is an automated email from the ASF dual-hosted git repository.

struberg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/openjpa.git

commit 2d3c6e6ee0f5f8f7f73291f33e5580455522070f
Author: Mark Struberg <st...@apache.org>
AuthorDate: Fri Jan 15 13:11:31 2021 +0100

    clean up unit test
    
    no need to call em.persist inside the loop on the same entity over again.
---
 .../openjpa/persistence/compat/TestNonPrimaryKeyQueryParameters.java   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/compat/TestNonPrimaryKeyQueryParameters.java b/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/compat/TestNonPrimaryKeyQueryParameters.java
index f214d19..7c2f77c 100644
--- a/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/compat/TestNonPrimaryKeyQueryParameters.java
+++ b/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/compat/TestNonPrimaryKeyQueryParameters.java
@@ -135,9 +135,8 @@ public class TestNonPrimaryKeyQueryParameters extends SQLListenerTestCase {
             LineItem item = new LineItem(String.valueOf(i), 10);
             item.setInvoice(invoice);
             invoice.getLineItems().add(item);
-            em.persist(invoice);
         }
-        em.flush();
+        em.persist(invoice);
         tran.commit();
         em.close();
     }