CCM NG: All tests in ccm-core now pass again

git-svn-id: https://svn.libreccm.org/ccm/ccm_ng@5888 8810af33-2d31-482b-a856-94f89814c4df

ccm-core/src/main/java/org/libreccm/security/GroupImExporter.java

@@ -27,6 +27,7 @@ import java.util.Set;
 
 import javax.enterprise.context.RequestScoped;
 import javax.inject.Inject;
+import javax.persistence.EntityManager;
 import javax.transaction.Transactional;
 
 /**
@@ -35,7 +36,10 @@ import javax.transaction.Transactional;
  */
 @RequestScoped
 @Processes(Group.class)
-public class GroupImExporter extends AbstractEntityImExporter<Group>{
+public class GroupImExporter extends AbstractEntityImExporter<Group> {
+    
+    @Inject
+    private EntityManager entityManager;
     
     @Inject
     private GroupRepository groupRepository;
@@ -50,7 +54,8 @@ public class GroupImExporter extends AbstractEntityImExporter<Group>{
     protected void saveImportedEntity(final Group entity) {
         
         entity.setPartyId(0);
-        groupRepository.save(entity);
+//        groupRepository.save(entity);
+        entityManager.persist(entity);
     }
     
     @Override

ccm-core/src/main/java/org/libreccm/security/PermissionManager.java

@@ -367,6 +367,7 @@ public class PermissionManager implements Serializable {
             permission.setObject(object);
             permission.setInherited(true);
             permission.setInheritedFrom(inheritedFrom);
+            permission.setUuid(UUID.randomUUID().toString());
 
             entityManager.persist(permission);
 
@@ -405,6 +406,7 @@ public class PermissionManager implements Serializable {
             permission.setGrantee(grantee);
             permission.setGrantedPrivilege(privilege);
             permission.setObject(null);
+            permission.setUuid(UUID.randomUUID().toString());
 
             entityManager.persist(permission);
         }

ccm-core/src/main/java/org/libreccm/security/UserImExporter.java

@@ -27,6 +27,7 @@ import java.util.Set;
 
 import javax.enterprise.context.RequestScoped;
 import javax.inject.Inject;
+import javax.persistence.EntityManager;
 import javax.transaction.Transactional;
 
 /**
@@ -37,6 +38,9 @@ import javax.transaction.Transactional;
 @Processes(User.class)
 public class UserImExporter extends AbstractEntityImExporter<User> {
 
+    @Inject
+    private EntityManager entityManager;
+    
     @Inject
     private UserRepository userRepository;
 
@@ -51,7 +55,8 @@ public class UserImExporter extends AbstractEntityImExporter<User> {
 
         // Reset partyId.
         entity.setPartyId(0);
-        userRepository.save(entity);
+//        userRepository.save(entity);
+        entityManager.persist(entity);
     }
 
     @Override

ccm-core/src/test/java/org/libreccm/imexport/GroupImportTest.java

@@ -81,9 +81,9 @@ public class GroupImportTest {
                                                              + "/org.libreccm.imexport.GroupImportTest"
                                                          + "/ccm-export.json";
     private static final String IMPORT_GROUPS_TOC_SOURCE = "/imports"
-                                                              + "/org.libreccm.imexport.GroupImportTest"
+                                                               + "/org.libreccm.imexport.GroupImportTest"
-                                                          + "/org.libreccm.security.Group"
+                                                           + "/org.libreccm.security.Group"
-                                                          + "/org.libreccm.security.Group.json";
+                                                           + "/org.libreccm.security.Group.json";
     private static final String IMPORT_DATA_SOURCE = "/imports"
                                                          + "/org.libreccm.imexport.GroupImportTest"
                                                      + "/org.libreccm.security.Group"
@@ -93,7 +93,7 @@ public class GroupImportTest {
     private static final String CCM_TESTS_DIR = TMP_DIR + "/ccm-tests";
     private static final String IMPORTS_DIR = CCM_TESTS_DIR + "/imports";
     private static final String GROUP_IMPORT_TEST_DIR = IMPORTS_DIR
-                                                           + "/org.libreccm.imexport.GroupImportTest";
+                                                            + "/org.libreccm.imexport.GroupImportTest";
     private static final String IMPORT_DATA_DIR = GROUP_IMPORT_TEST_DIR
                                                       + "/org.libreccm.security.Group";
 
@@ -281,9 +281,11 @@ public class GroupImportTest {
 
     @Test
     @UsingDataSet("datasets/org/libreccm/imexport/GroupImportTest/data.yml")
-    @ShouldMatchDataSet(value = "datasets/org/libreccm/imexport/GroupImportTest"
+    @ShouldMatchDataSet(
-                                    + "/after-import-single-group.yml",
+        excludeColumns = {"party_id"},
-                        excludeColumns = {"party_id"}
+        orderBy = {"groups.group_id", "parties.party_id"},
+        value = "datasets/org/libreccm/imexport/GroupImportTest"
+                    + "/after-import-single-group.yml"
     )
     @InSequence(200)
     public void importSingleGroup() {

ccm-core/src/test/java/org/libreccm/imexport/UserImportTest.java

@@ -280,15 +280,18 @@ public class UserImportTest {
 
     @Test
     @UsingDataSet("datasets/org/libreccm/imexport/UserImportTest/data.yml")
-    @ShouldMatchDataSet(value = "datasets/org/libreccm/imexport/UserImportTest"
+    @ShouldMatchDataSet(
-                                    + "/after-import-single-user.yml",
+        excludeColumns = {"party_id"},
-                        excludeColumns = {"party_id"}
+        orderBy = {"users.user_id", "parties.party_id"},
+        value = "datasets/org/libreccm/imexport/UserImportTest"
+                    + "/after-import-single-user.yml"
     )
     @InSequence(200)
     public void importSingleUser() {
 
-        shiro.getSystemUser().execute(() -> 
+        shiro.getSystemUser().execute(()
-            importExport.importEntities("org.libreccm.imexport.UserImportTest")
+            -> importExport.importEntities(
+                "org.libreccm.imexport.UserImportTest")
         );
     }
 

ccm-core/src/test/java/org/libreccm/security/PermissionManagerTest.java

@@ -179,9 +179,11 @@ public class PermissionManagerTest {
     @UsingDataSet(
         "datasets/org/libreccm/security/PermissionManagerTest/data.yml")
     @ShouldMatchDataSet(
+        excludeColumns = {"permission_id", "uuid"},
+        orderBy = {"permissions.permission_id"},
         value = "datasets/org/libreccm/security/PermissionManagerTest/"
-                    + "after-grant.yml",
+                    + "after-grant.yml"
-        excludeColumns = {"permission_id"})
+    )
     @InSequence(200)
     public void grantPermission() {
         final Role role2 = roleRepository.findByName("role2").get();
@@ -226,9 +228,11 @@ public class PermissionManagerTest {
     @UsingDataSet("datasets/org/libreccm/security/PermissionManagerTest/"
                       + "data-recursivly.yml")
     @ShouldMatchDataSet(
+        excludeColumns = {"permission_id", "uuid"},
+        orderBy = {"permissions.permission_id"},
         value = "datasets/org/libreccm/security/PermissionManagerTest/"
-                    + "after-grant-recursivly.yml",
+                    + "after-grant-recursivly.yml"
-        excludeColumns = {"permission_id"})
+        )
     @InSequence(211)
     public void grantPermissionRecursively() {
         final Role role1 = roleRepository.findByName("role1").get();
@@ -688,7 +692,7 @@ public class PermissionManagerTest {
     @ShouldMatchDataSet(
         value = "datasets/org/libreccm/security/PermissionManagerTest/"
                     + "after-copy.yml",
-        excludeColumns = {"permission_id"})
+        excludeColumns = {"permission_id", "uuid"})
     @InSequence(400)
     public void copyPermissions() throws Throwable {
         final CcmObject object2 = ccmObjectRepository.findById(20002L).get();