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

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

      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.

        Acceptance criteria

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

                Created:
                Updated:
                Resolved: