Uploaded image for project: 'Migration 4.4 to 4.5 (closed)'
  1. Migration 4.4 to 4.5 (closed)
  2. MGNLMIGRATION-322

Improve comparison of users/userGroups/userRoles workspace

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • 1.2.10
    • None
    • None

    Description

      info.magnolia.module.migrationtests.CompareConfigsTest.testCompareUsersWorkspace() currently has a lot of ignores specified:

      public void testCompareUsersWorkspace() throws Exception {
              assertTrue(compare("/", RepositoryConstants.USERS, new String[] {
                      "/system/superuser/lastaccess",
                      "/system/superuser/pswd",
                      "/system/anonymous/pswd",
              }));
          }
      

      Although ignoring of password and lastaccess properties is not so bad, bigger problem is with ids of roles which could be added in different order on fresh/updated instance:
      http://hudson.magnolia-cms.com/job/int_migration-tests_4.4-to-4.5.x-branch/ws/magnolia-migration-integration/diff-reports/users.html

      • same applies to (userGroups - switched assigned roles and userRoles - switched permissions)

      Implement comparison which handle such cases (parse out the report if no better idea). Please move such tests out of CompareConfigsTest to something like Compare*Workspace if it would require new implementation.

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                rsiska Robert Šiška
                rkovarik Roman Kovařík
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: